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

Fixed Build Behavior on Mac OSX Snow Leopard #8

Closed
wants to merge 1 commit into from
Closed

Fixed Build Behavior on Mac OSX Snow Leopard #8

wants to merge 1 commit into from

Conversation

chmp
Copy link
Contributor

@chmp chmp commented Jul 31, 2011

On OSX libpng is installed in a non standard directory
(/usr/X11). The png package included in cmake takes care
of this.
Additionally fltk currently has to be built as 386 on
Snow Leopard, this then requires stage to be built as
386 as well.
This commit addopts the CMakeLists file to ensure the
correct behavior with OSX Snow Leopard.

 on osx libpng is installed in a non standard directory
 (/usr/X11) the png package included in cmake takes care
 of this.
 Additionally fltk currently has to be built as 386 on
 Snow Leopard, this then requires stage to be built as
 386 as well.
 This commit addopts the CMakeLists file to ensure the
 correct behavior with OSX Snow Leopard.
@rtv
Copy link
Owner

rtv commented Aug 5, 2011

I've committed a more radical overhaul of the CMake build scripts that makes things a lot more simple. Needs to be tested on Snow Leopard and Linux, but works for me on 10.7 with homebrew. I used your suggestions - thanks very much! The problem with MacPorts for a while only building the 32 bit FLTK should be worked around outside of the Stage build system. I left your suggested fix in the code but commented out.

@rtv rtv closed this Aug 5, 2011
@chmp
Copy link
Contributor Author

chmp commented Aug 5, 2011

Thanks. Works perfectly on Snow Leopard after enabling the 32bit forcing. I'm also using homebrew with the standard formula for fltk. Maybe the fault really lies with OSX.

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

Successfully merging this pull request may close these issues.

2 participants