Skip to content

Add verbosity when building swiftpm #79330

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

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Feb 12, 2025

There have been cases where CI build failures on Linux did not have sufficient logs when it failed during a Swift PM build/test. Augment the Swift PM build/test by enabling verbosity!

@compnerd
Copy link
Member

Should this include the Windows side of the world as well? (i.e. build.ps1)

@bkhouri
Copy link
Contributor Author

bkhouri commented Feb 13, 2025

Should this include the Windows side of the world as well? (i.e. build.ps1)

@compnerd Likely, yes. I'm not familiar with Powershell script. Could you please share an equivalent diff that I can apply?

EDIT: After looking at the build.ps1, it does not appear the build-script is used on Windows. so this change may not be necessary as the build.ps1 does not appear to use SwiftPM Utilities/bootstrap script, though I could be wrong.

@bkhouri
Copy link
Contributor Author

bkhouri commented Feb 13, 2025

please test with PR:
swiftlang/swift-package-manager#8291

@swift-ci please test

@compnerd
Copy link
Member

Should this include the Windows side of the world as well? (i.e. build.ps1)

@compnerd Likely, yes. I'm not familiar with Powershell script. Could you please share an equivalent diff that I can apply?

EDIT: After looking at the build.ps1, it does not appear the build-script is used on Windows. so this change may not be necessary as the build.ps1 does not appear to use SwiftPM Utilities/bootstrap script, though I could be wrong.

Ah, I didn't realise that this was only for the build of SPM itself. Yes, we build SPM with CMake on Windows.

@bkhouri bkhouri merged commit ac52024 into swiftlang:main Feb 14, 2025
5 checks passed
@bkhouri bkhouri deleted the t/main/support-swifttesting-in-swiftpm-for-toolchain-build branch February 14, 2025 17:39
bkhouri added a commit to swiftlang/swift-package-manager that referenced this pull request Feb 14, 2025
There have been cases where CI build failurse did not have sufficient
logs when it failed during a Swift PM build/test. Augment the Swift PM
build/test by enabling verbosity. Update `Utilities/bootstrap` to call
swift pm executable targets using `--very-verbose` instead of
`--versobe` when the tool is invoked with `--verbose`.

Requires swiftlang/swift#79330
bripeticca pushed a commit to bripeticca/swift-package-manager that referenced this pull request Feb 28, 2025
There have been cases where CI build failurse did not have sufficient
logs when it failed during a Swift PM build/test. Augment the Swift PM
build/test by enabling verbosity. Update `Utilities/bootstrap` to call
swift pm executable targets using `--very-verbose` instead of
`--versobe` when the tool is invoked with `--verbose`.

Requires swiftlang/swift#79330
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.

4 participants