[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#856910: RFS: inotify-tools/3.14-3



control: tag -1 +moreinfo
control: owner -1 !

Dear Dmitry,

On Mon, Mar 06, 2017 at 09:05:19AM +0300, Dmitry Bogatov wrote:
> I am looking for a sponsor for my package "inotify-tools"

Thank you for your interest in adopting this package.  And great choice
for the git workflow ;)

A review of dca4111:

- Your conversion of d/copyright has lost some information, by merging
  the stanzas for McGovern and McCutchan.  If you use two separate
  stanzas the conversion would not be lossy.

- In your changelog you wrote that you bumped the std-ver.  It's
  conventional to say what version you bumped it to.

- The Lintian override could use a wildcard so that it only matches the
  doxygen file, not any other javascript that might appear in a new
  upstream release (unlikely though that is)

- You've modified debian/*.install but this isn't in the changelog.

- The dgit-maint-merge patch header has been updated to a better text.
  Please consider using it: https://manpages.debian.org/dgit-maint-merge

- Changes to d/rules not in changelog.

- Just fyi: https://manpages.debian.org/git-deborig

Don't forget `dch -r` once you've addressed these.

-- 
Sean Whitton

Attachment: signature.asc
Description: PGP signature


Reply to: