Skip to content

utils: update Foundation build configuration#90025

Merged
compnerd merged 1 commit into
swiftlang:mainfrom
compnerd:android
Jun 18, 2026
Merged

utils: update Foundation build configuration#90025
compnerd merged 1 commit into
swiftlang:mainfrom
compnerd:android

Conversation

@compnerd

Copy link
Copy Markdown
Member

When building Foundation for Android, we need to pass in the path to the OpenSSL libraries now that we build against the custom OpenSSL (BoringSSL) build.

When building Foundation for Android, we need to pass in the path to the
OpenSSL libraries now that we build against the custom OpenSSL
(BoringSSL) build.
@compnerd

Copy link
Copy Markdown
Member Author

@swift-ci please smoke test

@compnerd

Copy link
Copy Markdown
Member Author

CC: @finagolfin

I missed this in the previous set, it should repair the Android build.

@compnerd compnerd enabled auto-merge June 17, 2026 17:59
@compnerd

Copy link
Copy Markdown
Member Author

@swift-ci please smoke test Linux platform

@compnerd

Copy link
Copy Markdown
Member Author

@swift-ci please smoke test macOS platform

@compnerd compnerd merged commit f6e93b6 into swiftlang:main Jun 18, 2026
3 checks passed
@compnerd compnerd deleted the android branch June 18, 2026 05:07
@finagolfin

Copy link
Copy Markdown
Member

The Foundation build for Android on the Windows toolchain CI is still failing with this pull applied:

-- Build files have been written to: T:/aarch64-unknown-linux-android23/DynamicFoundation
ninja: error: 'T:/aarch64-unknown-linux-android23/DynamicBrotli/brotlidec.lib', needed by 'lib/libFoundationNetworking.so', missing and no known rule to make it
Error: Error: cmake.exe exited with code 1.

That pull is checking trunk using NDK 30 beta 1, but the minimal changes I made shouldn't affect this underlying Foundation config.

@compnerd

Copy link
Copy Markdown
Member Author

Ugh, sorry, not sure how that got through CI in the past. Thanks for pointing that out.

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