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

pipeline: Run proper cross compile host #8304

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Feb 24, 2025

The build-using-self script was always using macos arm64 as a cross compile host.

Modify the build-using-self's cross compile host to use x86_64 when on ARM, and arm64 when on another host architecture.

The build-using-self script was always using macos arm64 as a cross
compile host.

Modify the build-using-self's cross compile host to use x86_64 when on
ARM, and arm64 when on another host architecture.
@bkhouri
Copy link
Contributor Author

bkhouri commented Feb 24, 2025

@swift-ci please test

)
logging.info("Current working directory is %s", pathlib.Path.cwd())
logging.info("Bootstrapping with the XCBuild codepath...")
cross_compile_arch = "x86_64" if platform.machine() == "arm64" else "arm64"
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: would be nice to add an error case for a third architecture for future-proofing instead of assuming the else case wants arm64

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