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

linkchecker: running from a directory separate from the book #110565

Merged
merged 1 commit into from
Apr 20, 2023

Conversation

weihanglo
Copy link
Member

@weihanglo weihanglo commented Apr 19, 2023

Since rust-lang/cargo#11851, Cargo became a Cargo workspace of
itself. However, since src/tools/linkchecker cannot run inside
a workspace, Cargo needs a workaround that excludes src/doc
from workspace member probing.

To remove this hack, this PR adds a new optional argument --path
for linkchecker.sh. With this new argument, linkchecker.sh can
be run from a directory separate from the book. This also benefits
other projects using linkchecker, as they can run it under target
directory or any other directory, reducing leftover.

@rustbot
Copy link
Collaborator

rustbot commented Apr 19, 2023

r? @ehuss

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 19, 2023
Since rust-lang/cargo#11851, Cargo became a Cargo workspace of
itself. However, since `src/tools/linkchecker` cannot run inside
a workspace, Cargo needs a workaround that excludes `src/doc`
from workspace member probing.

To remove this hack, this PR adds a new optional argument `--path`
for `linkchecker.sh`. With this new argument, `linkchecker.sh` can
be run from a directory separate from the book. This also benefits
other projects using linkchecker, as they can run it under target
directory or any other directory, reducing leftover.
@ehuss
Copy link
Contributor

ehuss commented Apr 20, 2023

Thanks!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Apr 20, 2023

📌 Commit 60f1790 has been approved by ehuss

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 20, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 20, 2023
…iaskrgr

Rollup of 4 pull requests

Successful merges:

 - rust-lang#108795 (Add support for the x86_64h-apple-darwin target)
 - rust-lang#110558 (Add Call terminator to SMIR)
 - rust-lang#110565 (linkchecker: running from a directory separate from the book)
 - rust-lang#110599 (Remove an unused `&[Ty]` <-> `&[GenericArg]`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 9f39295 into rust-lang:master Apr 20, 2023
@rustbot rustbot added this to the 1.71.0 milestone Apr 20, 2023
@weihanglo weihanglo deleted the linkchecker branch April 20, 2023 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants