rustc/debian/patches/build/d-bootstrap-disable-git.patch
2024-07-29 10:24:19 +02:00

45 lines
2.0 KiB
Diff

From: Matthijs van Otterdijk <matthijs@wirevirt.net>
Date: Thu, 14 Jul 2022 13:17:38 +0200
Subject: Don't check for cargo-vendor when building from (Debian's) git
Forwarded: not-needed
---
src/bootstrap/src/core/build_steps/dist.rs | 6 ++++--
src/bootstrap/src/utils/channel.rs | 6 ++++++
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/bootstrap/src/core/build_steps/dist.rs b/src/bootstrap/src/core/build_steps/dist.rs
index 91039d0..f2793e4 100644
--- a/src/bootstrap/src/core/build_steps/dist.rs
+++ b/src/bootstrap/src/core/build_steps/dist.rs
@@ -1007,8 +1007,10 @@ impl Step for PlainSourceTarball {
// If we're building from git or tarball sources, we need to vendor
// a complete distribution.
- if builder.rust_info().is_managed_git_subrepository()
- || builder.rust_info().is_from_tarball()
+ //
+ // Debian: short-circuited because the Debian package is also in a git
+ // repository, but cargo-vendor should not be installed or run.
+ if false
{
// FIXME: This code looks _very_ similar to what we have in `src/core/build_steps/vendor.rs`
// perhaps it should be removed in favor of making `dist` perform the `vendor` step?
diff --git a/src/bootstrap/src/utils/channel.rs b/src/bootstrap/src/utils/channel.rs
index 88988c3..e5c6cd1 100644
--- a/src/bootstrap/src/utils/channel.rs
+++ b/src/bootstrap/src/utils/channel.rs
@@ -35,6 +35,12 @@ pub struct Info {
impl GitInfo {
pub fn new(omit_git_hash: 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) {