[request-tracker-maintainers] RT command-by-mail

[Niko Tyni]

On Thu, Mar 22, 2007 at 10:02:25PM +0000, Matthew Johnson wrote:

> This unpacks into the directory RT-Extension-CommandByMail-0.05, rather
> than librt-extension-commandbymail-perl-0.05, which is what I would
> expect from the deb. Is it correct to untar, mv the directory to the
> correct name and then retar (which I have done in the current draft); or
> is this not neccessary?

It's not necessary. There's magic in dpkg-source for this. See
the Developer's Reference:

 http://www.debian.org/doc/developers-reference/ch-best-pkging-practices.en.html#s-bpp-origtargz

> New (renamed and now signed) packages are at
> http://mjj29.matthew.ath.cx/debian-upload/librt-extension-commandbymail-perl/

Great, it looks good to me now.

I'm not sure if it's optimal to depend on request-tracker3.4 |
request-tracker3.6 . This rules out people with non-packaged RT
installations, which is probably quite common. (Well, they could use
equivs or something, but they probably don't.)  Maybe 

 Enhances: request-tracker3.4 (>= 3.4.6), request-tracker3.6 (>= 3.6.1)

(which is pretty much the reality anyway)?

A few minor things you might want to consider:

The comments at the end of debian/copyright look like leftovers from
a template.

There's an empty /usr/lib/perl5 directory in the .deb, which is a bit
ugly. The customary way around this is

    rmdir --ignore-fail-on-non-empty --parents $(TMP)/usr/lib/perl5

See eg. http://pkg-perl.alioth.debian.org/rules-template-noxs

An arguably cleaner way to remove Makefile.old is 'make distclean'.

Good luck finding a sponsor. If you want to join the Request Tracker
group and put it in the maintainer field, I'm happy with that,
but (obviously) I can't speak for Toni about uploading the package.

Cheers,
-- 
Niko Tyni   ntyni at iki.fi