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

"make check-notidy" runs tidy #3105

Closed
catamorphism opened this issue Aug 4, 2012 · 1 comment
Closed

"make check-notidy" runs tidy #3105

catamorphism opened this issue Aug 4, 2012 · 1 comment
Milestone

Comments

@catamorphism
Copy link
Contributor

--> make check-notidy TESTNAME=issue-3099
cfg: shell host triple x86_64-apple-darwin
cfg: disabling rustc optimization (CFG_DISABLE_OPTIMIZE)
cfg: host for x86_64-apple-darwin is x86_64
cfg: unix-y environment
cfg: using gcc
cfg: disabling valgrind (CFG_DISABLE_VALGRIND)
cfg: no llnextgen found, omitting grammar-verification
cfg: including dist rules
cfg: including test rules
rustdoc: doc/core/index.html
rustdoc: doc/std/index.html
compile_and_link: x86_64-apple-darwin/stage1/lib/rustc/x86_64-apple-darwin/bin/fuzzer
cp: x86_64-apple-darwin/stage2/bin/fuzzer
compile_and_link: x86_64-apple-darwin/stage1/lib/rustc/x86_64-apple-darwin/bin/cargo
cp: x86_64-apple-darwin/stage2/bin/cargo
check: formatting
/Users/tchevalier/rust/src/rustc/middle/resolve3.rs:821: line longer than 78 chars
/Users/tchevalier/rust/src/rustc/middle/resolve3.rs:828: trailing whitespace
@ghost ghost assigned graydon Aug 4, 2012
@brson
Copy link
Contributor

brson commented Aug 5, 2012

likely because of this line: check-stage$(1)-T-$(2)-H-$(3): tidy. There's also a CFG_NOTIDY variable that turns off tidy for every rule. Removing it from the per-stage rules seems fine to me.

RalfJung pushed a commit to RalfJung/rust that referenced this issue Oct 6, 2023
miri-script: print which sysroot target we are building
RalfJung pushed a commit to RalfJung/rust that referenced this issue Oct 7, 2023
miri-script: print which sysroot target we are building
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
This allows benchcomp to pass the list of results to an external program
for modification before the results are visualized. This can be used,
for example, to visualize only a relevant subset of results.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.
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

No branches or pull requests

3 participants