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

Support Conan 2 #212

Closed
nilanjan opened this issue Apr 24, 2023 · 9 comments · Fixed by #249
Closed

Support Conan 2 #212

nilanjan opened this issue Apr 24, 2023 · 9 comments · Fixed by #249
Labels
enhancement New feature or request pr-welcome

Comments

@nilanjan
Copy link

nilanjan commented Apr 24, 2023

I am trying to enable conan and facing the following error. Also can I get conan_extra_options?

-- Conan: Adding conancenter remote repository (https://center.conan.io/) verify ssl (True)
-- Configuring incomplete, errors occurred!

Upvote & Fund

  • I am using Polar.sh so you can upvote and help fund this issue. The funding is received once the issue is completed & confirmed by you.

  • Thank you in advance for helping prioritize & fund our backlog.

Fund with Polar
@aminya aminya added the more-information-needed The issue needs for information label Apr 24, 2023
@aminya
Copy link
Owner

aminya commented Apr 24, 2023

This cannot be reproduced inside CI. Can you provide a minimal reproducible example? What version of Conan do you have installed?

@nilanjan
Copy link
Author

I am on macOS ventura. Conan version 2.0.4. Conan is installed n a python virtual env. No conan profile is setup.
Screenshot 2023-04-23 at 9 33 15 PM

@nilanjan
Copy link
Author

MacOS 13.3.1 on M1 Max machine.

@FeignClaims
Copy link
Contributor

FeignClaims commented Apr 25, 2023

The official cmake support for conan 2.0 is experimental (see the README in conan-io/cmake-conan). Also, this repo hasn't update to the conan 2.0 version.

The way you use conan 2.0 with this repo for now, is to disable it in project_options options and run conan separately (see conan 2.0 tutorial for help if you need). I found this way is more useful (for now), as conan will generate CMakeUserPresets.json for you, so multiple toolchains can be configured easily by simply adding a conan profile.

My conan 2.0 template might help, but I documented it badly.

@aminya aminya changed the title conan remote fails on macOS Ventura Support Conan 2 Apr 25, 2023
@aminya aminya added enhancement New feature or request pr-welcome and removed more-information-needed The issue needs for information labels Apr 25, 2023
@nilanjan
Copy link
Author

@FeignClaims It was really helpful for most of the things other than google test. Do you have setup for the google test somewhere?

@FeignClaims
Copy link
Contributor

FeignClaims commented Apr 27, 2023

@FeignClaims Do you have setup for the google test somewhere?

@nilanjan By running conan install . -b missing, conan should install google test for you. I configured it in conanfile.py as test_requires in the section build_requirements(), which is a dedicated place to define tool_requires and test_requires.

@aminya
Copy link
Owner

aminya commented Jul 21, 2023

Conan 2 support is still experimental. Not sure if we should start adding it to project_options yet. There are also some issues with the latest released version (See #184)

Windows tests with LLVM and MSVC fail to build. Seems to be related to:
conan-io/cmake-conan#459
conan-io/cmake-conan#455
conan-io/cmake-conan#454

@aminya aminya pinned this issue Nov 14, 2023
@aminya
Copy link
Owner

aminya commented Nov 14, 2023

Is there anyone interested in integrating CMake Conan 2? Contributions are welcome.

@FeignClaims
Copy link
Contributor

@aminya conan 1 adds support for clang 17 in 1.62.0 now, so the test projects in this repo should work after updating setup-cpp.

I'm willing to integrate conan 2, but I'm afraid I won't have enough time until next month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pr-welcome
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants