fedora-infra / mirrormanager2

Rewrite of the MirrorManager application in Flask and SQLAlchemy
https://mirrormanager.fedoraproject.org
GNU General Public License v2.0
65 stars 49 forks source link

Python 3 compatibility #139

Closed Conan-Kudo closed 6 years ago

Conan-Kudo commented 8 years ago

With Fedora working hard to have Python 3 as the default, and EPEL now including python34 for EL7 (and of course, there's Software Collections for Python 3.4 for EL6 and EL7), it would be awesome if MirrorManager2 added Python 3 support.

A cursory check at the requirements files in the repository indicate that there are only four modules that don't indicate Python 3 support in PyPi:

I don't know to what extent these modules are required, but perhaps a first step would be to make MirrorManager2 itself Py3 ready with the six module? If the modules above aren't mandatory to MirrorManager2's functionality, then perhaps splitting the stuff that uses that out as a subpackage to make the core available with Py3 is an option?

pypingou commented 8 years ago

Porting to py3 is a nice idea, but do you know any potential users of MM2 in py3?

Conan-Kudo commented 8 years ago

@pypingou Yes. Mageia would be a potential user, as I'm currently evaluating how to deploy the mirroring of rpm-md repositories for DNF on Mageia. If it is possible to deploy MirrorManager2 as a Python 3 application, that would make it more attractive, as I don't particularly enjoy the idea of deploying new infrastructure that depends on Python 2 if I can help it.

pypingou commented 8 years ago

Roger that, thanks

ralphbean commented 8 years ago

FYI, all of those deps are all py3-ready now. All that is left is to port MM2 itself.

keszybz commented 6 years ago

This can be closed now. If any python3 issues appear, they should probably be tracked separately.

adrianreber commented 6 years ago

@keszybz thanks