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.

Sign up for GitHub

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

Jump to bottom

rust.rc driver wrapping rustdoc does not propogate some errors #7380

Closed
pnkfelix opened this issue Jun 25, 2013 · 1 comment
Closed

rust.rc driver wrapping rustdoc does not propogate some errors #7380

pnkfelix opened this issue Jun 25, 2013 · 1 comment
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@pnkfelix
Copy link
Member 

% echo > tmp.rs && rustdoc tmp.rs --output-format html --output-dir foo ; echo $?
rust: ~"pandoc-out: "
rust: ~"pandoc-err: pandoc: foo/index.html: openFile: does not exist (No such file or directory)\n"
rust: task failed at 'pandoc failed', /Users/fklock/Dev/Mozilla/rust.git/src/librustdoc/markdown_writer.rs:121
rust: domain main @0x7fb163011a10 root task failed
101
% echo > tmp.rs && rust doc tmp.rs --output-format html --output-dir foo ; echo $?
0
@pnkfelix pnkfelix mentioned this issue Jun 25, 2013
Closed
@catamorphism
Copy link
Contributor

I suspect this might be because fail doesn't set the exit code. Not a blocker IMO, anyway, but should be fixed.

alexcrichton added a commit to alexcrichton/rust that referenced this issue Sep 24, 2013
@alexcrichton
Use sundown for markdown highlighting in rustdoc
19ac447 
This takes rendering times of documentation down from 30s to 0.5s. Kinda sad
that none of the parallelism is needed, but oh well!

Closes rust-lang#7380
cc rust-lang#3546
flip1995 pushed a commit to flip1995/rust that referenced this issue Jul 1, 2021
@bors
Auto merge of rust-lang#7380 - popzxc:compile-test-helper, r=flip1995
d3327bd 
Improve panic message on "Found multiple rlibs" error in compile-test

Related to rust-lang#7343

When I first met this error I was pretty much confused, so I thought it may be a good idea to:

- Give a hint on what to do to users that don't want to dig into specifics and just want to quickly resolve the issue.
- Give a link for those who are interested in details.

## Old appearance:

<img width="1121" alt="Screenshot 2021-06-20 at 08 30 34" src="https://user-images.githubusercontent.com/12111581/122663361-df8ae780-d1aa-11eb-9236-775b4fd754d5.png">

## New appearance:

<img width="1121" alt="Screenshot 2021-06-20 at 08 32 18" src="https://user-images.githubusercontent.com/12111581/122663363-e4e83200-d1aa-11eb-9c46-f62d83eb79e2.png">

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pnkfelix @catamorphism