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.

Sign up for GitHub

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

Jump to bottom

Added moveDir #6015

Merged
merged 1 commit into from Jun 27, 2017
Merged

Added moveDir #6015

merged 1 commit into from Jun 27, 2017

Conversation

yglukhov
Copy link
Member

@yglukhov yglukhov commented Jun 24, 2017 •
edited

Added moveDir which is mostly a copy-paste of moveFile, but falls back to copyDir/removeDir on EXDEV error.

Removed extra include ospaths

@yglukhov
Copy link
Member Author

Dried up the code a bit.

@yglukhov yglukhov force-pushed the movedir branch 2 times, most recently from d2b1114 to e856226 Compare June 24, 2017 10:32
@yglukhov
Copy link
Member Author

Added include ospaths back because it breaks koch for some reason.

@yglukhov
Copy link
Member Author

Fixed windows version and rebased.

@yglukhov
Copy link
Member Author

Appveyor timed out :(

@yglukhov
Copy link
Member Author

Appveyor failure looks unrelated.

@Araq Araq merged commit 0345238 into nim-lang:devel Jun 27, 2017
@yglukhov yglukhov deleted the movedir branch June 27, 2017 08:32
@Yardanico Yardanico mentioned this pull request Jul 13, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@yglukhov @Araq