Skip to content
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

CI: reduce verbosity of build.sh #3715

Merged
merged 1 commit into from
Jun 16, 2024
Merged

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented May 19, 2024

-vv produces lots of unrelated warnings in other crates when building
with -Zbuild-std. That makes it hard to load and see errors when tests fail.

A single -v is enough for most testing purposes.

Build log size before and after:

Before: ~12 MB (raw build log)
After: 2,127.86 KB (raw build log)

@rustbot
Copy link
Collaborator

rustbot commented May 19, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

`-vv` produces lots of unrelated warnings in other crates when building
with `-Zbuild-std`. A single `-v` is enough for most testing purposes.

Build log size before and after:

Before: ~12 MB
After: 71.15 KB
@JohnTitor JohnTitor enabled auto-merge June 16, 2024 11:09
@JohnTitor JohnTitor added this pull request to the merge queue Jun 16, 2024
Merged via the queue into rust-lang:libc-0.2 with commit b9b31dc Jun 16, 2024
57 checks passed
@tesuji tesuji deleted the ci-verbosity branch June 16, 2024 15:30
@tgross35 tgross35 added the main-needs-cherry-pick This PR was merged against libc-0.2 but does not yet exist in main. label Aug 13, 2024
This was referenced Aug 15, 2024
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Aug 29, 2024
`-vv` produces lots of unrelated warnings in other crates when building
with `-Zbuild-std`. A single `-v` is enough for most testing purposes.

Build log size before and after:

Before: ~12 MB
After: 71.15 KB

(apply <rust-lang#3715> to `main`)
(cherry picked from commit 9c7a4d5)
@tgross35 tgross35 added main-applied This PR was merged against libc-0.2 and main has since been updated to include it and removed main-needs-cherry-pick This PR was merged against libc-0.2 but does not yet exist in main. labels Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
main-applied This PR was merged against libc-0.2 and main has since been updated to include it S-waiting-on-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants