Copy built tools to stage sysroot#85496
Conversation
Motivation for this is to enable tools usage when using `rustup toolchain link`.
|
(rust-highfive has picked a reviewer for you, use r? to override) |
1ca6eb1 to
1cfec67
Compare
|
cc @vext01 |
|
Note: You will still need to add And unfortunately it seems like Possibly related to #82208 (comment)? (last bullet point) |
|
Hm, seems like I didn't properly understand the |
|
☔ The latest upstream changes (presumably #85711) made this pull request unmergeable. Please resolve the merge conflicts. |
| let bin = builder.tools_dir(compiler).join(&exe); | ||
| builder.copy(&cargo_out, &bin); | ||
|
|
||
| // Don't create a stage0-sysroot/bin directory. |
There was a problem hiding this comment.
Doing this for all tools is likely not a good idea, as it'll place things like tidy and other internal tools in the PATH (particularly on windows, that may be a problem). It should be possible to add this after the calls to ToolBuild, as we return the appropriate path already.
|
I'm going to go ahead and close this PR - but feel free to reopen or open a new one if you come back to finishing this work. Thanks! |
Motivation for this is to enable tools usage when using
rustup toolchain link.Just copies the tool directly after it's been built like the
Rustdocstep does it.Fixes #81431.