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

Extremely verbose GCC build fills CI log for x86_64-gnu-llvm-18 #138213

Closed
RalfJung opened this issue Mar 8, 2025 · 3 comments · Fixed by #138232
Closed

Extremely verbose GCC build fills CI log for x86_64-gnu-llvm-18 #138213

RalfJung opened this issue Mar 8, 2025 · 3 comments · Fixed by #138232
Labels
A-CI Area: Our Github Actions CI A-gcc Things relevant to the [future] GCC backend C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@RalfJung
Copy link
Member

RalfJung commented Mar 8, 2025

See for instance the log here: more than 80% of the logfile seem to be about the GCC build, completely drowning any signal we might get from the rest of the build. I don't think that has always been the case? Please be mindful of CI log spam when adding new things to the build. :)

Cc @GuillaumeGomez @antoyo @Kobzol

@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Mar 8, 2025
@RalfJung RalfJung changed the title Extremelt verbose GCC build fills CI log for x86_64-gnu-llvm-18 Extremely verbose GCC build fills CI log for x86_64-gnu-llvm-18 Mar 8, 2025
@jieyouxu jieyouxu added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. C-bug Category: This is a bug. A-gcc Things relevant to the [future] GCC backend A-CI Area: Our Github Actions CI and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Mar 8, 2025
@Kobzol
Copy link
Contributor

Kobzol commented Mar 8, 2025

Sorry for that. Soon-ish that build should be cached most of the time, but still it's probably a good idea to reduce the verbosity here. Any idea of how to force make do that, but still get some useful (less verbose) output (antoyo or Guillaume :) )?

@antoyo
Copy link
Contributor

antoyo commented Mar 8, 2025

You have two options:

  • make -s, but this will still show the configure messages.
  • make -s > /dev/null if the above is not enough (that will still show warnings and errors)

@Kobzol
Copy link
Contributor

Kobzol commented Mar 8, 2025

Thanks, that second one is probably the better choice, because the configure output was still quite spammy. I'll send a PR.

jieyouxu added a commit to jieyouxu/rust that referenced this issue Mar 9, 2025
…llaumeGomez

Reduce verbosity of GCC build log

It was a bit too spammy before.

Fixes: rust-lang#138213

r? `@GuillaumeGomez`
@bors bors closed this as completed in 37ff680 Mar 9, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Mar 9, 2025
Rollup merge of rust-lang#138232 - Kobzol:gcc-reduce-verbosity, r=GuillaumeGomez

Reduce verbosity of GCC build log

It was a bit too spammy before.

Fixes: rust-lang#138213

r? ``@GuillaumeGomez``
lnicola pushed a commit to lnicola/rust-analyzer that referenced this issue Mar 10, 2025
Reduce verbosity of GCC build log

It was a bit too spammy before.

Fixes: rust-lang/rust#138213

r? ``@GuillaumeGomez``
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI Area: Our Github Actions CI A-gcc Things relevant to the [future] GCC backend C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants