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

Fix EPERM error on Windows on rename #104

Closed
wants to merge 1 commit into from
Closed

Fix EPERM error on Windows on rename #104

wants to merge 1 commit into from

Conversation

manuelcabral
Copy link

Fixes #83 by writing to output file on callback

@j201
Copy link

j201 commented Nov 5, 2014

So what's the problem causing #83? I've been experiencing the bug and I'm anxious to get around it, so I'd like to take a look or help out with it.

@manuelcabral
Copy link
Author

The fs.rename() seems to fail because the temporary file is still being used. Perhaps the file is still opened by browserify, but I couldn't find it.

Although this fix (writing directly to the output file) seems to work on Windows it may cause some problems, such as triggering a watch before the file has been completely written. Renaming would definitely be best.

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.

Error: EPERM, rename '...something...'
2 participants