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

incr.comp.: Remove ability to produce incr. comp. hashes during metadata export. #46370

Merged
merged 2 commits into from
Dec 1, 2017

Conversation

michaelwoerister
Copy link
Member

@michaelwoerister michaelwoerister commented Nov 29, 2017

This functionality has been superseded by on-import hashing, which can be less conservative and does not require extra infrastructure.

r? @nikomatsakis

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 29, 2017
@eddyb
Copy link
Member

eddyb commented Nov 30, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Nov 30, 2017

📌 Commit 7ebccbb has been approved by eddyb

@michaelwoerister
Copy link
Member Author

@bors p=1

Bumping priority because it's blocking incr. comp. and high bitrot risk.

@bors
Copy link
Contributor

bors commented Nov 30, 2017

⌛ Testing commit 7ebccbb with merge d170379...

bors added a commit that referenced this pull request Nov 30, 2017
incr.comp.: Remove ability to produce incr. comp. hashes during metadata export.

This functionality has been superseded by on-import hashing, which can be less conservative and does not require extra infrastructure.

r? @nikomatsakis
@bors
Copy link
Contributor

bors commented Nov 30, 2017

💔 Test failed - status-appveyor

@arielb1
Copy link
Contributor

arielb1 commented Nov 30, 2017

Appveyor 3h timeout for some reason, looking at the time log seems like general slowness, and there appears to be a large amount of variance between commits.

@bors retry

arielb1 pushed a commit to arielb1/rust that referenced this pull request Nov 30, 2017
…g, r=eddyb

incr.comp.: Remove ability to produce incr. comp. hashes during metadata export.

This functionality has been superseded by on-import hashing, which can be less conservative and does not require extra infrastructure.

r? @nikomatsakis
@bors
Copy link
Contributor

bors commented Dec 1, 2017

⌛ Testing commit 7ebccbb with merge d8a60c9...

bors added a commit that referenced this pull request Dec 1, 2017
incr.comp.: Remove ability to produce incr. comp. hashes during metadata export.

This functionality has been superseded by on-import hashing, which can be less conservative and does not require extra infrastructure.

r? @nikomatsakis
@bors
Copy link
Contributor

bors commented Dec 1, 2017

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

@bors bors merged commit 7ebccbb into rust-lang:master Dec 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants