[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: dish



OoO  En cette  soirée bien  amorcée du  jeudi 29  mai 2008,  vers 22:04,
Dimitar Ivanov <drimiks@mirendom.net> disait:

> Thanks for your suggestions and help. I've resolved all issues and
> uploaded a new package.

> The package can be found on mentors.debian.net:
> - URL: http://mentors.debian.net/debian/pool/main/d/dish
> - Source repository: deb-src http://mentors.debian.net/debian unstable
> main contrib non-free
> - dget http://mentors.debian.net/debian/pool/main/d/dish/dish_1.16.1-2.dsc

> I would be glad if someone uploaded this package for me.

Hi Dimitar!

Sorry  for being  picky, but  you left  configure-stamp target  which is
useless too since you don't have configure target any more.

This  is a  matter  of taste,  but  I would  put "config_examples/*"  in
debian/examples instead of "config_examples".

Moreover,  you should  fix all  lintian  warnings (use  lintian -viI  on
.changes). For manual page, you can either use a patch management system
or use this snippet in install target of debian/rules:
 sed -i 's/\B-/\\-/g' debian/dish/usr/share/man/man1/*

Check what this command really does before using it in debian/rules.
-- 
I WILL NOT AIM FOR THE HEAD
I WILL NOT AIM FOR THE HEAD
I WILL NOT AIM FOR THE HEAD
-+- Bart Simpson on chalkboard in episode 8F13

Attachment: pgpctd0XIUrw1.pgp
Description: PGP signature


Reply to: