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

Downloading onto a different drive throws an EXDEV error #4

Merged
merged 4 commits into from
Jun 17, 2014
Merged

Conversation

flovan
Copy link

@flovan flovan commented Jun 17, 2014

As posted on the Headstart issues, downloading files into a directory on an external drive will throw an EXDEV error. This PR puts the tmp folder directly inside of the working directory (cwd) and cleans up afterwards through rimraf.

Florian added 2 commits June 17, 2014 20:27
The tmpdir is now placed directly in the cwd. After unzipping the tmp folder gets rimraf'ed (rm -rf).
@jprichardson
Copy link
Owner

Cool. Remove rimraf as a dependency. Use fs-extra remove() instead. Thanks.

@flovan
Copy link
Author

flovan commented Jun 17, 2014

Done! Didn't test though, but the change seemed pretty straight forward. If possible please patch or minor update this repo so I can update Headstart's npm deps accordingly in a later update.

jprichardson added a commit that referenced this pull request Jun 17, 2014
Downloading onto a different drive throws an EXDEV error
@jprichardson jprichardson merged commit 948986d into jprichardson:master Jun 17, 2014
@jprichardson
Copy link
Owner

Published. Thanks again.

@flovan
Copy link
Author

flovan commented Jun 17, 2014

No problem. Thank you also for the great module! 👏

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.

2 participants