Skip to content
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

Use the new SwiftPM API to load the build plan #1973

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Feb 8, 2025

We previously skipped building/running tool plugins here, which meant that the compiler arguments for a target also missed any generated sources. Use the new BuildDescription.load API from SwiftPM to address this.

Fixes #665
Resolves rdar://102242345.
Resolves rdar://144557689.

@bnbarham bnbarham requested a review from ahoppen as a code owner February 8, 2025 04:14
@bnbarham bnbarham force-pushed the generated-source-should-work branch from 3e7ca95 to 0c4c75c Compare February 8, 2025 04:15
@bnbarham
Copy link
Contributor Author

bnbarham commented Feb 8, 2025

@bnbarham
Copy link
Contributor Author

bnbarham commented Feb 8, 2025

Will need to update this to only build plugins when background indexing is enabled so that we don’t conflict with the regular build directory.

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

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

Nice 😍

@bnbarham
Copy link
Contributor Author

@bnbarham
Copy link
Contributor Author

swiftlang/swift-package-manager#8286

@swift-ci please test Windows platform

@bnbarham bnbarham force-pushed the generated-source-should-work branch from 95e49b5 to 54a1e02 Compare February 11, 2025 21:29
@bnbarham
Copy link
Contributor Author

@bnbarham
Copy link
Contributor Author

swiftlang/swift-package-manager#8286

@swift-ci please test Windows platform

@bnbarham bnbarham force-pushed the generated-source-should-work branch 2 times, most recently from 61b34fb to d762363 Compare February 12, 2025 05:02
@bnbarham
Copy link
Contributor Author

@bnbarham
Copy link
Contributor Author

swiftlang/swift-package-manager#8286

@swift-ci please test Windows platform

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

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

Thanks. Looks good to me

We previously skipped building/running tool plugins here, which meant
that the compiler arguments for a target also missed any generated
sources. Use the new `BuildDescription.load` API from SwiftPM to address
this.

Resolves rdar://102242345.
@bnbarham bnbarham force-pushed the generated-source-should-work branch from d762363 to f142b5c Compare February 12, 2025 23:22
@bnbarham
Copy link
Contributor Author

@bnbarham
Copy link
Contributor Author

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.

Source generated by build plugin is ignored
2 participants