-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Diff on rustdoc test failure does not work on Windows #82409
Comments
How do you diff directories recursively on Windows? On Linux I would be very surprised for
Oh I just thought of something - do you have a rustup overide that defaults to your local toolchain? Right now compiletest assumes the default is nightly, I've been meaning to change it. |
No, I only set overrides on specific projects (and rust-lang/rust is not one of them). My default is the stable compiler. |
Git for Windows provides
Are you building Rust using Git Bash prompt? |
I don't think it'd be a good idea to require testing rustdoc in the Git for Windows bash shell. I'd like for this to work in cmd.exe and PowerShell if possible. |
…k-Simulacrum Use a crate to produce rustdoc tree comparisons instead of the `diff` command It doesn't come with Windows, so bring [our own](https://github.com/notriddle/rust-unified-diff/). Fixes rust-lang#82409 ![image](https://user-images.githubusercontent.com/1593513/109230755-9a1d5700-7782-11eb-8359-353a506875ab.png)
…k-Simulacrum Use a crate to produce rustdoc tree comparisons instead of the `diff` command It doesn't come with Windows, so bring [our own](https://github.com/notriddle/rust-unified-diff/). Fixes rust-lang#82409 ![image](https://user-images.githubusercontent.com/1593513/109230755-9a1d5700-7782-11eb-8359-353a506875ab.png)
…k-Simulacrum Use a crate to produce rustdoc tree comparisons instead of the `diff` command It doesn't come with Windows, so bring [our own](https://github.com/notriddle/rust-unified-diff/). Fixes rust-lang#82409 ![image](https://user-images.githubusercontent.com/1593513/109230755-9a1d5700-7782-11eb-8359-353a506875ab.png)
When running
x .py test .\src\test\rustdoc
on Linux and a failure happens, a really nice diff of the output vs rustdoc-nightly is produced which very clearly shows the issue. This does not work on Windows for multiple reasons.One the call to
diff
fails:Second, the output from rustdoc-nightly seems to exactly match that from the local compiler making it not possible to do manual diffs.
The text was updated successfully, but these errors were encountered: