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

The --install command line argument isn't available in the earlier CMake versions #9850

Closed
yowidin opened this issue Oct 22, 2021 · 3 comments
Assignees

Comments

@yowidin
Copy link

yowidin commented Oct 22, 2021

I'm trying to check out the new and blessed packaging approach and having an issue with the cmake.install() call inside the package() method on older systems.

My recipe works fine with the cmake version 3.17.3 and breaks when used with cmake version 3.13.4.
Of course replacing the cmake.install() call with the cmake.build(target='install') call fixes the issue.

The --install command line argument isn't available in earlier versions of the CMake, it was probably introduced only in CMake 3.15 please consider changing the call back to a build call with the install target.

Originally posted by @yowidin in #9668 (comment)

@memsharded memsharded self-assigned this Oct 22, 2021
@memsharded
Copy link
Member

Hi @yowidin

yes, this is expected. All the new integrations work with CMake > 3.15, this was discussed and approved by the Tribe: conan-io/tribe#4

We need to set a base version to be able to test and develop correctly, and we settled for 3.15. Please upgrade CMake to use the modern integrations (and also for Conan 2.0) (you might use conan install cmake/3.15... -g VirtualRunEnv to install it)

@yowidin
Copy link
Author

yowidin commented Oct 22, 2021

Okay, but the documentation is a bit outdated on that front:

https://docs.conan.io/en/latest/reference/conanfile/tools/cmake/cmake.html#install:

Equivalent to run cmake --build . --target=install

@yowidin
Copy link
Author

yowidin commented Oct 22, 2021

Closing the issue, since the change is intentional.

@yowidin yowidin closed this as completed Oct 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants