-
Notifications
You must be signed in to change notification settings - Fork 124
Add a bare metal plugin (for "none" triples) #732
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
Conversation
@swift-ci test |
Please keep in mind that a none triple does not mean Swift Embedded, or that you even have Swift at all. Do not make assumptions that you're in such an environment when using this SDK/platform. It should generalize to any "none" triple, including a plain C environment. |
803970c
to
b1c9466
Compare
@swift-ci test |
Correct. From the SwiftPM perspective, these platforms are generally accompanied by a toolset.json file which contains most importantly a That said, the extraCLIOptions from the toolset get tacked into the extraArgs and passed to the SwiftBuild request along with the other swift build CLI args. Longer term, we probably should look at how to do these platforms as proper SDKs with an SDKSettings.json instead of the toolset.json/sysroot to unify how we present platforms to SwiftBuild. |
Maybe. It's not clear to me yet whether SDKSettings should be the canonical format used everywhere vs one remaining specific to Apple platforms. Its schema is fairly ad-hoc and maybe not the best fit for something more general purpose. |
b1c9466
to
acef58b
Compare
@swift-ci test |
@swift-ci test macOS |
acef58b
to
b58dafc
Compare
@swift-ci test |
@swift-ci test macOS |
This is just going in SWBUniversalPlatform for now for simplicity / bringup, and should maybe move into a new SWBNonePlatformPlugin later on. Closes swiftlang#726
b58dafc
to
457abfc
Compare
@swift-ci test |
This is just going in SWBUniversalPlatform for now for simplicity / bringup, and should maybe move into a new SWBNonePlatformPlugin later on.
Closes #726