Thank you for the review. I've applied

Copyright:
- mixture of BSD-3-clause and MPL-2.0 --> Done
- pmezard --> Done
- urfave --> Done
- sirupsen --> Done

Lintian
- I've overridden 2 more and remove the unused override. I don't have all your 
`missing-notice-file-for-apache-license` warnings, nor the `unused override 
hardening-no-bindnow`. This library is built with lazy loading on purpose so 
it's pretty certain that this override is required?

General
- dh_dwz fails and it's not a critical step anyway. It sounds like I'm not the 
only one that has problems with it: 
https://groups.google.com/g/linux.debian.bugs.dist/c/EuNwwn5TZ0Y
- dh_link: That's a very much nicer way to do it. It's less aligned with 
upstream but definitely more elegant. Applied and verified.
- removed d/prepare and d/changelog.old
- d/tests/build is indeed  superficial, So I added Restrictions: superficial to 
it.
- '>' added in the email address in the patches
- Order of patches reworked

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2138390

Title:
  [needs-packaging] nvidia-container-toolkit

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/2138390/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to