Skip to content

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented Oct 10, 2025

To achieve this, add a new cache variable
SWIFT_EMBEDDED_STDLIB_SDKS_FOR_TARGET_TRIPLES to be set like in the
following examples.

-DSWIFT_EMBEDDED_STDLIB_SDKS_FOR_TARGET_TRIPLES=aarch64-vendor-os@/usr/local/aarch64-vendor-os-sdk;aaarch-vendor-anotheros@/opt/aarch64-vendor-anotheros-sdk

We chose to use another setting instead of e.g. folding this into
SWIFT_EMBEDDED_STDLIB_EXTRA_TARGET_TRIPLES so it is clear this is opt
in and it does not regress existing configurations that set the SDK
directly (like it's the case for the WASM stdlib).

Addresses rdar://162368529

@edymtt edymtt requested review from a team and ktoso as code owners October 10, 2025 17:02
@edymtt
Copy link
Contributor Author

edymtt commented Oct 10, 2025

@swift-ci please smoke test

@edymtt
Copy link
Contributor Author

edymtt commented Oct 10, 2025

@swift-ci please build toolchain

@edymtt edymtt force-pushed the edymtt/specify-sdks-for-embededded-swift branch from 6dcf1ab to a1c98eb Compare October 13, 2025 15:59
@edymtt
Copy link
Contributor Author

edymtt commented Oct 13, 2025

@swift-ci please smoke test

@edymtt
Copy link
Contributor Author

edymtt commented Oct 13, 2025

@swift-ci please build toolchain

To achieve thig, add a new cache variable
`SWIFT_EMBEDDED_STDLIB_SDKS_FOR_TARGET_TRIPLES` to be set like in the
following examples.

```
-DSWIFT_EMBEDDED_STDLIB_SDKS_FOR_TARGET_TRIPLES=aarch64-vendor-os@/usr/local/aarch64-vendor-os-sdk;aaarch-vendor-anotheros@/opt/aarch64-vendor-anotheros-sdk
```

We chose to use another setting instead of e.g. folding this into
`SWIFT_EMBEDDED_STDLIB_EXTRA_TARGET_TRIPLES` so it is clear this is opt
in and it does not regress existing configurations that set the SDK
directly (like it's the case for the WASM stdlib).

Addresses rdar://162368529
@edymtt edymtt force-pushed the edymtt/specify-sdks-for-embededded-swift branch from a1c98eb to 2074116 Compare October 13, 2025 22:14
@edymtt
Copy link
Contributor Author

edymtt commented Oct 13, 2025

@swift-ci please smoke test

@edymtt
Copy link
Contributor Author

edymtt commented Oct 13, 2025

@swift-ci please build toolchain

@edymtt
Copy link
Contributor Author

edymtt commented Oct 14, 2025

@swift-ci please build toolchain macOS

@edymtt
Copy link
Contributor Author

edymtt commented Oct 14, 2025

@swift-ci please smoke test Windows

1 similar comment
@edymtt
Copy link
Contributor Author

edymtt commented Oct 14, 2025

@swift-ci please smoke test Windows

@edymtt edymtt merged commit e798e9c into swiftlang:main Oct 15, 2025
6 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.

1 participant