nvm-windows icon indicating copy to clipboard operation
nvm-windows copied to clipboard

getting Access Denied at os.Rename()

Open metanomi opened this issue 6 years ago • 1 comments

os.Rename() of node_modules/npm on Node version installs throws Access Denied (win 7) -- seems to be this issue https://github.com/coreybutler/nvm-windows/issues/302 and that seems to be caused by a golang behavior change in os.Rename() described in https://github.com/golang/go/issues/14527 ...fixed by using copy package instead

metanomi avatar Jul 18 '18 21:07 metanomi

I have two issues with this:

  1. It leaves an orphan file in the temp directory.
  2. The added dependency on a new external lib (trying to minimize them).

Copying is fine, but it would be better to use the standard features and cleanup after the copy is complete.

coreybutler avatar Aug 02 '18 03:08 coreybutler