Skip to content

fixing the build-requires#81

Draft
memsharded wants to merge 1 commit into
feature/ci_with_build_requiresfrom
fix/ci_with_build_requires
Draft

fixing the build-requires#81
memsharded wants to merge 1 commit into
feature/ci_with_build_requiresfrom
fix/ci_with_build_requires

Conversation

@memsharded
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

@michaelmaguire michaelmaguire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @memsharded thanks for this. We tried it out and in our sandbox it did indeed fix the issue we saw with ERROR: Build-require 'cmake' cannot be found in lockfile errors during the conan install build-order build as raised in conan-io/conan#8205 (comment)

Question: Earlier in conan-io/conan#8205 (comment) we reported seeing similar errors like:

ERROR: Build-require 'gtest' cannot be found in lockfile

and

ERROR: Build-require 'cmake' cannot be found in lockfile

but these were happening before we got into the conan install prove-out build of the build-orders. I.e. before the conan lock create --reference=app1/0.1@user/testing in your example.

Do we need to use --build=missing earlier on too here, in the conan lock create conanfile.py --name=libb --version=0.2 --base and conan lock create conanfile.py --name=libb at https://github.com/conan-io/examples/blob/master/features/lockfiles/ci/build.py#L73 and https://github.com/conan-io/examples/blob/master/features/lockfiles/ci/build.py#L84 ?

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.

2 participants