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

std: Consider directory junctions as directories #26929

Merged
merged 1 commit into from
Jul 11, 2015

Conversation

alexcrichton
Copy link
Member

Previously on Windows a directory junction would return false from is_dir,
causing various odd behavior, specifically calls to create_dir_all might fail
when they would otherwise continue to succeed.

Closes #26716

@alexcrichton
Copy link
Member Author

cc @retep998

@rust-highfive
Copy link
Collaborator

r? @brson

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

@brson
Copy link
Contributor

brson commented Jul 10, 2015

@bors r+

dammit windows

@bors
Copy link
Contributor

bors commented Jul 10, 2015

📌 Commit b612d3b has been approved by brson

@bors
Copy link
Contributor

bors commented Jul 10, 2015

☔ The latest upstream changes (presumably #26751) made this pull request unmergeable. Please resolve the merge conflicts.

@retep998
Copy link
Member

Sorry for the merge conflicts 😛

Previously on Windows a directory junction would return false from `is_dir`,
causing various odd behavior, specifically calls to `create_dir_all` might fail
when they would otherwise continue to succeed.

Closes rust-lang#26716
@alexcrichton
Copy link
Member Author

@bors: r=brson 72750c7

@bors
Copy link
Contributor

bors commented Jul 10, 2015

⌛ Testing commit 72750c7 with merge 30aeaba...

@bors
Copy link
Contributor

bors commented Jul 11, 2015

💔 Test failed - auto-mac-64-opt

@alexcrichton
Copy link
Member Author

@bors: retry

On Fri, Jul 10, 2015 at 5:01 PM, bors notifications@github.com wrote:

[image: 💔] Test failed - auto-mac-64-opt
http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/5657


Reply to this email directly or view it on GitHub
#26929 (comment).

@bors
Copy link
Contributor

bors commented Jul 11, 2015

⌛ Testing commit 72750c7 with merge 64db3aa...

bors added a commit that referenced this pull request Jul 11, 2015
Previously on Windows a directory junction would return false from `is_dir`,
causing various odd behavior, specifically calls to `create_dir_all` might fail
when they would otherwise continue to succeed.

Closes #26716
@bors bors merged commit 72750c7 into rust-lang:master Jul 11, 2015
@alexcrichton alexcrichton deleted the windows-dir-junction branch July 17, 2015 18:46
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