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

Backport "Don't give a hard error when the end-user specifies RUSTC_BOOTSTRAP=crate_name" #9385

Merged
merged 5 commits into from
Apr 21, 2021

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented Apr 20, 2021

Backports #9365, fixing #9362.

This doesn't work because it uses the name of the build script, which is
always build_script_build. I'm not sure what to change it to - the name
of the library crate could be different than the name of the package,
and there could be multiple different crates being compiled in the same
package.
… from the top-level.

If there's no library, give a hard error unless features are
unconditionally allowed with RUSTC_BOOTSTRAP=1.
@rust-highfive
Copy link

r? @alexcrichton

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.52.0. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 20, 2021
@jyn514
Copy link
Member Author

jyn514 commented Apr 20, 2021

r? @ehuss

@rust-highfive rust-highfive assigned ehuss and unassigned alexcrichton Apr 20, 2021
@ehuss
Copy link
Contributor

ehuss commented Apr 21, 2021

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Apr 21, 2021

📌 Commit 2dbeda8 has been approved by ehuss

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 21, 2021
@bors
Copy link
Contributor

bors commented Apr 21, 2021

⌛ Testing commit 2dbeda8 with merge 6976741...

@bors
Copy link
Contributor

bors commented Apr 21, 2021

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing 6976741 to rust-1.52.0...

@bors bors merged commit 6976741 into rust-lang:rust-1.52.0 Apr 21, 2021
@jyn514 jyn514 deleted the beta-bootstrap-backport branch April 21, 2021 01:17
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 21, 2021
[beta] Beta rollups

- Upgrade expat dependency in riscv64 to newer version. rust-lang#84394
- Fix Self keyword doc URL conflict on case insensitive file systems (until definitely fixed on rustdoc) rust-lang#83678
- Cargo:
    - Backport "Don't give a hard error when the end-user specifies RUSTC_BOOTSTRAP=crate_name" (rust-lang/cargo#9385)
    - [beta] Revert rust-lang/cargo#9133, moving to git HEAD dependencies by default (rust-lang/cargo#9383)
@ehuss ehuss added this to the 1.52.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants