Hi, On Thu, 10 Sep 2020, Jörg Frings-Fürst wrote: > > I'd like to see this new upstream release in sid so I'm taking a look > > but I'm quite surprised by many things. And actually I want this fix > > in the package too: > > https://gitlab.com/sane-project/backends/-/merge_requests/502 > > I have add this merge request as d/p/0165-respect_local_only_parameter.patch
Thanks. From the comments in that merge request, it seems that https://gitlab.com/sane-project/backends/-/merge_requests/506 should be included too to have a complete fix. And as we said, at this point we should upload this to "unstable" and not to "experimental". Can you add the missing change and fix the version and target distribution? I'm happy to review and upload afterwards. I have not found anything problematic in a quick look. BTW, looking at the added patch, it's wrong to write "Forwarded: no" when you import an upstream patch. The correct value would be "not-needed" but in fact DEP-14 tells that you don't need the field when you have a "Bug" header. https://dep-team.pages.debian.net/deps/dep3/ For the other patches, you have set "Forwared: not needed" when the value documented in DEP3 is "Forwarded: not-needed". Cheers, -- ⢀⣴⠾⠻⢶⣦⠀ Raphaël Hertzog <hert...@debian.org> ⣾⠁⢠⠒⠀⣿⡁ ⢿⡄⠘⠷⠚⠋ The Debian Handbook: https://debian-handbook.info/get/ ⠈⠳⣄⠀⠀⠀⠀ Debian Long Term Support: https://deb.li/LTS
