Bug#803620: RFS: afl-cov/0.3-1 [ITP] -- code coverage for afl
Control: owner -1 !
Control: tags -1 moreinfo
Hi Daniel!
quick review:
1) debian/patches/correct-fsf-address...
well, I usually don't ask to change the address but to forward a patch upstream instead.
Doing both is for sure really nice
2) you changed to subprocess32, this is nice, but can you please provide a rationale for it?
better subprocess handling? why you didn't forward the patch upstream?
I'm asking because such Debian delta usually might introduce downstream problems :)
3) and now the important part, the reason for the moreinfo tag:
are you sure about the license?
I am not :p
cheers!
G.
Reply to: