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

CMake Fetch content #220

Merged
merged 5 commits into from
Jan 25, 2021
Merged

CMake Fetch content #220

merged 5 commits into from
Jan 25, 2021

Conversation

9prady9
Copy link
Member

@9prady9 9prady9 commented Jan 25, 2021

Change(s)

Current Behavior

Examples CMake file has OpenGL linking logic which isn't necessary if library transitive linking flags are set appropriately.

New Behavior
Examples CMake file doesn't call find_package(OpenGL ...)

Breaking Change(s)
No

Please check if the PR fulfills these requirements

  • Guidelines
  • [ ] Relevant Examples are added/modified
  • [ ] Relevant Documentation has been added/updated

Includes using custom FindOpenGL for building the library since
there is no way to ensure the FindOpenGL module available on the
system is from required cmake version.
@9prady9 9prady9 force-pushed the fetch_content branch 5 times, most recently from 928b155 to c168299 Compare January 25, 2021 11:47
@9prady9 9prady9 force-pushed the fetch_content branch 11 times, most recently from 067a2f4 to cab9e00 Compare January 25, 2021 15:56
@9prady9 9prady9 force-pushed the fetch_content branch 12 times, most recently from f4cdd47 to 3c0a7a6 Compare January 25, 2021 22:08
Also added freeimage location status message in cmake

Corrected ci badges in README also
@9prady9 9prady9 changed the title Fetch content CMake Fetch content Jan 25, 2021
@9prady9 9prady9 merged commit f146e26 into arrayfire:master Jan 25, 2021
@9prady9 9prady9 deleted the fetch_content branch January 25, 2021 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant