On Thu, Jul 21, 2011 at 8:38 AM, Thomas Preud'homme <robo...@celest.fr> wrote: > Le jeudi 21 juillet 2011 01:29:09, Vincent Cheng a écrit : >> (I sent out a RFS for gecrit a week ago and have yet to receive a >> single reply, so I'm giving this a second try.) > Sorry, forgot to reply. >> >> Dear mentors, > Hi Vincent, >> >> I am looking for a sponsor for my package "gecrit". > [SNIP] >> >> * Package name : gecrit >> Version : 2.7-1 >> Upstream Author : Groza Cristian <kristi9...@gmail.com> >> * URL : http://sourceforge.net/projects/gecrit/ >> * License : GPL-3+ >> Section : editors >> >> It builds these binary packages: >> gecrit - simple, easy-to-use Python IDE >> >> The package appears to be lintian clean. >> >> The upload would fix these bugs: 633420 >> >> My motivation for maintaining this package is: gEcrit is a relatively >> new and promising Python editor/IDE that I hope will be of use to >> other Debian users. >> >> The package can be found on mentors.debian.net: >> - URL: http://mentors.debian.net/debian/pool/main/g/gecrit >> - Source repository: deb-src http://mentors.debian.net/debian unstable >> main contrib non-free >> - dget http://mentors.debian.net/debian/pool/main/g/gecrit/gecrit_2.7-1.dsc >> >> I would be glad if someone uploaded this package for me. > Below are my remarks but note that I'm just DM and hence can't sponsor your > package.
Regardless, thanks for taking the time to review my packaging! > Things to improve: > * Your orig.tar.gz has a different md5 than the upstream tarball. It looks like upstream released a newer source tarball under the same name; I've re-uploaded my packaging to mentors.d.n with this new tarball. > * Vcs-Svn should use anonscm link Fixed. > * Dependency on ${python:Depends} is for python modules, not python programs > and hence is useless > * Same thing for X-Python-Version, it's for python modules > > Neutral comments: > dpkg-gencontrol generate a warning about shlibs:Depends [0] but I'm not sure > it's a good idea to remove the ${shlibs:Depends}. A future version of the > package could needs shared library so you can ignore this warning. > > [0] dpkg-gencontrol: warning: Depends field of package gecrit: unknown > substitution variable ${shlibs:Depends} I suppose I'll keep ${shlibs:Depends}, then. > On the good sides: > * No lintian warning, even with -I --pedantic. Congratulations. > * Happy to see a DEP5 copyright (not required by policy), especially since > config-edit -application dpkg-copyright -ui none seems happy with it > * Package don't FTBFS. > > Best regards, > > Thomas Preud'homme >> >> Kind regards >> Vincent Cheng > - Vincent -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/CACZd_tDsoeUPFhuiukwbUjQA=pn0b5e1z4swzxzehn7scwa...@mail.gmail.com