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

remove python2 #22641

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

remove python2 #22641

wants to merge 1 commit into from

Conversation

ognevny
Copy link
Collaborator

@ognevny ognevny commented Nov 24, 2024

gimp: add --disable-python so it won't depend on it

@lazka
Copy link
Member

lazka commented Nov 24, 2024

This is still needed by gimp upstream until 3.0 is released as far as I know

@ognevny
Copy link
Collaborator Author

ognevny commented Nov 24, 2024

python plugins are optional for gimp. CI shows that it's fine to build gimp without python. as we package gimp3 here, it won't be a problem to ask users to switch to gimp3 if they need those plugins

edit: Arch already removed python2 entirely

@lazka
Copy link
Member

lazka commented Nov 24, 2024

Yes, I agree, but gimp is using MSYS2 to create their installer.

edit: I can ask them in IRC again if needed, I doubt anything has changed though

@ognevny
Copy link
Collaborator Author

ognevny commented Nov 24, 2024

I thought that gimp2 won't be updated anymore...

@lazka
Copy link
Member

lazka commented Nov 24, 2024

That would be weird, given it is the only supported stable branch atm.

@lazka
Copy link
Member

lazka commented Nov 24, 2024

We could drop everything besides mingw32/mingw64 though

@ognevny
Copy link
Collaborator Author

ognevny commented Nov 24, 2024

dropping 32 bit here will be good, it doesn't seem that someone really needs such version of gimp

@lazka
Copy link
Member

lazka commented Nov 24, 2024

Upstream gimp2 uses mingw32 for their installer.

@ognevny
Copy link
Collaborator Author

ognevny commented Nov 24, 2024

ok, leaving it draft until gimp3 is released

@MehdiChinoune
Copy link
Collaborator

We could drop everything besides mingw32/mingw64 though

So, we could remove it from clang64 and ucrt64.

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.

3 participants