Skip to content

Update CI environment - #321

Merged
sidepelican merged 3 commits into
uber:masterfrom
sidepelican:fix_ci
Nov 13, 2025
Merged

sidepelican merged 3 commits into
uber:masterfrom
sidepelican:fix_ci

Conversation

@sidepelican

Copy link
Copy Markdown
Collaborator

The CI configuration had become outdated.
I update it to ensure everything works correctly.

Comment thread install-script.sh
Comment on lines +68 to +70
Linux*) swift build -c release \
-Xlinker -rpath -Xlinker /usr/lib/swift/linux \
-Xlinker -rpath -Xlinker /usr/share/swift/usr/lib/swift/linux

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Binaries built with Swiftly set the rpath to Swiftly's toolchain, this has portability issues. So I using a standard rpath instead:

  • /usr/lib/swift/linux
    The standard path used for Linux builds
  • /usr/share/swift/usr/lib/swift/linux
    The path used by GitHub Actions

@uhooi uhooi left a comment

Copy link
Copy Markdown
Collaborator

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 54a59e1 into uber:master Nov 13, 2025
8 checks passed
@sidepelican
sidepelican deleted the fix_ci branch February 5, 2026 03:28
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