Skip to content
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

Drop support for Python2 #84

Closed
1 of 16 tasks
ngosang opened this issue Apr 7, 2019 · 13 comments
Closed
1 of 16 tasks

Drop support for Python2 #84

ngosang opened this issue Apr 7, 2019 · 13 comments

Comments

@ngosang
Copy link
Member

ngosang commented Apr 7, 2019

Python2 support ends in 2019. https://pythonclock.org/

Changes in qBittorrent:

Changes in search-plugins:

@ngosang
Copy link
Member Author

ngosang commented Apr 7, 2019

@Chocobo1 Would you like to help me with the qBittorrent part? I can do most of the Python work and review your changes, but I will need some help with the C++ part.
There is no hurry, but I want to drop support before the end of the year. At that time most Linux distributions will have python3 as default version so the user don't have to worry about that...

@sledgehammer999 I would like to hear your thoughts.

@ngosang ngosang changed the title [WIP] Drop support for Python2 Drop support for Python2 Apr 7, 2019
@ngosang ngosang added the wip label Apr 7, 2019
@sledgehammer999
Copy link
Member

With the recent minimum Qt version change (aka 5.9) I think we require fairly recent distros anyway. So personally I don't have a problem. The rest is up to you, since I am not using this feature.

@Chocobo1
Copy link
Member

Chocobo1 commented Apr 7, 2019

I'm only interested in dropping py2 support, can't say about the new features.

Just a reminder, the reason of supporting py2 is because of macOS, so I think we can only drop it after the date expires not before.

@sledgehammer999
Copy link
Member

Just a reminder, the reason of supporting py2 is because of macOS

ohh I forgot about that...

@ngosang
Copy link
Member Author

ngosang commented Aug 26, 2019

@Chocobo1
Copy link
Member

Chocobo1 commented Jan 3, 2020

I've opened PR qbittorrent/qBittorrent#11813
Also I would expect search-plugins repo not drop py2 scripts until qbt has a new release containing changes in that PR.

@FranciscoPombal
Copy link
Member

@ngosang python 2 stuff can now be dropped, qbittorrent/qBittorrent#11813 has been merged and included in some official releases by now. Plugins that use it need to be updated for python 3 or removed.

@ngosang
Copy link
Member Author

ngosang commented Apr 26, 2020

@FranciscoPombal Thanks for your notice. At this moment all official plugins are working in Python2 and Python3. There are still users who have older versions and the official Python 2 plugins still work. Since these definitions do not affect the new versions of qBittorrent I prefer to leave them a little longer until main distributions update the package at least.

@joseeloren @nicolasalonzo @darktohka @hannsen @imDMG @iordic Thank you for keeping the unofficial list updated.
Since March 24th 2020 / v4.2.2 qBittorrent only supports Python3. From now, the Python 3 support is mandatory and the Python 2 is optional (Python2 is not required for new plugins of after updating an existing plugin). If you know any plugin that doesn't work with Python3, please move the plugin to a new table at the bottom called "Deprecated plugins" or something like that and reach the original author to update the plugin. Also try to add a more specific information about the version of Python used to test the plugin. Eg Python 3.6 instead of Python 3. There are huge differences between 3.4 and 3.9.

@joseeloren
Copy link

@ngosang ,my plugins work with Python 3. Thank you!

I made a PR to autoupdate plugins. You can check it here:
issue: qbittorrent/qBittorrent#12594
PR: qbittorrent/qBittorrent#12469

@FranciscoPombal
Copy link
Member

@ngosang

Since these definitions do not affect the new versions of qBittorrent I prefer to leave them a little longer until main distributions update the package at least.

Ubuntu 20.04 was just released with no python 2 by default, and /usr/bin/python is deprecated, but points to /usr/bin/python3, meaning the user has to go out of their way to use python 2 (which is technically still available in the repos, but just due to a very small number of packages). Other fixed release distros are in similar or better situations.

Meanwhile, there is no currently supported mainstream linux distro that doesn't offer some kind of official python 3 release by default (I think RHEL 6 might be an exception to this, but it is on life support and I doubt we care about it anyway). Ubuntu 16.04 has python 3.5, for example.

There is no more official upstream support for python 2, meaning no more security and bug fixes. Furthermore, I should mention that the minimum supported python 3 version right now is python 3.5, and will soon be 3.6 (https://devguide.python.org/#status-of-python-branches). So, IMO new plugins/updates shouldn't be required to support python < 3.5 (and later on in 2020, <3.6).

@ngosang
Copy link
Member Author

ngosang commented Apr 26, 2020

@joseeloren The other developers will take care. We have in the roadmap to remove the search plugins and use Jackett instead. Maybe you can help #121

@FranciscoPombal Are Python2 plugins causing any problem in qBittorrent? qBittorrent checks the Python version and download only the folder that match so shouldn't be any problem.
You are right about the distributions. Linux users worry me the least because they receive automatic updates from Python and qBittorrent. I am thinking more on Windows users with updates disabled. I am sure there are thousands.
The official plugins are compatible with Python 2.7 and Python 3.4 - 3.9. We have automatic tests. In few months I will remove the nova folder.

@p060477
Copy link

p060477 commented Jul 25, 2020

hi
i'd like to know ,a simple clarification, is jackett needs to be -installed- for having the jackett plugin workin in qbittorrent 4.2.5..?
i ask cause the python needs is well clarified in qbittorrent installation if you want the search option..
but there is no warning about jackett to be installed...

@FranciscoPombal
Copy link
Member

@p060477

hi
i'd like to know ,a simple clarification, is jackett needs to be -installed- for having the jackett plugin workin in qbittorrent 4.2.5..?
i ask cause the python needs is well clarified in qbittorrent installation if you want the search option..
but there is no warning about jackett to be installed...

Please stop spamming multiple issues with questions that could be solved with a simple google search and are unrelated to the current topic. If you need further support please use the forum, IRC, or Discord. The issue tracker is for bug reports/feature requests and related discussion only.

ngosang added a commit to ngosang/search-plugins that referenced this issue Aug 18, 2020
ngosang added a commit to ngosang/search-plugins that referenced this issue Aug 18, 2020
ngosang added a commit to ngosang/search-plugins that referenced this issue Aug 18, 2020
ngosang added a commit to ngosang/search-plugins that referenced this issue Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants