Update SwiftBuild progress reporting#9519
Merged
bripeticca merged 22 commits intoswiftlang:mainfrom Jan 27, 2026
Merged
Conversation
* Planning start / complete under higher verbosity * Target up to date message emitted * Build progress messages only emitted if text is useful (omit fractional messages)
Contributor
Author
|
@swift-ci please test |
bripeticca
changed the title
bripeticca
requested review from
bkhouri,
cmcgee1024,
daveyc123,
dschaefer2,
owenv,
plemarquand and
rconnell9
as code owners
Contributor
Author
|
@swift-ci please test |
1 similar comment
Contributor
Author
|
@swift-ci please test |
Contributor
Author
|
@swift-ci please test |
Contributor
Author
|
@swift-ci please test windows |
Before, we were emitting a task started event on both stdout and stderr ONLY if the logLevel was verbose. Do the same here to avoid test failures; to revisit new mechanism for avoiding duplicate messages.
Contributor
Author
|
@swift-ci please test |
Contributor
|
@swift-ci please test |
Contributor
|
@swift-ci test macOS self hosted |
Contributor
|
@swift-ci test macOS |
Contributor
|
@swift-ci test macOS self hosted |
Contributor
|
@swift-ci test |
Contributor
Author
|
@swift-ci please test |
Contributor
Author
|
@swift-ci please test windows |
Contributor
Author
|
@swift-ci test self hosted linux |
Contributor
Author
|
@swift-ci please test linux |
Contributor
Author
|
@swift-ci test macOS |
Contributor
Author
|
@swift-ci please test |
Contributor
Author
|
@swift-ci test windows |
Contributor
Author
|
@swift-ci test |
Contributor
Author
|
@swift-ci test windows |
Contributor
Author
|
@swift-ci test macOS |
Contributor
Author
|
@swift-ci test windows |
Contributor
Author
|
@swift-ci test self hosted macOS |
Contributor
Author
|
@swift-ci test macOS |
Contributor
Author
|
@swift-ci test windows |
2 similar comments
Contributor
Author
|
@swift-ci test windows |
Contributor
Author
|
@swift-ci test windows |
bripeticca
added a commit
to bripeticca/swift-package-manager
that referenced
this pull request
Jan 27, 2026
Addresses: * Comments under swiftlang#9362 related to testing * Removes the stepping fraction from being printed when dealing with events that report a negative percentage; the idea here is that if the progress is in the negative, this seems to be a pre-build step where there is no real benefit to reporting the "percentage complete" here. * Omit messages that do not contain useful information (i.e. messages that are of the format "14 / 27" without any identifying text, etc.) * Emit planning operation start + complete in a similar fashion to the native build system * Emit instances of a target being already up to date and not requiring a build.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Addresses: