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 Python 2 support, removed in qBittorrent v4.2.2 #77

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

ngosang
Copy link
Contributor

@ngosang ngosang commented May 25, 2020

linuxserver.io

Description:

Drop Python 2 support, removed in qBittorrent v4.2.2

I'm qBittorrent developer. We dropped Python 2 support a couple of months ago, it's time to clean up the Docker image.

Benefits of this PR and context:

Resolves #67
Reduces Docker image in 22 MB.

How Has This Been Tested?

Yes. Local build and test.

Source / References:

qbittorrent/search-plugins#84 (comment)

@ngosang
Copy link
Contributor Author

ngosang commented May 26, 2020

@thelamer ping

@ngosang
Copy link
Contributor Author

ngosang commented Jun 6, 2020

@nemchik ping

@nemchik
Copy link
Member

nemchik commented Jun 6, 2020

Ref: #71 discussion

I agree python2 is EOL and many apps have updated to no longer depend on it. That being said it's still available to install in Ubuntu's repos for bionic and having it alongside python3 isn't problematic.

It will very likely get removed when we rebase our Ubuntu images to focal. We're waiting on an upstream issue to be fixed. Anyway I'm not against it being removed right now either. It seems like it would only be used by users running custom scripts, and they could use our custom init to get python 2 back if they still require it. I'll bring this up internally and we'll make a determination on it.

@drizuid drizuid merged commit d905eb1 into linuxserver:master Jun 6, 2020
@renannprado
Copy link

Sorry for my ignorance, but even the latest unstable version is still using python2 even though I see in python3 in the docker image... I'm trying to patch on my own, but either I didn't understand how you guys are approaching this fix or there's indeed something going wrong in the releases.

@ngosang
Copy link
Contributor Author

ngosang commented Jul 6, 2020

@renannprado Just tested the latest Docker image. qBittorrent is using Python3 as expected. You can take a look at the logs:

(I) 2020-07-06T06:38:34 - Python detected, executable name: 'python3', version: 3.6.9

Make sure you remove all the previous Docker images. docker rmi ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: python 3 instead of python 2
5 participants