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

No longer rely on system Python for installs of python3 runtime #93

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

trek
Copy link
Contributor

@trek trek commented Sep 21, 2023

Back in #34 we added Python 3 that would fallback to system Python 3 if it exists. In MacOS 12.3 Apple removed system python.

This PR updates python3 to behave like the other python runtimes by calling installPython.

See:

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@EndangeredMassa
Copy link
Contributor

Can we also update the readme to not say that it falls back to system python?
https://github.com/vercel/fun/blob/main/README.md?plain=1#L83

@trek trek force-pushed the trek/zero-859-python-3-bootstrap-error branch from c9420f6 to f49d88d Compare September 21, 2023 19:02
@trek trek merged commit 07447ac into main Sep 22, 2023
@trek trek deleted the trek/zero-859-python-3-bootstrap-error branch September 22, 2023 00:00
EndangeredMassa added a commit that referenced this pull request Dec 12, 2024
@TooTallNate TooTallNate mentioned this pull request Dec 19, 2024
TooTallNate added a commit that referenced this pull request Dec 19, 2024
* Revert #93.
* Revert #99.
* Remove Go setup from CI (the test was already previously disabled in #97).
* Remove `pkg` (the test was also disabled in #97).
* Remove `prebuild` and `postbuild` npm scripts - they do not get invoked with pnpm.
* Use `pnpm` in CI.
* Update TypeScript to v4.9.3.

---

Co-authored-by: Sean Massa <[email protected]>
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