Bug#915061: gap: improved watchfile [patch]

2018-11-29 Thread Mattia Rizzolo
On Fri, Nov 30, 2018 at 12:20:28AM +0100, Bill Allombert wrote:
> Hello Mattia, the actual Debian tarball is generated by
> debian/tarball, so if debian/watch mangles the version 
> then debian/tarball must be changed not to do it.

oh, sorry.  I clearly didn't look around the package.  However I believe
you wouldn't need to modify anything in that file even after my proposed
change.  However looking at that file I couldn't help but think that
what you do that is nowadays suggested to do though a Files-Excluded
field in d/copyright following copyright-format 1.0 (something that gap
is not using, and could potentially be quite a big task to adopt, even
if I recommend it).

What my changes do is only to mangle the internal view that uscan uses
to compare against the version used upstream, i.e. even after that
change it would still download a file named e.g. 1.10.0.

> It seems to me there might be some benefit to your change that you are
> not communicating.

I was only thinking of that because I noticed the message "A new
upstream version is available" on the tracker, that I believe is always
present with the watchfile as it is now, even when the software is fully
up to date.  Whereas I think that with my changes uscan would finally be
able to properly compare the debian version against upstream and such
you wouldn't needlessly have such message (same for the DDPO, of
course).

-- 
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540  .''`.
more about me:  https://mapreri.org : :'  :
Launchpad user: https://launchpad.net/~mapreri  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-


signature.asc
Description: PGP signature


Bug#915061: gap: improved watchfile [patch]

2018-11-29 Thread Bill Allombert
On Thu, Nov 29, 2018 at 11:46:58PM +0100, Mattia Rizzolo wrote:
> Source: gap
> Severity: wishlist
> Tags: patch
> 
> Dear maintainer,
> 
> I notice that the watchfile used in the gap pacakge is very simple and
> doesn't really do anything about the different versioning used between
> upstream and debian.

Hello Mattia, the actual Debian tarball is generated by
debian/tarball, so if debian/watch mangles the version 
then debian/tarball must be changed not to do it.

It seems to me there might be some benefit to your change that you are
not communicating.

Cheers,
-- 
Bill. 

Imagine a large red swirl here. 



Bug#915061: gap: improved watchfile [patch]

2018-11-29 Thread Mattia Rizzolo
Source: gap
Severity: wishlist
Tags: patch

Dear maintainer,

I notice that the watchfile used in the gap pacakge is very simple and
doesn't really do anything about the different versioning used between
upstream and debian.

For this, I'm proposing here a line that would make uscan correctly
compare the versions, mangling the debian version to change the 'r' and
'p' into dots.

version=3
options=dversionmangle=s/(\d+)r(\d+)p(\d+)/$1.$2.$3/ \
https://www.gap-system.org/pub/gap/gap4core/gap-(\d+\.\d+\.\d+)-core.zip debian

I haven't changed anything, just added the line in the middle doing the
mangling.


Hope this can be of use.

-- 
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540  .''`.
more about me:  https://mapreri.org : :'  :
Launchpad user: https://launchpad.net/~mapreri  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-


signature.asc
Description: PGP signature