control: owner -1 !
control: tags -1 moreinfo

Hi,

>Some comments:
>- this is ITP bug so in d/changelog you should have one entry. with
>one version, with Initial release and (Closes: #XXXX).
>- in your patch there seems to be some UTF problem in the From:
>- can you push pristine tar branches to the git ? I can't build it.
>In d/control, the passage is not needed, options, arguments have place
>in a man page.
>"When given the -d option, duperemove will submit those extents for
>deduplication using the btrfs-extent-same ioctl."


Since a DD already reviewed it, and the points have been addressed,
I'm adding a little review on top of that one, and probably I'll sponsor soon.

1) missing licenses

rbtree_augmented.h: GPL (v2 or later) (with incorrect FSF address)


docs/stylesheets/github-light.css: MIT/X11 (BSD like)

interval_tree_generic.h: GPL (v2 or later) (with incorrect FSF address)


2) wrong copyright years
Copyright (C) 2016 SUSE.

./filerec.c: * which is Copyright 2003 by Theodore Ts'o and released under the 
GPL.

./docs/stylesheets/github-light.css:Copyright (c) 2016 GitHub, Inc.

and probably more
(license-reconsile)

other stuff LGTM
thanks,

Gianfranco

Reply via email to