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

Utils.js: function copyFile always prints "chmod <perms> on <file> failed after copying, ignoring". #147

Closed
bambooCZ opened this issue Jan 5, 2015 · 6 comments · Fixed by #148

Comments

@bambooCZ
Copy link
Contributor

bambooCZ commented Jan 5, 2015

solution is to check if there actually was an error, ex.

if (err) {
    _event.emit('log', 'chmod ' + stats.mode + ' on ' + dest + ' failed after copying, ignoring');
}
@bambooCZ bambooCZ changed the title Utils.js: function copyFile always prints "chmod <perms> failed on <file>, after copy". Utils.js: function copyFile always prints "chmod <perms> on <file> failed after copying, ignoring". Jan 5, 2015
@bambooCZ
Copy link
Contributor Author

bambooCZ commented Jan 5, 2015

also mentioned in #141

@bambooCZ
Copy link
Contributor Author

bambooCZ commented Jan 7, 2015

Actualy, whole copyFile method is pretty fucked up...

@mdb
Copy link

mdb commented Jan 28, 2015

Is there a planned release now that this issue is closed? My team's CI environment relies upon an npm release.

Apologies for nagging ;)

Thanks for the fix!

@adam-lynch
Copy link
Contributor

Yeah, sometime today (Irish time).

On Wed, 28 Jan 2015 15:11 Mike Ball [email protected] wrote:

Is there a planned release now that this issue is closed? My team's CI
environment relies upon an npm release.

Apologies for nagging ;)

Thanks for the fix!


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

@mdb
Copy link

mdb commented Jan 28, 2015

@adam-lynch Great - thanks so much.

@adam-lynch
Copy link
Contributor

1.0.8 published.

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 a pull request may close this issue.

3 participants