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

Change /usr/bin/python to /usr/bin/python3 Fixes #190 #191

Closed
wants to merge 3 commits into from
Closed

Change /usr/bin/python to /usr/bin/python3 Fixes #190 #191

wants to merge 3 commits into from

Conversation

juandesant
Copy link
Contributor

Change of interpreter to /usr/bin/python3 to avoid issues with macOS 12.3 and later.

Change of interpreter to /usr/bin/python3 to avoid issues with macOS 12.3 and later.
Copy link
Contributor Author

@juandesant juandesant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Pull Request fixes the interpreter used by the LaTeX bundle. I have tested it on macOS 12.3 with TextMate 2.0.23, and it works perfectly.

@juandesant
Copy link
Contributor Author

juandesant commented Mar 31, 2022

[null comment]

@sanssecours
Copy link
Member

sanssecours commented Apr 3, 2022

Thank you for providing a solution for the recent problems of the Python code in the Bundle 🙏. For now, I think I will close this in favor of the changes of commit 54d6709. While the code in this pull request should improve the situation in macOS 12.3 and later, it would most probably make the situation worse for people who use an earlier version of macOS.

In earlier versions of macOS /usr/bin/python3 seems to also exist and point to a “stub” that downloads Python3, just like it does now. I mistakenly thought /usr/bin/python3 does not exist in earlier versions of the OS first. However, as I see it /usr/bin/python has two advantages over /usr/bin/python3:

  • No need to download the intepreter, if you have not done so already.
  • The intepreter /usr/bin/python3 does not ship with PyObjC. Unfortunately some Commands of the Bundle depend on this library.

@sanssecours sanssecours closed this Apr 3, 2022
@juandesant
Copy link
Contributor Author

Thanks, @sanssecours for having looked at this, and having provided an alternative way of fixing #190 instead…

Also, in general, many thanks for providing a very useful resource for everyone using TextMate and LaTeX on our Macs… is there a Patreon of Koffi I could contribute to?

@sanssecours
Copy link
Member

Thank you for your kind words 💖.

is there a Patreon of Koffi I could contribute to?

I do not think so. I added a PayPal button to the readme a long time ago once, but removed it after some time. I already feel bad about not doing enough work here. At least in the current situation I can honestly state that I work on this Bundle in my free time without any payment 😅.

@juandesant juandesant deleted the fix-190-python-to-python3 branch April 4, 2022 22:40
@juandesant
Copy link
Contributor Author

Having some kind words to someone that out of his kindness gives his time to benefit many people It is the least a decent person can do, really… it is a bit sad that the bar is so low, but then let's try to jump it a bit more often 😉

In any case, yes, I think the personal freedom that that state guarantees you is worth keeping… all the best!

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.

2 participants