Skip to content

Build rustdoc only at the topmost stage.#43528

Closed
Mark-Simulacrum wants to merge 1 commit into
rust-lang:masterfrom
Mark-Simulacrum:single-rustdoc
Closed

Build rustdoc only at the topmost stage.#43528
Mark-Simulacrum wants to merge 1 commit into
rust-lang:masterfrom
Mark-Simulacrum:single-rustdoc

Conversation

@Mark-Simulacrum

Copy link
Copy Markdown
Member

This effectively ensures that rustdoc will only ever be built once for
any given host. This does not introduce cycles into the build since we
only depend on rustdoc from documentation or test steps, which run after
the required compiler was built.

I've tested locally and ./x.py doc --stage 1 src/libstd does not build more than all stage0 artifacts and stage1 libstd.

I'm not entirely sure that this will work, but I think it should. We'll see if Travis passes.

This effectively ensures that rustdoc will only ever be built once for
any given host. This does not introduce cycles into the build since we
only depend on rustdoc from documentation or test steps, which run after
the required compiler was built.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@Mark-Simulacrum

Copy link
Copy Markdown
Member Author

r? @alexcrichton

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Jul 28, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit ff2f7c5 has been approved by alexcrichton

@Mark-Simulacrum

Copy link
Copy Markdown
Member Author

Hm, I actually want to run some more tests on this before we land it. I worry that it might break testing stage1 std, but am uncertain.

@bors r-

@carols10cents carols10cents added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jul 31, 2017
@shepmaster

Copy link
Copy Markdown
Member

@Mark-Simulacrum This is your friendly 7-day ping for activity!

@Mark-Simulacrum

Copy link
Copy Markdown
Member Author

Closing in favor of #43630.

@Mark-Simulacrum Mark-Simulacrum deleted the single-rustdoc branch August 4, 2017 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants