Ensure libraries built in stage0 have unique metadata
authorJosh Stone <jistone@redhat.com>
Wed, 16 May 2018 00:48:02 +0000 (17:48 -0700)
committerXimin Luo <infinity0@debian.org>
Thu, 31 May 2018 16:25:17 +0000 (17:25 +0100)
Issue #50786 shows a case with local rebuild where the libraries built
by stage0 had the same suffix as stage0's own, and were accidentally
loaded by that stage0 rustc when compiling `librustc_trans`.

Now we set `__CARGO_DEFAULT_LIB_METADATA` to "bootstrap" during stage0,
rather than the release channel like usual, so the library suffix will
always be completely distinct from the stage0 compiler.

Gbp-Pq: Name u-0001-Ensure-libraries-built-in-stage0-have-unique-metadat.patch

src/bootstrap/builder.rs

index 0464840c3e8187c89d195a47a1902bd446aa2ff1..7618cee32dfb60b28057ddb5a7f8c890183e1b7a 100644 (file)
@@ -542,7 +542,15 @@ impl<'a> Builder<'a> {
 
         // FIXME: Temporary fix for https://github.com/rust-lang/cargo/issues/3005
         // Force cargo to output binaries with disambiguating hashes in the name
-        cargo.env("__CARGO_DEFAULT_LIB_METADATA", &self.config.channel);
+        let metadata = if compiler.stage == 0 {
+            // Treat stage0 like special channel, whether it's a normal prior-
+            // release rustc or a local rebuild with the same version, so we
+            // never mix these libraries by accident.
+            "bootstrap"
+        } else {
+            &self.config.channel
+        };
+        cargo.env("__CARGO_DEFAULT_LIB_METADATA", &metadata);
 
         let stage;
         if compiler.stage == 0 && self.local_rebuild {