After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 788779 - Add the ability to override tarball names
Add the ability to override tarball names
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2017-10-10 14:56 UTC by Emmanuele Bassi (:ebassi)
Modified: 2017-10-10 16:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add the ability to override tarball names (3.43 KB, patch)
2017-10-10 14:56 UTC, Emmanuele Bassi (:ebassi)
none Details | Review
Add the ability to override tarball names (5.32 KB, patch)
2017-10-10 15:16 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review

Description Emmanuele Bassi (:ebassi) 2017-10-10 14:56:18 UTC
New WebKit dependencies come from GitHub in the form of tarballs, and GitHub automatically generates release tarballs with spectacularly unhelpful names.
Comment 1 Emmanuele Bassi (:ebassi) 2017-10-10 14:56:29 UTC
Created attachment 361250 [details] [review]
Add the ability to override tarball names

Modules hosted on GitHub have autogenerated tarball names coming from
the release tag. This means a lot of them have unhelpful names like
'v1.0.1.tar.gz', or '296.zip'.

If two different modules have the same tag, we end up with a collision
when trying to check that the tarball downloaded have the same checksum.

We can add a simple XML attribute that allows us to rename the tarball
if that's the case; we can also throw in variables support for the name
of the module and the version, which will hopefully help us to avoid
collisions.
Comment 2 Emmanuele Bassi (:ebassi) 2017-10-10 15:16:20 UTC
Created attachment 361252 [details] [review]
Add the ability to override tarball names

Modules hosted on GitHub have autogenerated tarball names coming from
the release tag. This means a lot of them have unhelpful names like
'v1.0.1.tar.gz', or '296.zip'.

If two different modules have the same tag, we end up with a collision
when trying to check that the tarball downloaded have the same checksum.

We can add a simple XML attribute that allows us to rename the tarball
if that's the case; we can also throw in variables support for the name
of the module and the version, which will hopefully help us to avoid
collisions.
Comment 3 Michael Catanzaro 2017-10-10 15:51:10 UTC
Comment on attachment 361252 [details] [review]
Add the ability to override tarball names

r=me if Gabriel has tested it
Comment 4 Gabriel Ivașcu 2017-10-10 16:05:21 UTC
Review of attachment 361252 [details] [review]:

OK, tested. It works perfectly.

Thanks for working on this, ebassi!
Comment 5 Gabriel Ivașcu 2017-10-10 16:18:11 UTC
Attachment 361252 [details] pushed as ef5f40c - Add the ability to override tarball names