-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Port to Python 3 #78
Comments
We thought about this but it's quite complicated, maybe in the future (for the moment being it's just an idea). |
This is something we definitely want in my opinion, but it's pretty far away. As an aside, "reactions" on GH are ridiculous. |
When this is implemented, I think that the (currently rather complex) concurrent Tasks can be simplified into asyncio Tasks. (I'm subscribing to this issue to be pinged when it becomes active). |
Yeah, there was a brief exchange about it here. |
JFYI, python2 is going to be definitively deprecated in around 3 years. So, even if it’s not tomorrow, this is indeed something that should be considered. |
yes would like to see python3 support |
https://lists.fedoraproject.org/archives/list/[email protected]/thread/2XDT6S62SEN4KGWQCOUYKXLBYVTKJ5VI/ was linked in the IRC channel and is about Fedora starting to drop packages that depend on Python 2 prior to Python 2 becoming unsupported upstream on Jan. 1st, 2020 (less than 2 years from now). If we want whipper to be shipped with Fedora, we may want to consider increasing the priority of porting the code to Py3. |
You mean it would be wiser to reschedule this issue for milestone 1.0? |
Please, let's not do that. Let's just get 1.0 out first. |
More like it should be one of the top priorities once 1.0 is out. |
RHEL is deprecating Python 2 support with RHEL 7.5 and thus Centos 7.5. My understanding is Python 2 will remain available until RHEL/Centos 8 but Python 3 has been swapped to the primary python. I'm not sure it matter much if at all because I'm not packaging in EPEL7 due to the very outdated libcdio-paranoia version but I just figured I'd throw the info in here. |
For quite some time I've been waiting for this port to Python 3 to happen. Since my system has no Legacy Python installed, whipper would be the only project that would rely on Python 2. So the progress here is good news... Unfortunately I can't find any information on how usable the Python 3 version already is. Is it still experimental or ready for everyday use? |
Currently the python 3 version doesn't exist (I'm trying to incrementally update the code to make it compatible with both python 2 and python 3 - if easily doable). Unfortunately, as it is right now, whipper can't be used with python 3. |
Thanks for the update - keep up the good work! |
Just wanted to clarify that the aim of the port is to support only Python 3. |
So with the first Python 3 version the support for Legacy Python will be dropped? |
|
In preparation for dropping Python 2.7 support[1], this specifies versions that *are* compatible with Python 2.7, so anyone installing via pip (if we end up publishing to PyPI) will get the proper version. See https://packaging.python.org/guides/dropping-older-python-versions/ [1] #78 Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
this is blocking whipper entering Debian & Ubuntu, py2 support is being removed from unstable, and no new packages that provide py2 modules (pycdio) or packages that depend on py2 (whipper) can enter the archive anymore |
that probably sounded like a rant, sorry.. just stating the facts, I'll package it in a ppa for now :) |
Just tossing a note in here. Fedora is retiring Python2 once Fedora 31 is branched from Rawhide and any dependent packages (including whipper) will be removed 2 weeks before the Fedora 32 mass rebuild. https://fedoraproject.org/wiki/Changes/RetirePython2 I don't know if the Mass Rebuild date has been set, at least I cannot find one. Guessing off the dates from the Fedora 30 schedule, Fedora 31 is supposed to branch off rawhide next week, there's max 5 months before the package is removed before the Fedora 32 Mass Rebuild. A bug has filed against the package if anyone would like to comment on the timeline for this to the Fedora powers that be. https://bugzilla.redhat.com/show_bug.cgi?id=1739019 |
Python 2 is EOL on December 31st, so yes most distros will drop it before this date. On Arch we are not decided yet, it will depend on whether there is anything big left unported, but the list is shrinking days after days. |
Right now the focus is to get 0.8.0 made ready for release and pushed out, which will include (Edit: And yes, we know Python 2 is getting EOL’d soon. However, while we are a handful of devs working on whipper on and off, none of us are paid to work full time on it, so progress happens as we find time for it—or when outside people decide to contribute. Posting more updates here about this or that distribution dropping whipper due to still requiring Python 2 will not make the port happen any faster. Actual contributions (not necessarily PRs) might though.) |
The Fedora 32 schedule is now published. The Python 2 EOL is included for 1 January 2020. The F32 Mass Rebuild is scheduled from 22 Jan to 11 Feb 2020. I am inclined to say that Py2 packages could be dropped between the Py2 EOL and the Mass Rebuild.
It's unlikely an exception will be approved. 😞 I didn't realize this blocked dropping the But I get this project is a volunteer-driven initiative too. If I can offer any advice or opinions from the Fedora/Linux packaging ecosystem POV, I'm happy to be a resource. |
@jwflory There is an ongoing PR for porting at #411. There is one test failure remaining (#411 (comment)) and several warnings/errors though. |
@ArchangeGabriel Thanks for pointing it out, I should have looked. 🙂 I left some feedback there. |
No description provided.
The text was updated successfully, but these errors were encountered: