Skip to content

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

Open
emyl wants to merge 1 commit into
applicationsonline:masterfrom
emyl:move
Open

Replace possibly buggy FileUtils.mv with a two step solution#30
emyl wants to merge 1 commit into
applicationsonline:masterfrom
emyl:move

Conversation

@emyl
Copy link
Copy Markdown

@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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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