[Python-apps-team] [RFS] ttb/1.1-4 ITA

Jeroen Schot schot at A-Eskwadraat.nl
Wed Apr 25 14:05:09 UTC 2012


Hello Sandro,

Thanks for your review. I have tried to address all your remarks.

On Fri, Apr 06, 2012 at 01:32:48PM +0200, Sandro Tosi wrote:
> here some notes:
> 
> debian/control
> - the short description should be something like:
>   <pkg> is (a) <short description>
>   in your case, this doesn't work, please rephrase; also is should
> start lowercase

Rephrased to '(Dutch) teletekst browser for the desktop'.
 
> debian/patches/01-small_update.diff
> - what is this patch about? "some bugfixes and small improvements" is
> not exactly a descriptive text.. is it backported from upstream?
> Please try to use DEP-3 fields, f.e. for Origin, Forwarded, etc etc.
> link: http://dep.debian.net/deps/dep3/ . You can also consider
> packaging a upstream repo snapshot, instead of patching with new
> upstream code.

I decided to pack an upstream repo snapshot and removed the patch.

> debian/README.source
> - you don't need this if you're using 3.0 (quilt) source format, so
> please remove it.

Yes, removed.
 
> debian/ttb.1
> - have you forwarded upstream?

Send upstream, no response (yet).

The updated package can be found at http://mentors.debian.net/package/ttb:

dget -x http://mentors.debian.net/debian/pool/main/t/ttb/ttb_1.0.1+20101115-1.dsc

Kind regards,

Jeroen Schot



More information about the Python-apps-team mailing list