On Thu, Apr 07, 2005 at 01:13:44PM -0400, Jay Bonci wrote:
> Package: devscripts
> Version: 2.8.14
> Severity: minor
> Tags: patch
> 
> Hello there.
> A minor flaw in uscan is that it gives uninitialized value warnings when
> considering this watchfile:
> 
> version=2
> http://www.cpan.org/modules/by-module/Curses/Curses-([0-9].*?)\.t(ar
> \.)?gz \
>  debian uupdate

You're right - thanks for the patch.  Now fixed in SVN.

However, there's a big bug in your watchfile: read the output
carefully:

> libcurses-perl: Newer version (1.12.) available on remote site:
>   http://www.cpan.org/modules/by-module/Curses/Curses-1.12.tgz
>   (local version is 1.11)

The newer version is recorded as "1.12.", not "1.12".  This is
because, from the manpage:

       Having  got  a list of `files' matching the pattern, their version num-
       bers are extracted by  treating  the  part  matching  the  Perl regexp
       groups,  demarcated  by  `(...)', joining them with `.' as a separator,
       and using the result as the version number of the  file.

So you have matched "1.12" and "", giving "1.12.".  And if the file
were called "Curses-1.12.tar.gz" your version number would be
"1.12.ar."  Certainly not what you want!  Try this instead:

http://www.cpan.org/modules/by-module/Curses/Curses-([0-9].*?)\.t(?:ar\.)?gz \

which uses clustering, not capturing, for the ar\. bit (see
perlre(1)).  (Incidentally, you could replace .*? with .* without
there being a difference, since the part after it has to match the end
of the URL name.)

   Julian


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to