Drop python-six dependency

[bluca]

In Ubuntu and Debian we no longer ship Python 2.7, so the translation module is an unnecessary dependency. In Python 3 the text type is always 'str', so it is not needed. Simply set the value to 'unicode' on Python2, and 'str' on Python3.

[ppisar]

The enhancement is a good direction, but I worry it makes the only important case -- when using Python2 without six module installed difficult to debug: In that case it won't report an import failure. Instead it will assign str to the variable and happily continue. That means the code will behave differently.

I'd like rather see and error on Python2 without six module. That means adding a condition on Python major version. In in that case it's probably simpler to set the variable to str or unicode directly then importing the six module. Could rework the patch like that?

Also fedora/libmodulemd.spec for RPM packages still carry the dependency on the six module. That should also be adjusted.

[bluca]

The enhancement is a good direction, but I worry it makes the only important case -- when using Python2 without six module installed difficult to debug: In that case it won't report an import failure. Instead it will assign str to the variable and happily continue. That means the code will behave differently.

I'd like rather see and error on Python2 without six module. That means adding a condition on Python major version. In in that case it's probably simpler to set the variable to str or unicode directly then importing the six module. Could rework the patch like that?

Also fedora/libmodulemd.spec for RPM packages still carry the dependency on the six module. That should also be adjusted.

Sure, done

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[ppisar]

Great. Thanks for the cooperation.