Skip to content

Test the interaction between --remap-path-prefix and doctests - #164220

Open
Urgau wants to merge 3 commits into
rust-lang:mainfrom
Urgau:remap-path-prefix-doctest-panic
Open

Urgau wants to merge 3 commits into
rust-lang:mainfrom
Urgau:remap-path-prefix-doctest-panic

Conversation

@Urgau

@Urgau Urgau commented Oct 12, 2026

Copy link
Copy Markdown
Member

This PR adds a run-make test to test the interaction between --remap-path-prefix and doctests.

Related to #163895 (comment)

r? @GuillaumeGomez
cc @fmease

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 12, 2026
@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Oct 12, 2026
Comment thread tests/run-make/remap-path-prefix-doctest-panic/rmake.rs Outdated
@Urgau
Urgau force-pushed the remap-path-prefix-doctest-panic branch from 488637f to fbf53be Compare October 13, 2026 16:41
@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks!

@bors r+ rollup

@rust-bors

rust-bors Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

📌 Commit fbf53be has been tentatively approved by GuillaumeGomez

It will be put into the queue for this repository once PR CI succeeds.

@rust-bors rust-bors Bot 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 Oct 13, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Oct 13, 2026
…nic, r=GuillaumeGomez

Test the interaction between `--remap-path-prefix` and doctests

This PR adds a run-make test to test the interaction between `--remap-path-prefix` and doctests.

Related to rust-lang#163895 (comment)
@JonathanBrouwer

Copy link
Copy Markdown
Member

💔 I suspect this PR failed tests as part of a rollup
@bors r-

After fixing the problem, consider running a try job for the failed job before re-approving.

Link to failure: #164276 (comment)

@rust-bors rust-bors Bot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 13, 2026
@rust-bors

rust-bors Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved.

This PR was contained in a rollup (#164276), which was unapproved.

View changes since this unapproval

@Urgau

Urgau commented Oct 13, 2026

Copy link
Copy Markdown
Member Author

@bors try jobs=test-various

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Oct 13, 2026
Test the interaction between `--remap-path-prefix` and doctests


try-job: test-various
@rust-bors

rust-bors Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: fb2c9fa (fb2c9fafcf480428d90b3dd14c4cb77bf501a1f1)
Base parent: 1e16699 (1e1669960262eb54461f7fc6b721e0fae47821f6)

@Urgau

Urgau commented Oct 13, 2026

Copy link
Copy Markdown
Member Author

@bors r=GuillaumeGomez

@rust-bors

rust-bors Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

📌 Commit c2a249d has been approved by GuillaumeGomez

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 13, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Oct 14, 2026
…nic, r=GuillaumeGomez

Test the interaction between `--remap-path-prefix` and doctests

This PR adds a run-make test to test the interaction between `--remap-path-prefix` and doctests.

Related to rust-lang#163895 (comment)
rust-bors Bot pushed a commit that referenced this pull request Oct 14, 2026
Rollup of 9 pull requests

Successful merges:

 - #162749 (Recover missing `turbofish` for `lifetime` arguments in expression position)
 - #164170 (Prefer WriteConsoleW for stdout on Windows)
 - #164255 (test DSL eagerly handle false constraints)
 - #163230 (Document zero-sized allocations properly for `Box` and `Vec`)
 - #163940 (Parse move exprs as a bottom expr instead of a prefix expr)
 - #164219 (Windows FileExt trait doesn't need to handle ErrorKind::Interrupted; test improvements for same)
 - #164220 (Test the interaction between `--remap-path-prefix` and doctests)
 - #164266 (rustdoc: Improvements for `rustc_must_implement_one_of`)
 - #164278 (Output remapped paths in rustdoc JSON with the `documentation` scope)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Oct 14, 2026
…nic, r=GuillaumeGomez

Test the interaction between `--remap-path-prefix` and doctests

This PR adds a run-make test to test the interaction between `--remap-path-prefix` and doctests.

Related to rust-lang#163895 (comment)
rust-bors Bot pushed a commit that referenced this pull request Oct 14, 2026
…uwer

Rollup of 11 pull requests

Successful merges:

 - #162749 (Recover missing `turbofish` for `lifetime` arguments in expression position)
 - #164170 (Prefer WriteConsoleW for stdout on Windows)
 - #164255 (test DSL eagerly handle false constraints)
 - #163230 (Document zero-sized allocations properly for `Box` and `Vec`)
 - #163940 (Parse move exprs as a bottom expr instead of a prefix expr)
 - #164219 (Windows FileExt trait doesn't need to handle ErrorKind::Interrupted; test improvements for same)
 - #164220 (Test the interaction between `--remap-path-prefix` and doctests)
 - #164266 (rustdoc: Improvements for `rustc_must_implement_one_of`)
 - #164272 (Add regression test for #155977)
 - #164287 (Add regression test for hang in overlap check with GAT alias)
 - #164301 (fix issue number for feature(unsized_const_params))
@JonathanBrouwer

Copy link
Copy Markdown
Member

💔 I suspect this PR failed tests as part of a rollup
@bors r-

After fixing the problem, consider running a try job for the failed job before re-approving.

Link to failure: #164305 (comment)

@rust-bors rust-bors Bot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 14, 2026
@rust-bors

rust-bors Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved.

This PR was contained in a rollup (#164305), which was unapproved.

View changes since this unapproval

@Urgau

Urgau commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

@bors try jobs=aarch64,test-various

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Oct 14, 2026
Test the interaction between `--remap-path-prefix` and doctests


try-job: *aarch64*
try-job: test-various
@rust-bors

rust-bors Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: da7f6e9 (da7f6e953de7d4fd1bd9b14d175b09e9f5373c9e)
Base parent: f6c1107 (f6c1107f8be13e48465e12c0fd26e6a39fe18e4f)

@Urgau

Urgau commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

I added 6ecca07 to fix the \ vs / path separator. Try is passing. Let's try again.

@bors r=GuillaumeGomez

@rust-bors

rust-bors Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 6ecca07 has been approved by GuillaumeGomez

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 14, 2026
Zalathar added a commit to Zalathar/rust that referenced this pull request Oct 15, 2026
…nic, r=GuillaumeGomez

Test the interaction between `--remap-path-prefix` and doctests

This PR adds a run-make test to test the interaction between `--remap-path-prefix` and doctests.

Related to rust-lang#163895 (comment)

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants