Package: developers-reference Version: 3.3.7 Severity: minor On Fri, Feb 03, 2006 at 01:23:24PM +0100, Romain Beauxis wrote: > On Thu, 2 Feb 2006 23:49:42 -0500, Justin Pryzby wrote > > This could be priority: extra, unless something not in extra will > > depend on it. > > Well ,according to policy: > "extra > This contains all packages that conflict with others with required, > important, standard or optional priorities, or are only likely to be useful > if you already know what they are or have specialized requirements." > > I don't see well why this should apply to this package.. Okay, up to you. I think people often leave the default priority when it could be bumped back, without really giving it any thought.
> > Build-Depends: @cdbs@ > > I think this is RC and not allowed.. Build-Depends have to be > > available before the build starts, and mustn't be modified by > > standard targets: http://release.debian.org/etch_rc_policy.txt But I > > might be wrong, and it might be okay as written. > > Hum... > This is the meaning of this: > # In order to regenerate 'debian/control' : > # DEB_AUTO_UPDATE_DEBIAN_CONTROL=yes fakeroot debian/rules clean > # Then check manually if everything's ok > > debian/control.in is only used when manualy invoking it, and never during a > buildd. This the way I did for other packages that are allready in the > archive, like linuxdcpp or mediawiki... Okay, then I guess it is okay. Sorry for the confusion; I don't know the first thing about cdbs (though it does make a nice short rules file!). > > ./debian/watch > > Please consider using the sf.net format now documented. > Done! Nope. Check uscan.1; the sf.net format actually uses http://sf.net/, and is rewritten internally by uscan. I don't know if your method would fail, but it is unintentional, and more ugly :) For more info: http://lists.debian.org/debian-mentors/2005/12/msg00035.html Also, your ./debian/dirs is now exactly one line long. If I were using a non-cdbs debhelper, I would remove the file, and put the dir as an argument to dh_installdirs; but, I don't know if you can do that with cdbs. Since you "repackaged" the .orig tarball [0] (right?), could you make it extract to ./php-getid3-1.7.5.orig/ instead, as documented in the DevRef? [0] because upstream has a .zipball only [1] http://www.us.debian.org/doc/developers-reference/ch-best-pkging-practices.en.html#s-repackaged origtargz 6.7.8.2; the anchor for this is broken; it has a space, which doesn't work for me as a +, thus the bug. Probably hasn't been found before only because since the TOC doesn't link to it .. which IMO it should.. Justin -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
