-
-
Notifications
You must be signed in to change notification settings - Fork 35.4k
Module.globalPaths under Windows #6434
Copy link
Copy link
Closed
Labels
feature requestIssues that request new features to be added to Node.js.Issues that request new features to be added to Node.js.moduleIssues and PRs related to the module subsystem.Issues and PRs related to the module subsystem.windowsIssues and PRs related to the Windows platform.Issues and PRs related to the Windows platform.
Metadata
Metadata
Assignees
Labels
feature requestIssues that request new features to be added to Node.js.Issues that request new features to be added to Node.js.moduleIssues and PRs related to the module subsystem.Issues and PRs related to the module subsystem.windowsIssues and PRs related to the Windows platform.Issues and PRs related to the Windows platform.
Looks like code at https://github.com/nodejs/node/blob/master/lib/module.js#L590:
is Unix specific and since
nodejsis installed asc:\program files\nodejs\node.exethe base path resolved to'C:\\Program Files\\lib\\node', and it's pointless.Guess this should be fixed to
path.resolve(process.env.AppData, 'npm', 'node_modules')for windows, or something similar since right there node_modules are placed by default in win7+.