Skip to content

Conversation

@lexming
Copy link
Contributor

@lexming lexming commented Mar 6, 2025

Verification of checksums can fail for sources of git repos in non-reproducible format (i.e. .tar.gz). This is known, but it can be confusing when those fail as there is no clue that this is the reason:

== FAILED: Installation ended unsuccessfully: Checksum verification for /user/brussel/101/vsc10122/easybuild/install/skylake/sources/p/polars/jsonpath_lib-0.3.0-24eaf0b4416edff38a4d1b6b17bc4b9f3f047b4b.tar.gz using {'jsonpath_lib-0.3.0-24eaf0b4416edff38a4d1b6b17bc4b9f3f047b4b.tar.gz': 'dfb5eb7f47b5b5a7b35262f7b41787a785958001a23ff039cdae926396c6af96'} failed. (took 3 secs)
== Results of the build can be found in the log file(s) /tmp/eb-ubv_m0gz/easybuild-polars-0.20.2-20250306.134946.tPBiI.log
ERROR: Installation of polars-0.20.2-gfbf-2023a.eb failed: "Checksum verification for /user/brussel/101/vsc10122/easybuild/install/skylake/sources/p/polars/jsonpath_lib-0.3.0-24eaf0b4416edff38a4d1b6b17bc4b9f3f047b4b.tar.gz using {'jsonpath_lib-0.3.0-24eaf0b4416edff38a4d1b6b17bc4b9f3f047b4b.tar.gz': 'dfb5eb7f47b5b5a7b35262f7b41787a785958001a23ff039cdae926396c6af96'} failed."

Unfortunately, it is not trivial to add specific information in that error, because at that point the knowledge that the failed tarball is from a git repo is already lost.

So this PR adds a big warning message whenever such a tarball is found in the list of sources.

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel merged commit fda1fa2 into easybuilders:5.0.x Mar 7, 2025
37 checks passed
@lexming lexming deleted the warn-nonreprod-sources branch March 7, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants