Hi Étienne, On 2020-04-10 18:54, Étienne Mollier wrote: > For the record, in addition to the ITP #956384, there is a > repository open for prinseq-lite on Salsa: > > https://salsa.debian.org/med-team/prinseq-lite
Great job! I have a couple of comments: 1. In Debian it is usual to strip programming language extensions from file names (.pl in this case). I see lintian already complains about that. I suggest stripping them off in debian/rules. The reverse dependencies will be easy to patch to use the extensionless command names. 2. I see you are installing the manpage (debian/prinseq-lite.pl.1) "manually". You can considerably simplify this just by adding 'debian/prinseq-lite.pl.1' to debian/manpages, the rest will be done automatically! Best wishes, Andrius
signature.asc
Description: OpenPGP digital signature
