On Tue, Mar 26, 2013 at 07:33:10PM +0000, Thorsten Glaser wrote:
> Bart Martens dixit:
> 
> >What would be bogus/broken/wrong about them ?
> 
> cvs misses one mangle that's needed due to a DEHS bug,

No it doesn't.  And your watch file won't see versions above 1.12.13, while the
separate watch file fixes that.

  |  version=3
  |  opts="dversionmangle=s/^[0-9]+://;s/\+real$//" \
  |  http://ftp.gnu.org/non-gnu/cvs/source/feature/1.12.13/ 
cvs-([0-9.]*)\.tar\.gz

  |  version=3
  |  opts=dversionmangle=s/^(1\.12\.13)\+real$/$1/ \
  |  
http://ftp.gnu.org/non-gnu/cvs/source/feature/(\d+(?:\.\d+)*)/cvs[\-\._]?(\d\S*)\.(?:tar\.xz|txz|tar\.bz2|tbz2|tar\.gz|tgz)

> and both have absolutely insane file extension matches that, in the case of
> mksh, definitely will never happen.

It does happen.  Upstream did change the file extension between versions 40 and
41.  Checking for only one file extension is a common mistake, see:
http://wiki.debian.org/debian/watch#Common_mistakes

Regards,

Bart Martens


-- 
To UNSUBSCRIBE, email to debian-qa-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20130326203304.gc8...@master.debian.org

Reply via email to