]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #63342 - ehuss:remap-path-dep-info, r=alexcrichton
authorMazdak Farrokhzad <twingoow@gmail.com>
Thu, 8 Aug 2019 14:33:39 +0000 (16:33 +0200)
committerGitHub <noreply@github.com>
Thu, 8 Aug 2019 14:33:39 +0000 (16:33 +0200)
commiteeacbee5804e488e3f5995be63a166df47abd9e8
tree640ced3ef10ca4361e013e0f68d7bf887c367987
parent76938abba13de41dd9da498226d6d549a29a3e47
parent04eee2eca2051d2b10287d87174073b73871d13b
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