Hi Nigel, (I do not intend to sponsor this package, sorry.)
I had a few minutes, so here is a very short review (for sure not complete:) You are also upstream, aren't you? Then I suggest reading https://wiki.debian.org/UpstreamGuide (So you should for example consider removing the debian directory from your repository; it will be hard for you to keep that in sync... :)) d/changelog: for a new package its only "Initial release (Closes #xxxxxx)"; delete the rest. There's should be an empty line between the "header" line and the entries Hint: Use dch(1) to create your d/changelog, don't do it manually You need to remove your logfiles when the package is purged. (using a postrm script) See policy 10.8 There is a linitian problem with the long description, refer also to policy 3.4.1 d/rules I'd suggest not to override targets if there are other means, like d/clean instead of override_dh_autoclean AUTHORS and TODO needs not to be installed in the binary package, as it does not contain information useful to the user. Thanks! -- tobi -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/1409515724.19157.26.ca...@edoras.loewenhoehle.ip
