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

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: