Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Define macOS 11 as earliest supported version #789

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

fhanau
Copy link
Collaborator

@fhanau fhanau commented Jun 20, 2023

macOS 11 is the oldest version still being maintained, V8 also depends on multithreading-related functions introduced with it. This is hopefully sufficient to fix cloudflare/workers-sdk#3307 cloudflare/workers-sdk#3315 cloudflare/workers-sdk#3432 – macOS shouldn't require static linking of system libraries and at least two of the issues are on macOS 11.

Also updated the readme, the ubuntu-latest Github Actions runner image is now identical to ubuntu-22.04 so it works just fine. Based on #170 the commend-line tools should be sufficient for compiling.

@fhanau fhanau requested review from mrbbot and penalosa June 20, 2023 13:36
@mrbbot
Copy link
Contributor

mrbbot commented Jun 20, 2023

Nice! Tentatively approving, but I'd like to test this on actual hardware before merging. 👍

macOS 11 is the oldest version still being maintained, V8 also
depends on multithreading-related functions introduced with it.
@fhanau fhanau force-pushed the felix/mmacosx-version-min branch from 6dc53f7 to 1ed9330 Compare June 21, 2023 22:03
Copy link
Contributor

@mrbbot mrbbot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅ Tested a CI build on macOS 11.7.7. 👍

@fhanau fhanau merged commit 5bc18f6 into main Jun 23, 2023
@fhanau fhanau deleted the felix/mmacosx-version-min branch June 23, 2023 13:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 BUG: Minimum required version for local development
2 participants