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 via email to