Re: librtr: Request for review / upload (Re: Git repository for librtr)
- To: Lukas Schwaighofer <lukas@schwaighofer.name>
- Cc: Debian Security Tools Packaging Team <debian-security-tools@lists.debian.org>
- Subject: Re: librtr: Request for review / upload (Re: Git repository for librtr)
- From: Samuel Henrique <samueloph@debian.org>
- Date: Mon, 22 Oct 2018 21:12:04 -0300
- Message-id: <[🔎] CABwkT9qXALi+3WoxsQ-s9-jXbXr0try-cBSnnp6-5k0x_Q2zYA@mail.gmail.com>
- In-reply-to: <20181010221324.45f4c8d6@localhost>
- References: <20180819192604.35280305@localhost> <20131029141235.14698.82471.reportbug@vps31047.fholler.de> <handler.s.B728206.153469957517922.transcript@bugs.debian.org> <20180819193500.7f62e355@localhost> <666152043.14603879.1534774697277@mail.yahoo.com> <20180820214317.4adf8990@localhost> <20181010221324.45f4c8d6@localhost>
Hello Lukas,
Since it has been some time since you asked and nobody replied yet, I
will list here what I already checked.
d/watch: ok
d/control: ok, but it's better with wrap-and-sort
d/copyright:
* it looks like there are two template lines on the file, #9 and #10
* there are files not listed on d/copyright, I noticed at least two
of them: scripts/checkpatch.pl and cmake/modules/FindGcov.cmake,
please do a general review and document all files, this is something
which makes the package get rejected by ftp-master
wrap-and-sort -a:
will better format some files under debian/ and remove an extra
comma in d/control
I'm not very experienced with shared libs, so pardon me for not
getting deeper into the review, it would be nice if somebody else from
the team could have a look at that. I'll try to study and understand
the nuances of packaging shared libs meanwhile.
Thanks for your work
--
Samuel Henrique <samueloph>
Reply to: