Don't check for cargo-vendor when building from (Debian's) git
authorMatthijs van Otterdijk <matthijs@wirevirt.net>
Thu, 14 Jul 2022 11:17:38 +0000 (13:17 +0200)
committerFabian Grünbichler <debian@fabian.gruenbichler.email>
Sun, 10 Sep 2023 17:22:53 +0000 (18:22 +0100)
Forwarded: not-needed

Forwarded: not-needed

Gbp-Pq: Name d-bootstrap-disable-git.patch

src/bootstrap/channel.rs
src/bootstrap/dist.rs

index eae81b9fc69c83c91919804bf18858b1ca66baed..6deab9b5b0b2fdfa819f9d9d6b1d845c2a81baad 100644 (file)
@@ -36,6 +36,12 @@ pub struct Info {
 
 impl GitInfo {
     pub fn new(ignore_git: bool, dir: &Path) -> GitInfo {
+        //
+        // Debian: returning early because the Debian package is also in a git
+        //         repository, but we don't want to parse gitinfo. This is
+        //         needed for the bootstrap tests to work which running for
+        //         Debian git.
+        return GitInfo::Absent;
         // See if this even begins to look like a git dir
         if !dir.join(".git").exists() {
             match read_commit_info_file(dir) {
index 6594b23c56b03421a0d8d125464b7b40f79a6e8d..12dee943b12f468f5f223c228bb7617316985ca9 100644 (file)
@@ -969,7 +969,10 @@ impl Step for PlainSourceTarball {
         }
 
         // If we're building from git sources, we need to vendor a complete distribution.
-        if builder.rust_info().is_managed_git_subrepository() {
+        //
+        // Debian: short-circuited because the Debian package is also in a git
+        //         repository, but cargo-vendor should not be installed or run.
+        if false && builder.rust_info().is_managed_git_subrepository() {
             // Ensure we have the submodules checked out.
             builder.update_submodule(Path::new("src/tools/rust-analyzer"));