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

Add x86_64 binary to an artifact bundle in a release #142

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

mtj0928
Copy link
Collaborator

@mtj0928 mtj0928 commented Sep 14, 2024

I noticed that we can add x86_64 binary to an artifact bundle by build the project twice.

When scipio is built for arm64, the binary is generated at ./.build/arm64-apple-macosx/release/scipio.
And when it is built for x86_64, the binary is generated at ./.build/x86_64-apple-macosx/release/scipio.

ArtifactBundleGen used by scipio can combines both binaries into an artifact bundle when they exist.

This is the artifact bundle that I made in my forked repository.
https://github.com/mtj0928/Scipio/releases/download/0.21.0/scipio.artifactbundle.zip

@mtj0928 mtj0928 requested a review from giginet September 14, 2024 02:36
@mtj0928
Copy link
Collaborator Author

mtj0928 commented Sep 14, 2024

Oh, a new rule, optional_data_string_conversion, was added to SwiftLint in 0.57.0, which was released a few days ago.
It seems that the test code in Scipio violates this rule.

Copy link
Owner

@giginet giginet left a comment

Choose a reason for hiding this comment

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

I didn't know the update! It may be introduced since SwiftPM 6.0.

On CI, we are still using Xcode 15.4. It may use Package@swift-5.10. This job was only executed on PR was merged into the main so x86_64 support will not be worked well.

DEVELOPER_DIR: "/Applications/Xcode_15.4.app/Contents/Developer"

Could you update DEVELOPER_DIR to Xcode 16?

@mtj0928
Copy link
Collaborator Author

mtj0928 commented Sep 14, 2024

Actually, this is not Swift 6 feature!
I could success the job in my forked repository, and the job also used Xcode 15.4!
https://github.com/mtj0928/Scipio/actions/runs/10858648433/job/30136902542

@giginet giginet merged commit 390b8b6 into giginet:main Sep 14, 2024
4 checks passed
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