Le 20/09/2010 09:15, David Paleino a écrit : > If you're still interested, I'm available to review and sponsor it. > > Thank you! >> It's available on debian mentors: >> http://mentors.debian.net/cgi-bin/sponsor-pkglist?action=details;package=valide >> >> Should you require any additional information, please do not hesitate to >> contact me. >> > Here are my comments on the packaging: > > - before seeing this bug, I downloaded the tarball from your website. Why is > the one on the website 24M, and the one on mentors just 1.4M ? > - debian/changelog: this package has never been released to Debian, so you > should only have one stanza, with "Initial upload (Closes: #547665)" > - debian/control: > - Build-Depends: you're missing at least valac as dependency here. I > haven't > checked further. > - Standards-Version: latest is 3.9.1, please update it > - Vcs-* missing: if you're maintaining the debian/ directory under some > VCS, > please point to it using Vcs-Browser and Vcs-Svn (or Vcs-Git, Vcs-Bzr, > [..]). > - hardcoding runtime dependencies on shared libraries is bad. It doesn't > let binNMUs. If ${shlibs:Depends} doesn't work, something's wrong, and > should be fixed. > - you're duplicating "Section: devel" in valide-common and valide-dev > - long descriptions are a bit too "short" :) > I find inspiration and I try to improve this. > - valide-dev should depend on valide (= ${binary:Version}) > - debian/copyright: you could choose to follow DEP-5, but that's entirely > optional. > - why do you have debian/patches/debian-changes-0.7.0-1? Please merge those > changes upstream, and rebuild the package with a fixed orig.tar.gz > - debian/valide*.install: you can just drop debian/tmp/ from there :) > - debian/valide.lintian: should be named valide.lintian-overrides. After > that, you can drop valide.dirs, and the hack in debian/rules to install > this file. > I renamed the lintian file, but I don't know how to hack debian/rules. > - debian/rules: why are you using that CMD=$(..) thing? You're using it > only > once, and I don't see any benefit in it :) > > > Please let me know whether you still intend to maintain it, and eventually fix > the above comments. > > Except my comments, I fixed the other problems in the upsteam repository: https://code.launchpad.net/~valide/valide/trunk
signature.asc
Description: OpenPGP digital signature