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
Attachment:
signature.asc
Description: OpenPGP digital signature