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

fse.copySync crashes while copying across devices D: #106

Closed
ghost opened this issue Jan 25, 2015 · 8 comments
Closed

fse.copySync crashes while copying across devices D: #106

ghost opened this issue Jan 25, 2015 · 8 comments

Comments

@ghost
Copy link

ghost commented Jan 25, 2015

If I try to use fs.copySync for copying across different disk drives, my app crashes T_T Copying across one drive works well.

@jprichardson
Copy link
Owner

Platform (Windows, Linux)? Node version? Snippet code to reproduce?

@ghost
Copy link
Author

ghost commented Jan 25, 2015

Windows 8.1, used in nw.js 0.11.5 https://github.com/nwjs/nw.js (actually it shouldn't affect node module), node.js 0.11.13-pre.

fs.copySync('C:\Users\Cosmo\Desktop\waves.jpg','D:\ct\blahblah\img\i10.jpg');
// crash! It doesn't even throw exceptions.

@jprichardson
Copy link
Owner

Ok, thanks. Does it work by chance if D:\ct\blahblah\img\ exists?

@ghost
Copy link
Author

ghost commented Jan 25, 2015

All the needed directories with the source file exist.
I encountered similar problem while using fs.link (?) function long ago, and solved that with streams. But it is an asynchronous technique ._.

@jprichardson
Copy link
Owner

But does it work if the destination directory already exists?

@ghost
Copy link
Author

ghost commented Jan 25, 2015

No :(

@jprichardson jprichardson changed the title fse.copy crashes while copying across devices D: fse.copySync crashes while copying across devices D: Jan 28, 2015
@jprichardson jprichardson modified the milestone: 1.0 Jul 2, 2015
@RyanZim
Copy link
Collaborator

RyanZim commented Oct 26, 2016

@jprichardson Can you reproduce?

@jprichardson jprichardson removed this from the 1.0.0 milestone Oct 27, 2016
@jprichardson
Copy link
Owner

jprichardson commented Oct 27, 2016

Nope, can't produce on Windows 10 and Node v6.5.0. Please reopen @ct-js if still an issue.

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

No branches or pull requests

2 participants