2009/1/29 Paul Wise <p...@debian.org>: > On Fri, Jan 30, 2009 at 12:24 AM, Jordi Gutiérrez Hermoso > <jord...@gmail.com> wrote: > >>> patches/add-soname seems to change the whitespace in the definition of SRC, >>> why? >> >> Why not? It fits into 80 columns that way. :-) > > Just a gratuitous change, also unrelated to the purpose of the patch.
Okay... undone. >>> patches/add-soname needs some more metadata: upstream status, author, >>> location it was taken from (if it was taken from somewhere). >> >> Done. Where is this requirement documented? > > Nowhere, it is just common sense really. I guess my sense is uncommon, then. *shrug* >>> tab character before the Source: package name in debian/control >> >> Hm, I don't see this... > > Look at the diff.gz with less. I don't see it there either. Heisenbug. >>> extra lines in debian/watch debian/*dirs debian/*install debian/docs >> >> Removed, but why does this matter? > > Doesn't matter, just slightly sloppy. ... I liked at least one of those whitespaces, the one in debian/watch, because it separates the version line from the actual content, but ok, I removed it. Okay, uploaded again, addressing all the things above. Oh, I forgot one thing. The Perl version. Does it matter if it's 5.10? Should I try to track down the earliest Perl version that works for this script? 5.10 is lenny, wouldn't backporters have to backport to lenny anyways? - URL: http://mentors.debian.net/debian/pool/main/q/qrupdate - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/q/qrupdate/qrupdate_1.0-1.dsc - Jordi G. H. -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org