Skip to content

Conversation

matklad
Copy link
Member

@matklad matklad commented Apr 21, 2018

Some noteble changes:

potentially breaking bug-fix: rust-lang/cargo#5390

Some noteble changes:

  * regression fix: rust-lang/cargo#5390
  * potentially breaking bug-fix: rust-lang/cargo#5389
  * Cargo now caches the result of `rustc -vV`. It checks `rustc` binary
    mtime and rustup toolchain settings, so it should probably "just work"
    with rustbuild.
@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Contributor

warning Warning warning

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 21, 2018
@kennytm
Copy link
Member

kennytm commented Apr 21, 2018

Cargo now caches the result of rustc -vV.

I don't see anywhere in the diff related to this. If you mean rust-lang/cargo#5389 it is already included by #50056.

@matklad
Copy link
Member Author

matklad commented Apr 21, 2018

@kennytm oh, indeed! Looks like 2 of 3 points are actually already here, updated the description!

@kennytm
Copy link
Member

kennytm commented Apr 21, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 21, 2018

📌 Commit cb46d28 has been approved by kennytm

@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, 2018
@bors
Copy link
Collaborator

bors commented Apr 22, 2018

⌛ Testing commit cb46d28 with merge 8887396...

bors added a commit that referenced this pull request Apr 22, 2018
Update Cargo

Some noteble changes:

 * ~~regression fix: rust-lang/cargo#5390
  * ~~potentially breaking bug-fix: rust-lang/cargo#5389
  * ~~Cargo now caches the result of `rustc -vV`. It checks `rustc` binary
    mtime and rustup toolchain settings, so it should probably "just work"
    with rustbuild.~~

potentially breaking bug-fix: rust-lang/cargo#5390
@bors
Copy link
Collaborator

bors commented Apr 22, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 8887396 to master...

@bors bors merged commit cb46d28 into rust-lang:master Apr 22, 2018
@matklad matklad deleted the update-cargo branch July 9, 2019 12:33
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