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

[bug] Conan 2.8.0 SHA1 differs from 2.7.1 #17125

Closed
mzukovec opened this issue Oct 7, 2024 · 6 comments
Closed

[bug] Conan 2.8.0 SHA1 differs from 2.7.1 #17125

mzukovec opened this issue Oct 7, 2024 · 6 comments
Assignees
Labels
stage: triaging staled The issue has been inactive for a while and will be closed soon

Comments

@mzukovec
Copy link

mzukovec commented Oct 7, 2024

Describe the bug

I'm using conan (Windows) in our production environment and the upgrade to the 2.8.0 fails to find one of the packages listed in conanfile.py due do different SHA1. We're using explicit revisions.

Explicit SHA1(Get-FileHash -Algorithm SHA1 conaninfo.txt) results in B692E48396E2CC29EE7048A4BE2DFDA907940859

Command:

Note that I'm using the same profiles and conanfile.py in both cases.

conan install . --profile default --profile:host wasm_Release --profile:build default

2.7.1 results in package:
vtk/7.1.1@xlab/wasm#7c3b7a01ee895d518cd31dfea7c6e5b3:b692e48396e2cc29ee7048a4be2dfda907940859

2.8.0 results in package:
vtk/7.1.1@xlab/wasm#7c3b7a01ee895d518cd31dfea7c6e5b3:e117c1d7a5f53b1e1b366d69168614542e4c3138

How to reproduce it

No response

@memsharded memsharded self-assigned this Oct 7, 2024
@memsharded
Copy link
Member

Hi @mzukovec

Thanks for your report.

I think it is very likely that this is the result of a bug fix in #16987

It seems there were cases in which the requires("dep/version", package_id_mode="xxxx") was being ignored, resulting in a buggy package_id, the bug fix in #16987 fixed it, but the result of applying the right package_id_mode is a new package_id that needs to be built.

You can follow some discussion about this that happened in #17086. If it is the same case, you should be able to see with conan list the previous (incorrect) package_id, and with Conan 2.8 you should see the correct one.

Can you please check that and let us know? Thanks for your feedback.

@mzukovec
Copy link
Author

mzukovec commented Oct 7, 2024

package_id_mode="urelated" is indeed the case of why this is happening. I'll migrate to the vendor=True as discussed in the issue that you linked.

@memsharded
Copy link
Member

package_id_mode="urelated" is indeed the case of why this is happening. I'll migrate to the vendor=True as discussed in the issue that you linked.

Sounds good, if you really wanted to achieve that "decoupling" effect of the vendor=True feature with the unrelated mode. The important bit is to understand it is a trade-off between decoupling the dependencies of an application-like package that fully vendors the dependencies and getting the exact information of when that application-like package needs to be re-built from source because some of its dependencies changed (correct package_id computation with the correct package-id-mode (not unrelated)).

I'll leave the ticket open then, please let us know how it goes with the vendor=True feature.

@memsharded
Copy link
Member

Hi @mzukovec

Any further feedback here? Did you try the vendor=True feature?

If not, I think we can close the issue as not a bug, as it was really a bugfix that affected some incorrectly created package-ids.

@memsharded memsharded added the staled The issue has been inactive for a while and will be closed soon label Dec 3, 2024
@mzukovec
Copy link
Author

mzukovec commented Dec 4, 2024

Hi @memsharded

After some investigation we decided that vendoring may not be an ideal option, so we rebuilt the packages with two conan versions in order to support compatibility with all versions.

I’ll keep it in mind though.

Thanks for your help and insights.

@memsharded
Copy link
Member

Thanks to you for your feedback @mzukovec !

I am then going to close this ticket as resolved, but please re-open or create new tickets for any further question or issue you might have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stage: triaging staled The issue has been inactive for a while and will be closed soon
Projects
None yet
Development

No branches or pull requests

2 participants