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

Attempt to build cache using osx target version 12. Added osx target version as cache key #627

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from

Conversation

rashidnhm
Copy link
Collaborator

@rashidnhm rashidnhm commented Mar 22, 2024

Context: Change the OSX Deployment version to 12, for more backwards compatibility.

Description of the Change:
This PR updates the OSX Deployment target version to a lower value than 14. This is to address compatibility concerns.

Furthermore, the target osx version was added to the cache key for arm. As we want to not use older cache if this target version is updated again.

Benefits:

Possible Drawbacks:

Related GitHub Issues:
None. Shortcut: sc-59543.

@rashidnhm rashidnhm added the ci:build-wheels Run the wheel building workflows on this Pull Request label Mar 22, 2024
@rashidnhm rashidnhm changed the title Attempt to build cache using osx target version 12. Added osx version… Attempt to build cache using osx target version 12. Added osx target version as cache key Mar 22, 2024
@dime10
Copy link
Collaborator

dime10 commented Mar 22, 2024

I think this might be relevant? https://forums.developer.apple.com/forums/thread/739342

Is there a way to verify that the deployment target variable is used correctly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:build-wheels Run the wheel building workflows on this Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants