Skip to content

Should miri be built by default with --enable-extended for realease tarballs? #139569

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

Closed
dilyanpalauzov opened this issue Apr 9, 2025 · 3 comments · Fixed by #139574
Closed
Labels
C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@dilyanpalauzov
Copy link

config.toml.sample contains

#tools = [
#    … #    "miri", "cargo-miri" # for dev/nightly channels
#]

This means that when building from a realease tarball with ./configure --enable-optimize-llvm --enable-extended --llvm-root=/usr/local --enable-profiler --enable-llvm-link-shared --enable-sanitizers --enable-local-rust --disable-docs --target=x86_64-unknown-linux-gnu --enable-vendor using https://static.rust-lang.org/dist/rustc-1.86.0-src.tar.xz miri will not be compiled or installed.

This was discussed in #129171 and the hyperlinks versioned there.

I do not understand, should miri and cargo-miri be built by default, from the realease tarball, or be built only by default for dev/nightly channels?

$ miri --version
rustc 1.86.0-nightly (05f9846f8 2025-03-31) (built from a source tarball)

$ cargo miri --version
miri 0.1.0

My understanding from #100134 is that miri should not be built by default when release tarball is used, but it is built by default.

@dilyanpalauzov dilyanpalauzov added C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Apr 9, 2025
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Apr 9, 2025
@onur-ozkan onur-ozkan removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Apr 9, 2025
@onur-ozkan
Copy link
Member

There is a logical conflict where tarball sources use the src/ci/channel value as the default channel if not set manually, but at the same time the "dist" profile (which is the default profile for tarball sources) sets the channel to "nightly".

#139574 attempts to resolve this.

@dilyanpalauzov
Copy link
Author

Just for my understanding, as this very same thing has changed several times.

Should miri be built by default with --enable-extended for realease tarballs or not?

@onur-ozkan
Copy link
Member

Answer is already in bootstrap.example.toml:

"miri", "cargo-miri" # for dev/nightly channels

@bors bors closed this as completed in fb7f0e4 Apr 11, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Apr 11, 2025
Rollup merge of rust-lang#139574 - onur-ozkan:better-channel-handling, r=onur-ozkan

bootstrap: improve `channel` handling

Fixes rust-lang#139569

See [this comment](rust-lang#139574 (comment)) for the explanation of this bug.
github-actions bot pushed a commit to rust-lang/rustc-dev-guide that referenced this issue Apr 14, 2025
…ozkan

bootstrap: improve `channel` handling

Fixes rust-lang/rust#139569

See [this comment](rust-lang/rust#139574 (comment)) for the explanation of this bug.
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. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants