Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
On Thu, Dec 6, 2018 at 2:03 PM Brent Clark wrote: > I am looking for a sponsor for my package "clamav-unofficial-sigs": ... > The respective dsc file can be found at: > > https://mentors.debian.net/debian/pool/main/c/clamav-unofficial-sigs/clamav-unofficial-sigs_5.6.2-1.dsc Sorry for the long delay, here is a review: These things need to be done before any upload: The upstream tarball fetched by uscan (using debian/watch) needs to be identical to the one you upload to mentors, the one on mentors currently has both missing files and additional files. If any changes are needed to upstream files, they should be represented as patches in debian/patches/ (or possibly other files in debian/). I generally don't use git for packaging but I hear that there are tools that can generate those patches from git commits. About the manual page, cron job and logrotate config, there is an upstream bug that means that `./clamav-unofficial-sigs.sh --config somedir/ --information` doesn't work as it always checks /etc/clamav-unofficial-sigs instead. If it worked, then you could have asked the script to install the files at package build time. A workaround for this bug would be to extract the snippets at build time instead using sed. Unfortunately that isn't feasible since the snippets in the script are only templates and the full output gets created dynamically. Since the fix for this bug is a one-liner, I think it would be best to include the change, since it allows creation of the files at build time and also fixes a bug that users might run into. Place the following line just before the second time that the config_files array is set. config_dir="$custom_config" The upgrade from the old version needs to be checked (with piuparts) to check that it works, that conffiles are handled correctly and that the package still works afterwards. These things would be nice to fix at some point: The second and third stansas of debian/copyright should probably be merged, details in the copyright format guide: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ The package description probably needs to be updated for the new/changed features of the new upstream. The Vcs-* fields in debian/control should point at the packaging VCS rather than the upstream VCS. There are some warnings when I run it on the .changes file after building the package. The new upstream doesn't appear to very active, you might want to offer your assistance or create a new upstream project to maintain the upstream code in, possibly within the ClamAV github organisation: https://github.com/clamav -- bye, pabs https://wiki.debian.org/PaulWise
Bug#915694: [Pkg-clamav-devel] Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
Good day Scott. As per your request, its done. Please can your check. Regards Brent On 2018/12/07 15:52, Scott Kitterman wrote: Yes. You'll need to do that. Scott K On Friday, December 07, 2018 10:25:40 AM Brent Clark wrote: Good day Scott and Paul Thanks so much for looking at my RFS. Sorry for not communicating that in the changelog. Do you guys want that in the changelog? Regards Brent On 2018/12/07 09:18, Paul Wise wrote: On Fri, Dec 7, 2018 at 3:12 PM Scott Kitterman wrote: This package currently has Paul Wise listed as a co-maintainer, but you've removed him. Did you coordinate that with him? In any case, the change needs to be documented in debian/changelog. He did, see the bug closed in debian/changelog.
Bug#915694: [Pkg-clamav-devel] Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
Yes. You'll need to do that. Scott K On Friday, December 07, 2018 10:25:40 AM Brent Clark wrote: > Good day Scott and Paul > > Thanks so much for looking at my RFS. > > Sorry for not communicating that in the changelog. > Do you guys want that in the changelog? > > Regards > Brent > > On 2018/12/07 09:18, Paul Wise wrote: > > On Fri, Dec 7, 2018 at 3:12 PM Scott Kitterman wrote: > >> This package currently has Paul Wise listed as a co-maintainer, but > >> you've > >> removed him. Did you coordinate that with him? In any case, the change > >> needs to be documented in debian/changelog. > > > > He did, see the bug closed in debian/changelog.
Bug#915694: [Pkg-clamav-devel] Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
Good day Scott and Paul Thanks so much for looking at my RFS. Sorry for not communicating that in the changelog. Do you guys want that in the changelog? Regards Brent On 2018/12/07 09:18, Paul Wise wrote: On Fri, Dec 7, 2018 at 3:12 PM Scott Kitterman wrote: This package currently has Paul Wise listed as a co-maintainer, but you've removed him. Did you coordinate that with him? In any case, the change needs to be documented in debian/changelog. He did, see the bug closed in debian/changelog.
Bug#915694: [Pkg-clamav-devel] Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
On Fri, Dec 7, 2018 at 3:12 PM Scott Kitterman wrote: > This package currently has Paul Wise listed as a co-maintainer, but you've > removed him. Did you coordinate that with him? In any case, the change needs > to be documented in debian/changelog. He did, see the bug closed in debian/changelog. -- bye, pabs https://wiki.debian.org/PaulWise
Bug#915694: [Pkg-clamav-devel] Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
This package currently has Paul Wise listed as a co-maintainer, but you've removed him. Did you coordinate that with him? In any case, the change needs to be documented in debian/changelog. Scott K On Thursday, December 06, 2018 08:01:39 AM Brent Clark wrote: > Package: sponsorship-requests > Severity: normal > > Good day Maintainer / Mentors, > > I am looking for a sponsor for my package "clamav-unofficial-sigs": > >* Package name: clamav-unofficial-sigs > Version : 5.6.2 > Upstream Author : Adrian Jon Kriel >* URL : https://github.com/extremeshok/clamav-unofficial-sigs > * License : BSD > Section : utils > > To access further information about this package, please visit the following > URL: https://mentors.debian.net/package/clamav-unofficial-sigs > https://salsa.debian.org/brentclark-guest/clamav-unofficial-sigs > > The respective dsc file can be found at: > > https://mentors.debian.net/debian/pool/main/c/clamav-unofficial-sigs/clamav > -unofficial-sigs_5.6.2-1.dsc > > More information about clamav-unofficial-sigs can be obtained from > https://github.com/extremeshok/clamav-unofficial-sigs > > If you have any concerns or questions, please do not hesitate to contact me. > > Much appreciated, regards > Brent Clark > > ___ > Pkg-clamav-devel mailing list > pkg-clamav-de...@alioth-lists.debian.net > https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-clamav-devel
Bug#915694: RFS: clamav-unofficial-sigs/5.6.2
Package: sponsorship-requests Severity: normal Good day Maintainer / Mentors, I am looking for a sponsor for my package "clamav-unofficial-sigs": * Package name: clamav-unofficial-sigs Version : 5.6.2 Upstream Author : Adrian Jon Kriel * URL : https://github.com/extremeshok/clamav-unofficial-sigs * License : BSD Section : utils To access further information about this package, please visit the following URL: https://mentors.debian.net/package/clamav-unofficial-sigs https://salsa.debian.org/brentclark-guest/clamav-unofficial-sigs The respective dsc file can be found at: https://mentors.debian.net/debian/pool/main/c/clamav-unofficial-sigs/clamav-unofficial-sigs_5.6.2-1.dsc More information about clamav-unofficial-sigs can be obtained from https://github.com/extremeshok/clamav-unofficial-sigs If you have any concerns or questions, please do not hesitate to contact me. Much appreciated, regards Brent Clark