Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rustc 1.84 beta3 can't build from tarball: bootstrap try to find git commit on non git directory #133860

Closed
12101111 opened this issue Dec 4, 2024 · 1 comment
Labels
C-bug Category: This is a bug. regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@12101111
Copy link
Contributor

12101111 commented Dec 4, 2024

Code

I tried to compile rustc 1.84 beta 3 from tarball (https://static.rust-lang.org/dist/2024-12-03/)

I expected to see this happen: It should compile

Instead, this happened: bootstrap try to use git and failed

Version it worked on

It most recently worked on: 1.83.0

Version with regression

rustc --version --verbose:

binary: rustc
commit-hash: ea5e5ca0553095786f968671ab1c222e5b1df5c3
commit-date: 2024-12-02
host: x86_64-unknown-linux-musl
release: 1.84.0-beta.3
LLVM version: 19.1.4

Backtrace

Backtrace

running: /tmp/portage/dev-lang/rust-1.84_beta20241203/work/rustc-beta-src/build/bootstrap/debug/bootstrap build -vvv --config=/tmp/portage/dev-lang/rust-1.84_beta20241203/work/rustc-beta-src/config.toml -j30
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
thread 'main' panicked at src/core/config/config.rs:2908:89:
called `Result::unwrap()` on an `Err` value: "command did not execute successfully: cd \"/tmp/portage/dev-lang/rust-1.84_beta20241203/work/rustc-beta-src\" && \"git\" \"rev-list\" \"[email protected]\" \"-n1\" \"--first-parent\" \"HEAD\"\nexpected success, got: exit status: 128\n"
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::result::unwrap_failed
   3: core::result::Result<T,E>::unwrap
             at /usr/lib/rust/1.83.0/lib/rustlib/src/rust/library/core/src/result.rs:1104:23
   4: bootstrap::core::config::config::Config::last_modified_commit
             at ./src/bootstrap/src/core/config/config.rs:2908:22
   5: bootstrap::core::config::config::Config::download_ci_rustc_commit
             at ./src/bootstrap/src/core/config/config.rs:2820:28
   6: bootstrap::core::config::config::Config::parse_inner
             at ./src/bootstrap/src/core/config/config.rs:1775:17
   7: bootstrap::core::config::config::Config::parse
             at ./src/bootstrap/src/core/config/config.rs:1305:9
   8: bootstrap::main
             at ./src/bootstrap/src/bin/main.rs:27:18
   9: core::ops::function::FnOnce::call_once
             at /usr/lib/rust/1.83.0/lib/rustlib/src/rust/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

@rustbot modify labels: +regression-from-stable-to-beta -regression-untriaged

The regression commit is 1837f6f

The fix commit is aafd56b

@12101111 12101111 added C-bug Category: This is a bug. regression-untriaged Untriaged performance or correctness regression. labels Dec 4, 2024
@rustbot rustbot added I-prioritize Issue: Indicates that prioritization has been requested for this issue. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. regression-from-stable-to-beta Performance or correctness regression from stable to beta. and removed regression-untriaged Untriaged performance or correctness regression. labels Dec 4, 2024
@jieyouxu jieyouxu added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) and removed I-prioritize Issue: Indicates that prioritization has been requested for this issue. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Dec 5, 2024
@jieyouxu
Copy link
Member

jieyouxu commented Dec 5, 2024

I believe this is because the download-rustc logic changed back to false which skipps running git logic as opposed to when it was defaulted to if-unchanged previously. AFAICT this can be closed after #133068 was merged.

@jieyouxu jieyouxu closed this as completed Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

No branches or pull requests

4 participants
@12101111 @jieyouxu @rustbot and others