On Fri, 11 Feb 2005, Bluefuture wrote:

> Package: diffutils
> Severity: wishlist
> Tags: patch
> 
> For helping Dehs qa project (http://dehs.alioth.debian.org) i send
> the link for watch file to include in your package:
>
> version=2
> ftp://ftp.gnu.org/pub/gnu/diffutils/diffutils-?_?([\w+\d+\.]+|\d+)(\.tar|\.tgz)(\.gz|\.bz2|)
>  debian uupdate

Ok, this is ugly for several reasons. Here are my suggestions:

* You should try to limit the lines in your messages to 80 chars or less.

* When referring to ftp.gnu.org, the "pub" part should probably be dropped.
Most of the software announced in the info-gnu mailing list does not have
the "pub" part, which is a symlink to the current directory.
(Yes, I'll change the copyright file accordingly).

* The proposed watch file references "uupdate". I think we should not
assume automatically that the maintainer want to use such tool.

* I agree that the pattern should be "tight enough so that it matches
only the upstream software you are interested in", as uscan(1) says,
but the proposed regexp is unnecessarily confusing. I'll use something
like this, which should work as well:

version=2
ftp://ftp.gnu.org/gnu/diffutils/diffutils-(.*)\.tar\.gz


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

Reply via email to