Pavel Šimerda

use `from six.moves import ...`

1

The resulting code for simple library imports makes code unnecessarily verbose. For example when you import `ConfigParser` in legacy code or `configparser` in Python 3.x code, you generally don't want...

bad relative imports are introduced

1

Modernize introduces bad syntax when dealing with imports from local packages. Original code: ``` import ssl.SSLCommon ``` Modernized code containing syntax error: ``` from . import ssl.SSLCommon ``` Probably desired...

why do we need to add `from __future__ import absolute_import` everywhere

3

I'm not entirely sure why we need to add the following line to so many files where it doesn't change any behavior. Is there some good reason to do that?...

rpmdiff reports changed size but unchanged checksum

4

Maybe I just misunderstood something but how can rpmdiff report difference in *size* and not difference in *checksum* for so many files? $ rpmdiff -i T /var/tmp/build-root/standard-x86_64/home/abuild/rpmbuild/RPMS/x86_64/coreutils-8.25-0.x86_64.rpm binaries/coreutils-8.25-4.1.x86_64.rpm removed PROVIDES...

ignore `ESRCH` on processes to be killed

5

This patch has been rejected but I'm still using it and this pull request will serve a new discussion. From the stacktrace below you can see that it's `debugger.quit()` procedure...