Hi Adam, On Sat, Feb 14, 2009 at 18:39, Adam D. Barratt <a...@adam-barratt.org.uk> wrote:
>> opts=uversionmangle="s/^\d\.\d$/$&00/;s/^\d\.\d\d0$/$&0" \ >> http://search.cpan.org/dist/Term-Size/ \ >> ..*/Term-Size-v?(\d[\d_.]+)\.(?:tar(?:\.gz|\.bz2)?|tgz|zip) > > Out of interest, where did that package come from? The version of > libterm-size-perl in the archive (0.2-4) doesn't appear to contain any > mangling options in its watch file. It's from the SVN repo on alioth at svn+ssh://svn.debian.org/svn/pkg-perl/trunk/libterm-size-perl > uscan *is* following the documented behaviour, it's just not checking > for errors when applying the mangling. A quick bit of debug printing > added to the code shows that the regex is being split by semicolons in > to the two portions you'd expect - "s/^\d\.\d$/$&00/ and s/^\d\.\d\d0 > $/$&0". The two portions are then being passed through Perl's eval() in > order to apply the mangling to the version string. > > However, there's no check as to whether the eval threw errors or not. > I'm not sure whether they're being intentionally or accidentally ignored > as yet, but they are being ignored. In the case of the sample watch > file you provided, afaics neither of the mangle options affects the > version number (currently 0.207), so the package is successfully > downloaded anyway. > > I've been through the lintian lab on gluck and that contains only one > packages that has syntax broken in the way you describe, which I'll file > a bug about. uscan does, however, apply the mangling as documented for > that package - i.e. the mangling the maintainer intended to apply fails > due to the eval() failing. I see, so the only problem then is that it doesn't report the errors. You could then interpret this bug report as a wishlist to at least print a message with the eval error. Thanks, Tincho. -- Martín Ferrari -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
