On 08/22/2011 07:47 PM, Gergely Nagy wrote: > And while I can't sponsor the package (not a DD myself, either), I had a > quick look at it (at least, on the latest version you posted here): > > * In debian/copyright, the paragraph separator dots are unindented - I'm > not sure whether that's acceptable or not, but it looks a bit awkward. > * debian/changelog is executable for some reason (o.O) > * You don't need to list ChangeLog in debian/docs, dh_installchangelogs > will install it as the upstream changelog by default. But.. do you > really want to do that? Upstream ChangeLog is zero-length, and thus not > really useful at this time. > * And as someone already mentioned, there's common-install-indep:: in > debian/rules which I believe is a cdbs-ism, and does not have any effect > in the current packaging. > > Pretty minor stuff, but thought I'll have a look and contribute > something else than nagging for a different package name O:)
Thanks, I just uploaded corrected version. -- Emmanuel Thomas-Maurin <[email protected]> -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: http://lists.debian.org/[email protected]
