Skip to content

Improve --remap-path-prefix documentation #142372

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Urgau
Copy link
Member

@Urgau Urgau commented Jun 11, 2025

This PR improves the --remap-path-prefix documentation by:

  • moving it into it's own page
  • adding a small example
  • mentioning that the remapping is best effort1
  • mentioning link.exe/PDB issues2
  • mentioning the path separator issue (textual replacement)

r? @wesleywiser

Footnotes

  1. #t-compiler > link.exe leaking paths in Windows PDB @ 💬

  2. #t-compiler > link.exe leaking paths in Windows PDB @ 💬

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 11, 2025
@Urgau Urgau added A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools A-path-remapping Area: path remapping, --remap-path-prefix, --remap-cwd-prefix, --remap-diagnostics-scope etc. labels Jun 11, 2025
@rust-log-analyzer

This comment has been minimized.

@Urgau Urgau force-pushed the improve-remap_path_prefix-doc branch from 993bf1e to e4938cb Compare June 11, 2025 17:56
@Urgau Urgau force-pushed the improve-remap_path_prefix-doc branch from e4938cb to b224947 Compare June 12, 2025 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools A-path-remapping Area: path remapping, --remap-path-prefix, --remap-cwd-prefix, --remap-diagnostics-scope etc. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants