diff options
| author | Mazdak Farrokhzad <twingoow@gmail.com> | 2019-08-08 16:33:39 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-08-08 16:33:39 +0200 |
| commit | eeacbee5804e488e3f5995be63a166df47abd9e8 (patch) | |
| tree | 640ced3ef10ca4361e013e0f68d7bf887c367987 /src | |
| parent | 76938abba13de41dd9da498226d6d549a29a3e47 (diff) | |
| parent | 04eee2eca2051d2b10287d87174073b73871d13b (diff) | |
| download | rust-eeacbee5804e488e3f5995be63a166df47abd9e8.tar.gz rust-eeacbee5804e488e3f5995be63a166df47abd9e8.zip | |
Rollup merge of #63342 - ehuss:remap-path-dep-info, r=alexcrichton
Don't use remap-path-prefix in dep-info files. This changes it so that dep-info files do not use remapped paths. Having remapped paths causes problems with Cargo because if you remap to a nonexistent path (like `/rustc/$HASH` which rustc distributions do), then Cargo's change tracking thinks the files don't exist and will always rebuild. I don't actually know if this is a good idea. I think it makes sense, but I do not know what the exact requirements are for reproducible builds. I consider these files separate from the binary artifacts generated, and as a build-system helper, so it seems reasonable to me. I'm also not sure if this needs a test. I'll definitely add one on the cargo side if this is merged. Closes #63329
Diffstat (limited to 'src')
| -rw-r--r-- | src/librustc_interface/passes.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/librustc_interface/passes.rs b/src/librustc_interface/passes.rs index b91c5e40384..c9d68943b08 100644 --- a/src/librustc_interface/passes.rs +++ b/src/librustc_interface/passes.rs @@ -661,7 +661,7 @@ fn write_out_deps(compiler: &Compiler, outputs: &OutputFilenames, out_filenames: .iter() .filter(|fmap| fmap.is_real_file()) .filter(|fmap| !fmap.is_imported()) - .map(|fmap| escape_dep_filename(&fmap.name)) + .map(|fmap| escape_dep_filename(&fmap.unmapped_path.as_ref().unwrap_or(&fmap.name))) .collect(); if sess.binary_dep_depinfo() { |
