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

Error-index tests are spewing thousands of lines of errors #39327

Closed
alexcrichton opened this issue Jan 26, 2017 · 3 comments
Closed

Error-index tests are spewing thousands of lines of errors #39327

alexcrichton opened this issue Jan 26, 2017 · 3 comments

Comments

@alexcrichton
Copy link
Member

I've noticed this recently on the Travis/AppVeyor logs, but I haven't tracked it down yet to see what caused it.

@alexcrichton
Copy link
Member Author

Appears caused by #39199. @nikomatsakis perhaps caused by #39142?

@nikomatsakis
Copy link
Contributor

@alexcrichton ummmmmmmmm perhaps..... the tests are not failing though? Do you have a link to an example?

@alexcrichton
Copy link
Member Author

Oh right sorry I meant only that output is being generated, not that tests are failing. Most of the tests expect a failure which is the reason so many errors are printed.

An example is here (https://travis-ci.org/rust-lang/rust/jobs/192869604#L7886) from the logs of that PR (how nifty I can check that out!). You should be able to reproduce this locally as well via

./x.py test src/tools/error_index_generator

alexcrichton added a commit to alexcrichton/rust that referenced this issue Jan 27, 2017
Threads spawned by the test framework have their output captured by default, so
for `rustdoc --test` threads this propagates that capturing to the spawned
thread that we now have.

Closes rust-lang#39327
bors added a commit that referenced this issue Jan 31, 2017
rustdoc: Suppress warnings/errors with --test

Threads spawned by the test framework have their output captured by default, so
for `rustdoc --test` threads this propagates that capturing to the spawned
thread that we now have.

Closes #39327
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

2 participants