Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[patch] Fix SDL build on Haiku #1310

Closed
SDLBugzilla opened this issue Feb 10, 2021 · 0 comments
Closed

[patch] Fix SDL build on Haiku #1310

SDLBugzilla opened this issue Feb 10, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 2.1
Reported for operating system, platform: Haiku, x86_64

Comments on the original bug report:

On 2014-01-15 17:44:56 +0000, Alexander von Gluck IV wrote:

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

On 2014-01-15 17:46:25 +0000, Alexander von Gluck IV wrote:

Created attachment 1522
patch to remove all <be/*> includes

On 2014-02-28 18:42:06 +0000, Ryan C. Gordon wrote:

Comment on attachment 1522
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.

On 2018-08-07 21:35:16 +0000, Ryan C. Gordon wrote:

This got applied at some point. (although the video functions are still BE_funcname instead of HAIKU_funcname...)

--ryan.

On 2018-08-07 22:57:32 +0000, Ryan C. Gordon wrote:

(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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant