Skip to content

Add additional information #8291

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 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 bkhouri force-pushed the b/main/bootstrap_ensure_we_flush_stdout_and_stderr_on_command_failure branch from 6efa607 to 5ff251b Compare February 13, 2025 01:41
@bkhouri bkhouri force-pushed the b/main/bootstrap_ensure_we_flush_stdout_and_stderr_on_command_failure branch from 5ff251b to 30bf8f1 Compare February 13, 2025 01:42
@bkhouri
Copy link
Contributor Author

bkhouri commented Feb 13, 2025

Test with
swiftlang/swift#79330

@swift-ci please test

@bkhouri bkhouri merged commit bb959be into swiftlang:main Feb 14, 2025
5 checks passed
@bkhouri bkhouri deleted the b/main/bootstrap_ensure_we_flush_stdout_and_stderr_on_command_failure branch February 14, 2025 17:39
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.

3 participants