We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 2349 - [patch] Fix SDL build on Haiku
Summary: [patch] Fix SDL build on Haiku
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: HG 2.1
Hardware: x86_64 Haiku
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-15 17:44 UTC by Alexander von Gluck IV
Modified: 2018-08-07 22:57 UTC (History)
0 users

See Also:

Attachments
patch to remove all <be/*> includes (3.11 KB, patch)
2014-01-15 17:46 UTC, Alexander von Gluck IV 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander von Gluck IV 2014-01-15 17:44:56 UTC 
SDL references a lot of BeOS headers in be/*, this isn't very portable and is unneeded as be/*/* are all in the gcc search paths.

This patch makes SDL more portable and compile equally well on BeOS vs Haiku
Comment 1 Alexander von Gluck IV 2014-01-15 17:46:25 UTC 
Created attachment 1522 [details]
patch to remove all <be/*> includes
Comment 2 Ryan C. Gordon 2014-02-28 18:42:06 UTC 
Comment on attachment 1522 [details]
patch to remove all <be/*> includes

I don't know, I'm a little nervous about losing the "namespace" on those includes (especially the "String.h" one, even if Haiku's filesystem is case-sensitive).

--ryan.
Comment 3 Ryan C. Gordon 2018-08-07 21:35:16 UTC 
This got applied at some point. (although the video functions are still BE_funcname instead of HAIKU_funcname...)

--ryan.
Comment 4 Ryan C. Gordon 2018-08-07 22:57:32 UTC 
(In reply to Ryan C. Gordon from comment #3)
> This got applied at some point. (although the video functions are still
> BE_funcname instead of HAIKU_funcname...)

This is fixed in https://hg.libsdl.org/SDL/rev/09999a0e8f9a. Resolving bug!

--ryan.