Skip to content

Pass --edition after --crate-name#7710

Merged
bors merged 1 commit intorust-lang:masterfrom
dtolnay:argorder
Dec 16, 2019
Merged

Pass --edition after --crate-name#7710
bors merged 1 commit intorust-lang:masterfrom
dtolnay:argorder

Conversation

@dtolnay
Copy link
Member

@dtolnay dtolnay commented Dec 14, 2019

This PR swaps the order of the --edition and --crate-name args in the rustc invocation.

  • Before: rustc --edition=2018 --crate-name cargo ...
  • After: rustc --crate-name cargo --edition=2018 ...

The crate name is a lot more relevant when looking at processes in top for example, and should appear first.

This PR swaps the order of the --edition and --crate-name args in the
rustc invocation.

    Before: rustc --edition=2018 --crate-name cargo ...
    After: rustc --crate-name cargo --edition=2018 ...

The crate name is a lot more relevant when looking at processes in top
for example, and should appear first.
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Dec 16, 2019

📌 Commit 9aa65c5 has been approved by alexcrichton

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 16, 2019
@bors
Copy link
Contributor

bors commented Dec 16, 2019

⌛ Testing commit 9aa65c5 with merge d138d37...

bors added a commit that referenced this pull request Dec 16, 2019
Pass --edition after --crate-name

This PR swaps the order of the --edition and --crate-name args in the rustc invocation.

- Before: `rustc --edition=2018 --crate-name cargo ...`
- After: `rustc --crate-name cargo --edition=2018 ...`

The crate name is a lot more relevant when looking at processes in `top` for example, and should appear first.
@bors
Copy link
Contributor

bors commented Dec 16, 2019

☀️ Test successful - checks-azure
Approved by: alexcrichton
Pushing d138d37 to master...

@bors bors merged commit 9aa65c5 into rust-lang:master Dec 16, 2019
@ehuss ehuss added this to the 1.42.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.

4 participants