Skip to content

Conversation

@ahoppen
Copy link
Member

@ahoppen ahoppen commented Jun 6, 2024

The BuildParameters that we were previously inspecting contained some parameters that SwiftPM synthesizes and that weren’t specified on the command line. For example, it contains -g for all languages. This causes us to add -Xswiftc -g -Xcc -g -Xcxx -g to the swift build invocations for preparation, which is not necessary.

…swift build` invocations during preparation

The `BuildParameters` that we were previously inspecting contained some parameters that SwiftPM synthesizes and that weren’t specified on the command line. For example, it contains `-g` for all languages. This causes us to add `-Xswiftc -g -Xcc -g -Xcxx -g` to the `swift build` invocations for preparation, which is not necessary.
@ahoppen ahoppen requested review from bnbarham and hamishknight June 6, 2024 20:38
@ahoppen ahoppen requested a review from benlangmuir as a code owner June 6, 2024 20:38
@ahoppen
Copy link
Member Author

ahoppen commented Jun 6, 2024

@swift-ci Please test

@ahoppen ahoppen enabled auto-merge June 6, 2024 21:35
@ahoppen ahoppen merged commit 76a0db7 into swiftlang:main Jun 6, 2024
@ahoppen ahoppen deleted the use-build-setup branch June 6, 2024 23:23
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