Hi Jonas Thanks a lot for your remarks! I am a newbie in Debian packaging.
On Thu, Jun 4, 2020 at 10:18 PM Jonas Smedegaard <jo...@jones.dk> wrote: > > Quoting Ricardo Ribalda Delgado (2020-06-04 19:53:10) > > I have just updated my salsa to 2.2.0 > > https://salsa.debian.org/ribalda-guest/ugrep/-/tree/debian In case > > that you want to give it a try. > > Great! > > A few remarks about the packaging: > > The autopkgtest failed: > > upstream-test-suite FAIL stderr: configure.ac:31: installing './ar-lib' How did you trigger the error?. I was relying on: https://salsa.debian.org/ribalda-guest/ugrep/-/jobs/785720 > > Seems you need to add the allow-stderr restriction - more info here: > https://salsa.debian.org/ci-team/autopkgtest/raw/master/doc/README.package-tests.rst Added, thanks > > Related to autopkgtest I (just earlier today in fact) noticed the > "build-needed" restriction which seems perfectly suitable for the kind > of test you've setup. Instead of that I am configuring with dh_ and then using the system installed ugrep. Seems to work ;) https://salsa.debian.org/ribalda-guest/ugrep/-/commit/f8b63f4430818f6414b354804c70037d70a328da > > The package short description is wrongly used as a first line of the > long description - check Debian Policy ยง 5.6.13 for the details on that. Fixed, Thanks! > > I also would have expected libreflex to be built as a shared library for > reuse by other future packages besides ugrep - but perhaps you've > discussed that with Zumbi already and there is some sensible reason for > embedding the library with ugrep. I decided to follow the upstream methodology. Also considering that there is no other use of libreflex today on Debian and the library belongs to the same author of the utility. > > Are you aware that you can use wildcards with lintian overrides? Seems > your 18 almost identical overrides can be shrunk to just one line. And > while at it, please consider adding a comment describing why those > warnings are overridden (it is easier to agree or disagree with your > reasoning without first reading your mind :-) ). No, I was not aware of that, thanks :) > > You've listed only copyright and licensing for main upstream author and > yourself - but there are also (at least) some autotools-originated files > licensed as Expat, FSFAP, FSFUL, FSFULLR, GPL-2+, and GPL-3+. Possibly > you are already aware and consider those irrelevant to track in > debian/copyright, but mentioning in case the omission wasn't deliberate, > as I suspect ftpmaster might disagree with doing that. If interested, > then I can guide you in using licensecheck to check that (and keep track > of changes for later updates). This case it was deliberate. But I have added a simple tracking of licensecheck, thanks for the hint :). > > Thanks a lot for packaging ugrep. I hadn't heard about it before I saw > your ITP, and it looks like an amazing tool, that I will sure spend some > time getting familiar with now. I started using it when I discover that I could not grep unicode files, since then I use it when I have to deal with edk2 development. Now I have less grey hair. All your suggested changes are in https://salsa.debian.org/ribalda-guest/ugrep/-/commit/22f76fa5cebb249ffcb62950a8a7fe11eb9d1959 Thanks again! Cheers! > > > - Jonas > > -- > * Jonas Smedegaard - idealist & Internet-arkitekt > * Tlf.: +45 40843136 Website: http://dr.jones.dk/ > > [x] quote me freely [ ] ask before reusing [ ] keep private -- Ricardo Ribalda