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

Speed up aarch64 builds using Static Linux SDK #265

Merged
merged 7 commits into from
Oct 24, 2024

Conversation

sidepelican
Copy link
Collaborator

Motivation

Currently, the mockolo binary for aarch64 on Ubuntu is built on an x86_64 machine using Docker and qemu, which is extremely slow.
With the release of Static Linux SDK in Swift 6, we can speed up the build process for aarch64 Linux.

Changes

In the GitHub Actions for the release build, we install the Static Linux SDK and use it.
As a result, the size of the artifact bundle has increased. To address this, we provide a lightweight artifact bundle without Linux support for users who only use mockolo on macOS.

Misc

Since the Static Linux SDK uses musl, mockolo will be able to run on more Linux distributions. However, we do not have a strong motivation to actively support these additional distributions, so this is more of a bonus.

@sidepelican
Copy link
Collaborator Author

steps:
- uses: swift-actions/setup-swift@v2
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

setup-swift does not support Swift6🥲

Comment on lines 28 to 29
- if: startsWith(matrix.destination.name, 'macos')
run: sudo xcode-select -s /Applications/Xcode_16.app
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

swiftly's macOS support is not released yet so do the alternative here.

@sidepelican sidepelican requested review from uhooi and fummicc1 October 23, 2024 06:08
Copy link
Collaborator

@uhooi uhooi left a comment

Choose a reason for hiding this comment

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

LGTM!

@sidepelican sidepelican merged commit f7ff730 into uber:master Oct 24, 2024
3 checks passed
@sidepelican sidepelican deleted the swift_sdk_build branch October 24, 2024 02:26
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