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

Install the config #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

xsacha
Copy link

@xsacha xsacha commented Mar 16, 2019

Install config for cpuinfo so that it can found with find_package(cpuinfo) and used with target_link_libraries(... cpuinfo::cpuinfo)

@Maratyszcza
Copy link
Contributor

Could you explain the details? Also, which CMake version did you target (cpuinfo currently targets CMake 2.8, but we can update it to 3.5. Going above 3.5, however, would be hard as some cpuinfo users target 3.5)

@xsacha
Copy link
Author

xsacha commented Mar 17, 2019

Definitely not above 3.5. I think 3.0.

@xsacha xsacha force-pushed the hunter-0.0.0-d5e37ad branch from eb570f1 to e574dc1 Compare March 17, 2019 22:12
@xsacha xsacha force-pushed the hunter-0.0.0-d5e37ad branch from e574dc1 to 868bd11 Compare March 17, 2019 23:20
@xsacha
Copy link
Author

xsacha commented Mar 17, 2019

Just a note: If you set the Travis CI to use the version of CMake you desire, then you never need to worry about compatibility -- it would fail to compile.

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.

3 participants