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

Fix the fuck for Python 3.12 #1415

Closed
wants to merge 1 commit into from
Closed

Fix the fuck for Python 3.12 #1415

wants to merge 1 commit into from

Conversation

mbridon
Copy link

@mbridon mbridon commented Nov 13, 2023

The imp module has been long deprecated and has been removed entirely in Python 3.12.

This solution is what was suggested in the release notes: https://docs.python.org/3.12/whatsnew/3.12.html#imp

The imp module has been long deprecated and has been removed
entirely in Python 3.12.

This solution is what was suggested in the release notes:
https://docs.python.org/3.12/whatsnew/3.12.html#imp
@radrow
Copy link

radrow commented Nov 14, 2023

finally someone gave a fuck

@mbridon
Copy link
Author

mbridon commented Nov 14, 2023

finally someone gave a fuck

I'd say I even gave "the fuck" 🤪

@arion52
Copy link

arion52 commented Nov 17, 2023

oh wow cool thanks was about to check it out but yeah u got the job done 👍

@Payne680
Copy link

Waiting for the pull request to be merged 😎

@krossekrabbe
Copy link

krossekrabbe commented Nov 29, 2023

This bit fucked me after upgrading to Fedora 39.

Installing from fork worked for me

cd /tmp
git clone https://github.com/mbridon/thefuck.git
pip uninstall thefuck
pip install -e ./thefuck

@arion52
Copy link

arion52 commented Nov 29, 2023

This bit fucked me after upgrading to Fedora 39.

lmao same still hasn't been merged 😕

@radrow
Copy link

radrow commented Dec 10, 2023

Just FYI due to this bug my terminal is fucked up since the upgrade:

image

No, I am not going to comment the fuck out of my zshrc.

@LoZeno
Copy link

LoZeno commented Dec 11, 2023

Adding my +1 to get this merged soon!

@mbridon
Copy link
Author

mbridon commented Jan 31, 2024

I don't have Python 2.7 any more, so I'll be happy if you can help fix this issue so the fix I provided works on both 2.7 and 3.12 😁

@lucazenari
Copy link

As a workaround, while we wait for this PR to be merged, if anyone wants to install this fork the quickest way is:

pip uninstall thefuck
pip install https://github.com/mbridon/thefuck/archive/main.zip

@tanenbaum
Copy link

I've just learned that there is an imp compatibility package called python3-zombie-imp on most distros (e.g. https://packages.fedoraproject.org/pkgs/python-zombie-imp/python3-zombie-imp/) that patches around this.

@mbridon
Copy link
Author

mbridon commented Feb 19, 2024

I've just learned that there is an imp compatibility package called python3-zombie-imp on most distros (e.g. https://packages.fedoraproject.org/pkgs/python-zombie-imp/python3-zombie-imp/) that patches around this.

Sure, that's a potential crutch, but fixing the issue is better 😉

@fbe555
Copy link

fbe555 commented May 2, 2024

Bump.
I also ran into this issue after updating to ubuntu 24.04 (and by extension to python 3.12.3), and fixed it by installing from the fork. Seems like this is super duper ready to merge :)

@mbridon
Copy link
Author

mbridon commented May 2, 2024

Well, I guess the original author @nvbn seems to have disappeared, so all it would take is for someone to fork the repo and merge this PR and then assume maintenance of the project.

I won't do it for health reasons, so if someone who commented on this PR wants to have a go at it then go ahead. 🤷

And when @nvbn comes back eventually, assume joint maintenance all together.

Free Software sometimes loses maintainers and find new ones, that's how it sometimes evolves and continues.

@nomike
Copy link

nomike commented May 3, 2024

I believe this pull request could be closed.
A fix for this is already in the master branch.

@mbridon
Copy link
Author

mbridon commented May 4, 2024

Ah great, so the maintainer fixed the issue but didn't merge the PR which could have given me the credit...

Well, all that matters is that it's fixed. 🤷

@mbridon mbridon closed this May 4, 2024
@mbridon mbridon deleted the main branch May 4, 2024 09:23
@afunworm
Copy link

afunworm commented Sep 5, 2024

Ah great, so the maintainer fixed the issue but didn't merge the PR which could have given me the credit...

Well, all that matters is that it's fixed. 🤷

Hello.

I am running the same issue on Ubuntu 24.04. However, your archive on your repository is no longer there. Is there any work-around to install this, system-wide?

Thank you.

@cyuria cyuria mentioned this pull request Sep 8, 2024
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.