conan-package-tools icon indicating copy to clipboard operation
conan-package-tools copied to clipboard

[build=missing/outdated] don't seem work sometimes

Open solvingj opened this issue 6 years ago • 8 comments

I don't have time to put together a reproduction right now, but maybe others have experienced the same issue.

When working on this repo, [build=missing] worked once and skipped all the builds properly. Then, I added more pagination on travis, did a push, and it started to perform all the builds from scratch.

double travis pages again due to timeout, travis taking longer now [skip appveyor] [build=missing]

https://travis-ci.org/bincrafters/conan-boost_level14group/builds/447744670

solvingj avatar Oct 29 '18 11:10 solvingj

In the linked build you canceled the job when still was retrieving the recipes, and in the previous one it was still downloading the binary packages but I can't find any build. Take into account that the test_package will be run always, or by downloading the package or by building it.

lasote avatar Nov 02 '18 07:11 lasote

I excluded args from CPT some commits but it was replaced by build_policy many time ago. Also Grafikrobot reported a similar error when running Conan directly. Maybe this problem is no related to CPT but to Conan. :thinking:

uilianries avatar Nov 02 '18 11:11 uilianries

Related issue: https://github.com/conan-io/conan/issues/3901

uilianries avatar Nov 02 '18 17:11 uilianries

From my personal experience it seems never to work.

Croydon avatar Nov 15 '18 13:11 Croydon

@Croydon some log or example, we could check? we couldn't reproduce it so far.

lasote avatar Nov 15 '18 13:11 lasote

https://github.com/conan-community/conan-openssl/pull/77

Croydon avatar Nov 15 '18 15:11 Croydon

Maybe the PR doesn't manage correctly the TRAVIS_COMMIT_MESSAGE/APPVEYOR_REPO_COMMIT_MESSAGE env vars. Would be nice if some of you could you print these vars and confirm that the issue is there.

lasote avatar Nov 19 '18 13:11 lasote

By the way, Probably it is the correct behavior for PRs, I doubt we want to skip builds in a CI depending on the user's commit. Could be dangerous.

lasote avatar Nov 19 '18 13:11 lasote