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.

Sign up for GitHub

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

Jump to bottom

Remap paths from other crates #117836

Closed
wants to merge 3 commits into from
Closed

Remap paths from other crates #117836

wants to merge 3 commits into from

Commits on Nov 12, 2023

  1. Remap paths from other crates 

    Previously, `remap-path-prefix` only applied to files loaded *in the current session*. File names
loaded in a previous invocation were used unchanged. This is unfortunate for UI testing when some of
the crates involved are in the standard library (we don't want to recompile the standard library for
arbitrary tests).

Change it to apply to existing files as well. I believe this will eventually allow removing the
special-casing for `-Z simulate-remapped-rust-src-base`.
    @jyn514
    jyn514 committed Nov 12, 2023
    Configuration menu
    Copy the full SHA
    29ae877 View commit details
    Browse the repository at this point in the history

  2. Don't return whether the filename was remapped 

    It was never used, and it's ambiguous what the answer should be for paths that were remapped in a previous session.
    @jyn514
    jyn514 committed Nov 12, 2023
    Configuration menu
    Copy the full SHA
    844b157 View commit details
    Browse the repository at this point in the history

  3. add a test for remap-path-scope

    @jyn514
    jyn514 committed Nov 12, 2023
    Configuration menu
    Copy the full SHA
    448f92c View commit details
    Browse the repository at this point in the history