-
Notifications
You must be signed in to change notification settings - Fork 18
Add testing for swift-corelibs-foundation-android-test repo #170
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
Add testing for swift-corelibs-foundation-android-test repo #170
Conversation
I am currently in the process of updating the CI to cross-compile the latest snapshot tags of the SDK, including the two updated foundation repos for Android. So far it works well for trunk 6.1, going to try 6.0 next. I had not thought of building and running the foundation tests too, but you are right that since these are structured as Swift packages now, it should be possible to cross-compile the foundation test runners too. Hold off for a bit till I get that in: it may be easier to use those patches to build these tests, as I use patches like swiftlang/swift-foundation#871 natively on Android AArch64 to get almost all the tests passing. Also, I see a strange TLS issue when building this test runner natively, that required some workarounds to get all the tests running and get their output. |
Great! Is this pushed to a branch somewhere? Let me know if there is anything I can do to help … I'm very eager to get going with Swift 6 and swift-foundation.
Will do. I'll leave this as a draft PR for the time being, but feel free to close it and I'll open a new one later if you think that would be better.
Do you mean a TLS issue with URLSession and HTTPS URLs on Android? If so, you may be interested in this: https://gist.github.com/marcprux/34dd8da76ea56dca719b47b4b0072b98 |
No, still cleaning up and extracting my local patches.
OK, I will get that up by tomorrow, so we can iterate on it independently from then on.
Sounds good.
No, I mean Thread-Local Storage, which I now see the Swift stdlib just added for the first time this summer on linux platforms. This caused no problems for me before because it was always invoked from the shared library I'm not sure exactly why it causes problems for executables alone- haven't dug into it- but it doesn't appear to be using the emulated TLS talked about there. Could be a simple mis-configuration on my end, will need to look into it. |
I'm still sorting through the Android patches for Swift 6, should have that up tonight. |
Unless you really care about checking the APIs in |
The last time I looked, the |
I don't think we need this. Closing. |
I'll try to get this in once we have the emulated TLS fix. |
This PR adds testing for the https://github.com/marcprux/swift-corelibs-foundation-android-test.git repository, which is a fork of https://github.com/apple/swift-corelibs-foundation.git that builds only the
Tests/
part of the package and adapts a number of Foundation tests to accommodate Android. A number of tests are disabled (mostly some daylight savings and Process forking tests), but most pass on the Android emulator.Having this baseline of tests that pass for
swift-corelibs-foundation
will be useful to see what breaks (or is fixed) whenswift-foundation
is added to the mix.Note that this PR relies on the additional Termux libraries added in #167, since those are required to run the
XMLParser
andURLSession
tests.