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

PR: Use Python 3.9.14 for x86_64 macOS application build #21770

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

mrclary
Copy link
Contributor

@mrclary mrclary commented Feb 6, 2024

Description of Changes

Spyder 5.5.1rc0 macOS x86_64 exhibits

RuntimeError: tk.h version (8.5) doesn't match libtk.a version (8.6)

This exhibited previously for local arm64 builds and seemed to be the result of incorrect tcl-tk Python build flags. However, this did not exhibit for Python 3.9.14 x86_64. I suspect that there is a change in the setup-python build flags between 3.9, 3.10, x86_64, and arm64. Nevertheless, the easiest fix right now is to revert to 3.9.14 for x86_64. 🤞🏼

@mrclary mrclary force-pushed the x86-fix branch 2 times, most recently from ed9609d to 17ddd62 Compare February 6, 2024 23:47
@mrclary mrclary self-assigned this Feb 7, 2024
@mrclary mrclary marked this pull request as ready for review February 7, 2024 00:19
@mrclary mrclary requested a review from ccordoba12 February 7, 2024 00:20
@mrclary
Copy link
Contributor Author

mrclary commented Feb 7, 2024

@ccordoba12, the x86_64 artifact works properly on my local machine. I think we're good to go.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks @mrclary!

@ccordoba12 ccordoba12 merged commit 24c2a88 into spyder-ide:5.x Feb 7, 2024
3 checks passed
@mrclary mrclary deleted the x86-fix branch February 7, 2024 00:58
ccordoba12 added a commit that referenced this pull request Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants