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

Disable glfw only when it is not found and not external. #558

Merged
merged 1 commit into from Jun 23, 2018
Merged

Disable glfw only when it is not found and not external. #558

merged 1 commit into from Jun 23, 2018

Conversation

yurivict
Copy link
Contributor

No description provided.

@raysan5 raysan5 merged commit b1d3810 into raysan5:master Jun 23, 2018
@yurivict yurivict changed the title Fisable glfw only when it is not found and not external. Disable glfw only when it is not found and not external. Jun 23, 2018
@a3f
Copy link
Contributor

a3f commented Jun 29, 2018

What is wrong with the old behavior? This breaks USE_EXTERNAL_GLFW=IF_POSSIBLE (Link against GLFW if installed systemwide, otherwise use the version embedded in Repository), which the Travis CI build currently supplies.

find_package(glfw3 3.2.1 QUIET) in utils.cmake should set glfw3_FOUND if it's available and requested (either by USE_EXTERNAL_GLFW=ON or USE_EXTERNAL_GLFW=IF_POSSIBLE), so I am not sure what this patch fixes.

@a3f a3f mentioned this pull request Jun 29, 2018
@yurivict
Copy link
Contributor Author

The point is that disabling should be in sync wit its presence, and it wasn't.

a3f added a commit to a3f/raylib that referenced this pull request Jul 3, 2018
Fixes: e1e036a ("Disable glfw only when it is not found and not external", raysan5#558)
a3f added a commit to a3f/raylib that referenced this pull request Jul 3, 2018
Fixes: e1e036a ("Disable glfw only when it is not found and not external", raysan5#558)
a3f added a commit that referenced this pull request Jul 3, 2018
Fixes: e1e036a ("Disable glfw only when it is not found and not external", #558)
@a3f
Copy link
Contributor

a3f commented Jul 3, 2018

The point is that disabling should be in sync wit its presence, and it wasn't.

Could you give me an example where this matters?

@yurivict
Copy link
Contributor Author

yurivict commented Jul 3, 2018

For external glfw, you disable glfw when it is found.

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.

None yet

3 participants