Skip to content

Remove URL workaround for Linux crash once SCL-F is rebuilt #1252

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 23, 2025

Conversation

jrflat
Copy link
Contributor

@jrflat jrflat commented Apr 16, 2025

Tested with a recent nightly-main container, and removing the Linux-only workaround unsafeBitCast == 0x1 check works now that swift-corelibs-foundation was built with the new swift-foundation URL changes in #1238.

Removing the other private var _padding workaround still causes a crash in libXCTest on Linux, so that will require further investigation.

@jrflat
Copy link
Contributor Author

jrflat commented Apr 16, 2025

@swift-ci please test

@jrflat jrflat marked this pull request as draft April 16, 2025 20:59
@jrflat
Copy link
Contributor Author

jrflat commented Apr 16, 2025

Works locally on my Ubuntu 22.04 aarch64 image, but fails in CI's Ubuntu 22.04 x86 image

@jrflat
Copy link
Contributor Author

jrflat commented Apr 17, 2025

@swift-ci please test Linux Platform

@jrflat
Copy link
Contributor Author

jrflat commented Apr 21, 2025

Tested on an x86 Ubuntu 22.04 image locally with swift-DEVELOPMENT-SNAPSHOT-2025-04-12-a and it worked. Looks like CI was using an older swift-DEVELOPMENT-SNAPSHOT-2025-04-03-a based on the commit it logged. Let's try again

@jrflat
Copy link
Contributor Author

jrflat commented Apr 21, 2025

@swift-ci please test Linux Platform

@jrflat jrflat marked this pull request as ready for review April 21, 2025 19:39
@jrflat
Copy link
Contributor Author

jrflat commented Apr 23, 2025

swift-foundation tests pass in CI now that the newer toolchain is being used. I also ran Swift smoke tests to sanity check, and those passed, too.

@jrflat jrflat requested review from jmschonfeld and itingliu April 23, 2025 18:17
@jrflat jrflat merged commit 27476f9 into swiftlang:main Apr 23, 2025
3 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