-
Notifications
You must be signed in to change notification settings - Fork 206
Jobs: do not inject the swiftautolink file into archives #777
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
Jobs: do not inject the swiftautolink file into archives #777
Conversation
It has been observed that a static library may sometimes contain the autolink extracted rules. This causes autolink-extract to not be able to process the archive as a dependency, causing a build failure.
@swift-ci please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this.
Would be good to get this in. I'm putting together a pull to download the latest 5.5 snapshots and test them on my daily Android CI, and I currently have to disable using this new Swift driver because of this bug. |
@DougGregor, would be good to get this in, or using Swift static libraries on linux will regress with the 5.5 release. I'd like to get #784 in too. |
@artemcm, can we get this in too? This currently breaks building the Foundation static libraries for Android, when it tries to link |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one is good to go.
cc @airspeedswift
Ping @airspeedswift, would be good to get this in. |
Pinging @airspeedswift, can we get this into the point release? |
It has been observed that a static library may sometimes contain the
autolink extracted rules. This causes autolink-extract to not be able
to process the archive as a dependency, causing a build failure.