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

Replace possibly buggy FileUtils.mv with a two step solution #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emyl
Copy link

@emyl emyl commented Feb 24, 2014

On Windows 7/8 64bit, when using librarian-chef both standalone and trough vagrant, the FileUtils.mv operation fails with Errno::EACCES (permission denied). I suspect it could be a ruby bug, in any case it could be easily worked around.

@Zaylril
Copy link

Zaylril commented Mar 21, 2014

I also get this bug on both standalone and vagrant plugin. Would be nice to see it merged in and fixed.

@emyl
Copy link
Author

emyl commented Mar 22, 2014

@Zaylril in case you're interested, The following forks include the bug fix:

https://github.com/emyl/librarian-chef-nochef
https://github.com/emyl/vagrant-librarian-chef-nochef

@yfeldblum
Copy link
Contributor

I want atomic rename. I don't want partially-copied directories.

It looks like there is a bug with this description in some versions of Ruby: https://bugs.ruby-lang.org/issues/8109. Is this what you're running into?

@emyl
Copy link
Author

emyl commented May 20, 2014

Not really. After investigating a little bit more, I think that the issue could be caused by the virus scanner:

https://www.ruby-forum.com/topic/1044813
https://tickets.puppetlabs.com/browse/PUP-1918

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.

3 participants