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

lyra: conan v2 support. #13969

Merged
merged 3 commits into from
Nov 4, 2022
Merged

lyra: conan v2 support. #13969

merged 3 commits into from
Nov 4, 2022

Conversation

grafikrobot
Copy link
Contributor

No description provided.

@grafikrobot
Copy link
Contributor Author

FYI.. I got cmake related errors in the test package when testing locally (with conan 1) even though I essentially copied the test package template from https://github.com/conan-io/conan-center-index/blob/master/docs/package_templates/header_only/all/test_package/conanfile.py

@conan-center-bot

This comment has been minimized.

@mmatisko
Copy link
Contributor

mmatisko commented Nov 2, 2022

FYI.. I got cmake related errors in the test package when testing locally (with conan 1) even though I essentially copied the test package template from https://github.com/conan-io/conan-center-index/blob/master/docs/package_templates/header_only/all/test_package/conanfile.py

Consider removing following lines from test_package/CMakeLists.txt:

include(${CMAKE_BINARY_DIR}/conanbuildinfo.cmake)  
conan_basic_setup(TARGETS)

as you don't use legacy cmake generator there, which generates file conanbuildinfo.cmake with conan_basic_setup() command. That's the reason why this command is unknown while testing library package:

CMake Error at CMakeLists.txt:5 (conan_basic_setup):
  Unknown CMake command "conan_basic_setup".

@grafikrobot
Copy link
Contributor Author

you don't use legacy cmake generator there, which generates file conanbuildinfo.cmake with conan_basic_setup

Aha, of course! Thanks for pointing that out.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline

All green in build 3 (858dee2d1cc7a1fe37bfa869d834573c5f54773b):

  • lyra/1.0.0@:
    All packages built successfully! (All logs)

  • lyra/1.1.0@:
    All packages built successfully! (All logs)

  • lyra/1.2.0@:
    All packages built successfully! (All logs)

  • lyra/1.3.0@:
    All packages built successfully! (All logs)

  • lyra/1.5.0@:
    All packages built successfully! (All logs)

  • lyra/1.5.1@:
    All packages built successfully! (All logs)

  • lyra/1.4.0@:
    All packages built successfully! (All logs)

  • lyra/1.6.0@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit df53606 into conan-io:master Nov 4, 2022
@grafikrobot grafikrobot deleted the lyra branch November 7, 2022 22:01
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.

5 participants