Skip to content

Add verbosity when building swiftpm#79330

Merged
bkhouri merged 1 commit into
swiftlang:mainfrom
bkhouri:t/main/support-swifttesting-in-swiftpm-for-toolchain-build
Feb 14, 2025
Merged

Add verbosity when building swiftpm#79330
bkhouri merged 1 commit into
swiftlang:mainfrom
bkhouri:t/main/support-swifttesting-in-swiftpm-for-toolchain-build

Conversation

@bkhouri

@bkhouri bkhouri commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

@bkhouri

bkhouri commented Feb 13, 2025

Copy link
Copy Markdown
Contributor Author

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

bkhouri commented Feb 13, 2025

Copy link
Copy Markdown
Contributor Author

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

@swift-ci please test

@compnerd

Copy link
Copy Markdown
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
@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
bkhouri added a commit that referenced this pull request Oct 6, 2025
The SwiftPM stage added `--verbose` in #79330 to help troubleshoot build
error that were occuring on Linux.  These errors have since been
addresed, but the verbosity was not removed.

Remove the `--verbose` command line argument in the SwiftPM portion of
the `build-script`.
aviralg pushed a commit to aviralg/swift that referenced this pull request Oct 20, 2025
The SwiftPM stage added `--verbose` in swiftlang#79330 to help troubleshoot build
error that were occuring on Linux.  These errors have since been
addresed, but the verbosity was not removed.

Remove the `--verbose` command line argument in the SwiftPM portion of
the `build-script`.
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