Skip to content

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
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