Skip to content
This repository has been archived by the owner on Nov 16, 2019. It is now read-only.

Ignore missing directory error. #47

Closed
wants to merge 1 commit into from
Closed

Ignore missing directory error. #47

wants to merge 1 commit into from

Conversation

tracker1
Copy link

@tracker1 tracker1 commented Sep 7, 2014

Ignore the ENOENT error for a path that doesn't exist... this will correct the issues regarding...

fixes nodejs/node-v0.x-archive#8117

When the error is ignored here, the directory will be properly created/handled elsewhere in code.

Ignore the `ENOENT` error for a path that doesn't exist... this will correct the issues regarding...

nodejs/node-v0.x-archive#8117

When the error is ignored here, the directory will be properly created/handled elsewhere in code.
@samccone
Copy link

+1000000 for this fix, A new node/npm install is totally broken in windows atm

@isaacs
Copy link
Contributor

isaacs commented Sep 17, 2014

Silently ignoring this error is not a good idea.

It'd be better to mkdirp it here, if we can actually tolerate its non-existence and it will be created anyway.

@isaacs isaacs closed this in c8ccdc6 Sep 17, 2014
@isaacs isaacs removed the review label Sep 17, 2014
@tracker1 tracker1 deleted the patch-1 branch September 17, 2014 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to use NPM with fresh Install on Windows
4 participants