Skip to content

[5.7] Speculatively work around issue with linking concurrency libs #4323

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
Apr 25, 2022

Conversation

neonichu
Copy link
Contributor

It looks like swift-build started crashing in the nightly toolchains since we introduced concurrency into SwiftPM:

objc[47001]: Class _TtCs27_KeyedEncodingContainerBase is implemented in both /usr/lib/swift/libswiftCore.dylib (0x7ff851b7d558) and /Library/Developer/Toolchains/swift-DEVELOPMENT-SNAPSHOT-2022-04-23-a.xctoolchain/usr/lib/swift/macosx/libswiftCore.dylib (0x10467ed38). One of the two will be used. Which one is undefined.

Revert "update example to use async await APIs (#4263)"

This reverts commit 745ce76.

It looks like `swift-build` started crashing in the nightly toolchains since we introduced concurrency into SwiftPM:

```
objc[47001]: Class _TtCs27_KeyedEncodingContainerBase is implemented in both /usr/lib/swift/libswiftCore.dylib (0x7ff851b7d558) and /Library/Developer/Toolchains/swift-DEVELOPMENT-SNAPSHOT-2022-04-23-a.xctoolchain/usr/lib/swift/macosx/libswiftCore.dylib (0x10467ed38). One of the two will be used. Which one is undefined.```

Revert "update example to use async await APIs (#4263)"

This reverts commit 745ce76.
@neonichu
Copy link
Contributor Author

We should separately find out what's going on here, eventually we'd like to use concurrency. Also the change remains on main for now.

@neonichu neonichu changed the title Speculatively work around issue with linking concurrency libs [5.7] Speculatively work around issue with linking concurrency libs Apr 25, 2022
@neonichu neonichu added the 5.7 label Apr 25, 2022
@neonichu neonichu self-assigned this Apr 25, 2022
@neonichu
Copy link
Contributor Author

@swift-ci please smoke test

@tomerd
Copy link
Contributor

tomerd commented Apr 25, 2022

interesting that the clash is on libswiftCore.dylib not libconcurrency.dylib

@neonichu
Copy link
Contributor Author

interesting that the clash is on libswiftCore.dylib not libconcurrency.dylib

The concurrency lib also shows up. There's a lot of output, I just truncated it to the first error.

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