Skip to content

Conversation

@dschaefer2
Copy link
Member

Add a hidden HostCondition that will check the build parameters to see if we're building a target for host enable the include and link settings to use the prebuilts and disable the dependency on swift-syntax products.

Doesn't support SwiftBuild yet. Need to figure out how to use the same condition there.

Also cleaned up BuildPlanTests so they are just Native BuildPlan tests as designed and created a separate BuildTests for the two tests that used the build system kind setting.

Add a hidden HostCondition that will check the build parameters to see if we're building a target for host enable the include and link settings to use the prebuilts and disable the dependency on swift-syntax products. Doesn't support SwiftBuild yet. Need to figure out how to use the same condition there. Also cleaned up BuildPlanTests so they are just Native BuildPlan tests as designed and created a separate BuildTests for the two tests that used the build system kind setting.
@dschaefer2
Copy link
MemberAuthor

@swift-ci please test

/// A configuration condition implies that an assignment is valid on
/// a particular build configuration.
/// By the time we get to evaluating the condition, the package loader would have filtered out
/// disabled traits. As such, this should always evaluate to true.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

praise: thank you for updating this description!

@dschaefer2
Copy link
MemberAuthor

BTW, Will get SwiftBuild working before this is ready.

@dschaefer2dschaefer2 changed the title Use prebuilts for all targets that build for destination == host.[WIP] Use prebuilts for all targets that build for destination == host.Dec 12, 2025
@dschaefer2
Copy link
MemberAuthor

@swift-ci please test

Sign up for freeto 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

@dschaefer2@bripeticca