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

Possible Resolution to Issue 469 #470

Closed
wants to merge 3 commits into from
Closed

Conversation

corvinrok
Copy link

resolve npm module location without hard coding APPDATA on Windows installations. which is a problem identified in issue #469

@tjanczuk
Copy link
Owner

Does this fix work for you? This is what I get on my machine:

C:\projects\adhoc>node -e "require.resolve('npm/node_modules/node-gyp/bin/node-gyp')"
module.js:341
    throw err;
    ^

Error: Cannot find module 'npm/node_modules/node-gyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.require.resolve (internal/module.js:23:19)
    at [eval]:1:9
    at Object.exports.runInThisContext (vm.js:54:17)
    at Object.<anonymous> ([eval]-wrapper:6:22)
    at Module._compile (module.js:413:34)
    at node.js:550:27
    at _combinedTickCallback (node.js:376:9)
    at process._tickCallback (node.js:407:11)

C:\projects\adhoc>where node-gyp
C:\Users\Tomasz Janczuk\AppData\Roaming\npm\node-gyp
C:\Users\Tomasz Janczuk\AppData\Roaming\npm\node-gyp.cmd

@corvinrok
Copy link
Author

that fix did work for me, yes. I didn't get errors when running with the fix, but did without it. Here is my result on the where--

C:\git\translation-utility>where node-gyp
C:\Program Files\nodejs\node-gyp
C:\Program Files\nodejs\node-gyp.cmd

@agracio
Copy link
Collaborator

agracio commented Apr 24, 2024

@agracio agracio closed this Apr 24, 2024
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