Skip to content

[Build Script Helper] Install libArgumentParserToolInfo.dylib into the toolchain #963

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

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Jan 4, 2022

This is now a dependency of the main libArgumentParser.dylib and not having it installed casuses Library not loaded errors when using the toolchain.

Resolves rdar://86859779

…the toolchain

This is now a dependency of the main `libArgumentParser.dylib` and not having it installed casuses `Library not loaded` errors when using the toolchain.

Resolves rdar://86859779
@artemcm artemcm requested a review from edymtt January 4, 2022 19:04
@artemcm
Copy link
Contributor Author

artemcm commented Jan 4, 2022

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Jan 4, 2022

We should transition to linking statically to avoid this kind of brittleness in the future, but for the time being this is needed to get main snapshot toolchains operational.

@artemcm artemcm requested review from xymus and tomerd January 5, 2022 19:56
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