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

googletest library: download hash not matching anymore. #1168

Open
hzeller opened this issue Feb 1, 2022 · 3 comments
Open

googletest library: download hash not matching anymore. #1168

hzeller opened this issue Feb 1, 2022 · 3 comments
Labels
build system matters pertaining to building Verible

Comments

@hzeller
Copy link
Collaborator

hzeller commented Feb 1, 2022

The googletest release mentioned in the WORKSPACE just changed the content:
https://github.com/google/googletest/archive/refs/tags/release-1.11.0.zip

This used to have sha256sum of 353571c2440176ded91c2de6d6cd88ddd41401d14692ec1f99e35d013feda55a, now it is 63afcb1690b2139b2d9c7ad1df3de87697530fb1fd6869622cff26c77a3b0cfb.
This could be benign (e.g. github was changing their zip library and repackaged things), but is worth investigating first before changing our expected hash.

@hzeller hzeller added the build system matters pertaining to building Verible label Feb 1, 2022
@hzeller hzeller changed the title googletest library just changed content googletest library: download hash not matching anymore. Feb 1, 2022
@mglb
Copy link
Collaborator

mglb commented Feb 1, 2022

Recursive diff (diff -r -u new/googletest-release-1.11.0 fromcache/googletest-release-1.11.0) of unpacked zips didn't find any differences.

353571c2440176ded91c2de6d6cd88ddd41401d14692ec1f99e35d013feda55a  fromcache/release-1.11.0.zip
63afcb1690b2139b2d9c7ad1df3de87697530fb1fd6869622cff26c77a3b0cfb  new/release-1.11.0.zip

EDIT:
Diff of zipinfo -v outputs shows change in DOS last modification timestamp of every file from 10:42:26 to 17:42:26 (which is now equal to UTC timestamp).

@hzeller
Copy link
Collaborator Author

hzeller commented Feb 1, 2022

Thanks for the investigation, Mariusz!
I'll update the hash in the WORKSPACE.

hzeller added a commit to hzeller/verible that referenced this issue Feb 1, 2022
The sha256sum hash of
https://github.com/google/googletest/archive/refs/tags/release-1.11.0.zip

Changed from
353571c2440176ded91c2de6d6cd88ddd41401d14692ec1f99e35d013feda55a  old/release-1.11.0.zip
63afcb1690b2139b2d9c7ad1df3de87697530fb1fd6869622cff26c77a3b0cfb  new/release-1.11.0.zip

Investigation by @mglb revealed that there is no change in content for any file inside
the archive, just timestamps changed. Now why that was happening is a different question,
but the change is benign so ok to update our expectation of the hash.

Fixes chipsalliance#1168

Signed-off-by: Henner Zeller <h.zeller@acm.org>
@hzeller
Copy link
Collaborator Author

hzeller commented Feb 1, 2022

hzeller added a commit to hzeller/verible that referenced this issue Feb 1, 2022
Context:
Due to some glitch, the content hash of the googletest zip
changed (see chipsalliance#1168).

It looks like the cause was rolled back...
https://twitter.com/tgummerer/status/1488493440103030787

But now our CI caches are discombobulated. Change their names
to force a fresh re-build.

Signed-off-by: Henner Zeller <h.zeller@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system matters pertaining to building Verible
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants