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

Current beta cannot bootstrap itself #60265

Closed
Keruspe opened this issue Apr 25, 2019 · 5 comments
Closed

Current beta cannot bootstrap itself #60265

Keruspe opened this issue Apr 25, 2019 · 5 comments
Labels
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) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@Keruspe
Copy link
Contributor

Keruspe commented Apr 25, 2019

When trying to build the current beta (2bc1d40) using itself, the build of bootstrap fails with

error: the item `Seek` is imported redundantly
    --> src/bootstrap/lib.rs:1128:23
     |
126  | use std::io::{Seek, SeekFrom, Write, Read};
     |               ---- the item `Seek` is already imported here
...
1128 |         use std::io::{Seek, SeekFrom};
     |                       ^^^^
     |
note: lint level defined here
    --> src/bootstrap/lib.rs:107:9
     |
107  | #![deny(warnings)]
     |         ^^^^^^^^
     = note: #[deny(unused_imports)] implied by #[deny(warnings)]

error: the item `SeekFrom` is imported redundantly
    --> src/bootstrap/lib.rs:1128:29
     |
126  | use std::io::{Seek, SeekFrom, Write, Read};
     |                     -------- the item `SeekFrom` is already imported here
...
1128 |         use std::io::{Seek, SeekFrom};
     |                             ^^^^^^^^

error: aborting due to 2 previous errors

error: Could not compile `bootstrap`.
@hellow554
Copy link
Contributor

hellow554 commented Apr 25, 2019

@rustbot modify labels: A-rustbuild

also label:regression-from-stable-to-beta

@rustbot
Copy link
Collaborator

rustbot commented Apr 25, 2019

Error: Label regression-from-stable-to-beta can only be set by Rust team members

Please let @rust-lang/release know if you're having trouble with this bot.

@rustbot rustbot added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Apr 25, 2019
@Mark-Simulacrum Mark-Simulacrum added the regression-from-stable-to-beta Performance or correctness regression from stable to beta. label Apr 25, 2019
@pietroalbini
Copy link
Member

@rustbot modify labels: T-infra

I'm not sure we support bootstrapping a beta with itself. @Mark-Simulacrum?

@rustbot rustbot added the T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. label Apr 25, 2019
@mati865
Copy link
Contributor

mati865 commented Apr 25, 2019

Once beta is promoted to stable it will be bootstrapping stable with stable which Linux distributions do a lot so it'd be nice.

@Mark-Simulacrum
Copy link
Member

We try to support distro's efforts to support self-bootstrapping; the PR (#60273) already posted looks fine to me. I'll leave a comment to that effect there as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

6 participants