Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 579738 - =sci-physics/pythia-6.4.28 Upstream source URL changed
Summary: =sci-physics/pythia-6.4.28 Upstream source URL changed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Physics related packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-12 13:07 UTC by Oliver Freyermuth
Modified: 2016-04-14 00:51 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Freyermuth 2016-04-12 13:07:34 UTC
It's now to be found at: 
https://root.cern.ch/download/pythia6.tar.gz
instead of 
ftp://root.cern.ch/root/pythia6.tar.gz
as can be seen in the upstream bug report (c.f. See Also).
Comment 1 Guilherme Amadio gentoo-dev 2016-04-13 13:56:56 UTC
Fixed. Thanks for reporting!
Comment 2 Andrew Savchenko gentoo-dev 2016-04-13 14:47:25 UTC
Not really... They not only changed location, but tarball checksum, size and content, keeping the same file name, of course :/

I'm now validating build with new tarball.
Comment 3 Guilherme Amadio gentoo-dev 2016-04-13 16:13:34 UTC
(In reply to Andrew Savchenko from comment #2)
> Not really... They not only changed location, but tarball checksum, size and
> content, keeping the same file name, of course :/
> 
> I'm now validating build with new tarball.

Oops, sorry. I may have had the cached file in my $DISTDIR, so checksum did not change. In that case (same filename) how is the distfile file updated in the mirrors?
Comment 4 Oliver Freyermuth 2016-04-13 22:36:41 UTC
(In reply to Andrew Savchenko from comment #2)
> Not really... They not only changed location, but tarball checksum, size and
> content, keeping the same file name, of course :/

Oh my, I also missed that, thanks for checking! 
I alerted upstream (just to make sure they realize that silently changing an archive meant for distribution without bumping the version is a no-go) in the linked report. 

I also did a quick comparison of the old vs. the new tarfile. Two files have been changed: 
makePythia6.linuxx86_64 now uses gfortran instead of g77. 
makePythia6.macosx64 has been added. 

Thanks again!
Comment 5 Andrew Savchenko gentoo-dev 2016-04-14 00:51:29 UTC
(In reply to Guilherme Amadio from comment #3)
> (In reply to Andrew Savchenko from comment #2)
> > Not really... They not only changed location, but tarball checksum, size and
> > content, keeping the same file name, of course :/
> > 
> > I'm now validating build with new tarball.
> 
> Oops, sorry. I may have had the cached file in my $DISTDIR, so checksum did
> not change. In that case (same filename) how is the distfile file updated in
> the mirrors?

Our mirrors just continue to distribute distfiles because new one has different checksum. There are two ways to fix this:

1) Make revision bump and save offending tarball to a separate name using "->" operator. This is how problem is fixed now: pythia-6.4.28-r1: https://root.cern.ch/download/pythia6.tar.gz -> pythia6-20160413.tar.gz.

2) Delete tarball from the local distfiles _and_ corresponding entry from the Manifest, update SRC_URI, then regenerate manifest. But this will break older ebuild which is still there and may disrupt our mirroring, so I preferred a version bump aproach.

(In reply to Oliver Freyermuth from comment #4)
> (In reply to Andrew Savchenko from comment #2)
> > Not really... They not only changed location, but tarball checksum, size and
> > content, keeping the same file name, of course :/
> 
> Oh my, I also missed that, thanks for checking! 
> I alerted upstream (just to make sure they realize that silently changing an
> archive meant for distribution without bumping the version is a no-go) in
> the linked report. 

Looks like they made this change years ago, but we noticed only now due to SRC_URI move.

> I also did a quick comparison of the old vs. the new tarfile. Two files have
> been changed: 
> makePythia6.linuxx86_64 now uses gfortran instead of g77. 
> makePythia6.macosx64 has been added. 

Yeah, this is nothing critical to our build process here, but due to distfiles collision I had to revbump.