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

Don't search for FLUID when searching for FLTK #49

Merged
merged 2 commits into from
Apr 3, 2015
Merged

Conversation

richmattes
Copy link
Contributor

Stage doesn't need FLUID to be available during build time, but the
CMake FindFLTK package searches for it by default in newer versions.
This patch removes the requirement for FLUID to be present.

Stage doesn't need FLUID to be available during build time, but the
CMake FindFLTK package searches for it by default in newer versions.
This patch removes the requirement for FLUID to be present.
This commit fixes the pkgconfig file so that opengl and fltk
libraries and include directories are added to stage.pc, so that
third-party packages can use stage.pc to link with stage without any
other code.

This commit also adds stage-config and stage-config-version cmake module
files, so that find_package(stage) will work.  These cmake modules also
include the opengl and fltk libraries and include directories, so that
third-party packages can link against stage without any other code.
@richmattes
Copy link
Contributor Author

I just added another commit that fixes stage.pc.in so that it properly exports the link flags for fltk and opengl libraries. It also adds the files relevant to create a Stage cmake module so that third parties can use find_package(stage)

@rtv
Copy link
Owner

rtv commented Apr 3, 2015

Merged: thanks!

rtv added a commit that referenced this pull request Apr 3, 2015
Don't search for FLUID when searching for FLTK
@rtv rtv merged commit 3d0c4bd into rtv:master Apr 3, 2015
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