Skip to content

rustbuild: Fix a distribution bug with rustdoc - #44498

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:fix-nightlies
Sep 12, 2017
Merged

bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:fix-nightlies

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Apparently File::create was called when there was an existing hard link or the
like, causing an existing file to get accidentally truncated!

Closes #44487

Apparently `File::create` was called when there was an existing hard link or the
like, causing an existing file to get accidentally truncated!

Closes rust-lang#44487
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

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

@alexcrichton

Copy link
Copy Markdown
Member Author

r? @Mark-Simulacrum

@alexcrichton

Copy link
Copy Markdown
Member Author

I'm gonna go ahead an r+ and see if we can get this in to fix nightlies...

@bors: r+ p=100

@bors

bors commented Sep 11, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit eb26d5b has been approved by alexcrichton

@bors

bors commented Sep 11, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit eb26d5b with merge 07d950f...

bors added a commit that referenced this pull request Sep 11, 2017
rustbuild: Fix a distribution bug with rustdoc

Apparently `File::create` was called when there was an existing hard link or the
like, causing an existing file to get accidentally truncated!

Closes #44487
@Mark-Simulacrum

Copy link
Copy Markdown
Member

Looks good to me, though a comment would be nice.

@bors

bors commented Sep 12, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 07d950f to master...

@bors
bors merged commit eb26d5b into rust-lang:master Sep 12, 2017
@alexcrichton
alexcrichton deleted the fix-nightlies branch September 12, 2017 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants