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

Open crate files readonly first #3259

Merged
merged 1 commit into from Nov 8, 2016

Conversation

Projects
None yet
4 participants
@alexcrichton
Copy link
Member

alexcrichton commented Nov 6, 2016

This allows Cargo to work with read-only CARGO_HOME directories where the
cache was prepared ahead of time.

Closes #3256

Open crate files readonly first
This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256
@rust-highfive

This comment has been minimized.

Copy link

rust-highfive commented Nov 6, 2016

r? @brson

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

@brson

This comment has been minimized.

Copy link
Contributor

brson commented Nov 8, 2016

@bors r+ Thanks!

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 8, 2016

📌 Commit 27e439c has been approved by brson

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 8, 2016

⌛️ Testing commit 27e439c with merge 4cf71cb...

bors added a commit that referenced this pull request Nov 8, 2016

Auto merge of #3259 - alexcrichton:more-ro, r=brson
Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 8, 2016

💥 Test timed out

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Nov 8, 2016

@bors: retry force clean

On Tue, Nov 8, 2016 at 7:01 AM, bors notifications@github.com wrote:

💥 Test timed out


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#3259 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95GpiePubmPpfm-Csww33uTmHVkZeks5q8HKfgaJpZM4KqnL5
.

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 8, 2016

⌛️ Testing commit 27e439c with merge 0c155a4...

bors added a commit that referenced this pull request Nov 8, 2016

Auto merge of #3259 - alexcrichton:more-ro, r=brson
Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 8, 2016

@bors bors merged commit 27e439c into rust-lang:master Nov 8, 2016

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details

@alexcrichton alexcrichton deleted the alexcrichton:more-ro branch Nov 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment