-
Notifications
You must be signed in to change notification settings - Fork 100
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
Cannot install on Windows. #321
Comments
Please provide the entire, verbose log (add ``` like this, output goes here ``` Also: what's your Windows version and Visual Studio version? |
I also suspect that the error is accurate
Please can you post the file |
having the same issue (LLVM 9)
|
Facing same issue. |
nodejs/node-gyp@53d8cb9 (nodejs/node-gyp#1164) should be the cause. |
nodejs/node-gyp#1164 interprets additional node-gyp build parameters as target names, which breaks the current way CL configs are passed to msbuild in install.js. Move the detection logic into configuration phase and pass CL configs as gyp variables. Fixes nodejs#321, fixes nodejs#377
if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" build /p:CLToolExe="clang-cl.exe" /p:CLToolPath="C:\Program Files\LLVM\bin" ) else (node "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" build /p:CLToolExe="clang-cl.exe" /p:CLToolPath="C:\Program Files\LLVM\bin" )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
C:\Users\danny\AppData\Roaming\npm\node_modules\llnode\build\binding.sln : error MSB4014: The build stopped unexpectedly because of an internal failure. C:\Users\danny\AppData\Roaming\npm\node_modules\llnode\build\binding.sln : error MSB4014: System.ArgumentException: The name "plugin:CLToolExe=clang-cl. exe" contains an invalid character ".".
The text was updated successfully, but these errors were encountered: