-
Notifications
You must be signed in to change notification settings - Fork 40
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
Mock Devices Build failed with error code: 1 #17
Comments
@adamlash @microsoftopensource @stevebus @459680 @marvin-garcia |
Hi! Im also running into error code 1 when trying to open mocksensors. i think the origin of the bug is the code not being up to date with nodejs. (im running nodejs v18.12.0.) I'm adding my error text in case its of any use. best regards |
Turns out, for me there were multiple issues, firstly I was missing Python, then I had to add Python to "PATH" - I think it is an environment variable (though as a non-programmer myself, I don't really know what that means). Then I had to update and add a lot of C++ libraries from the Visual Studio Installer. Finally had to install NodeJS 16.13.0 (I had also originally installed 18.12.0) and then the mock devices started working. I think the devs should update the documentation for noobs like me who don't know what dependencies and other packages are to be installed in addition to what is already mentioned! The ADT Link is finally working for me now though, so I guess with a bit of effort, this repo does work decently enough :) |
@vararth @adamlash regards |
So I installed "Visual Studio Build Tools 2017" and in the installer, I chose the following options -
With regards the python issue, I installed both python 2 and python 3, but vertically inside the path list, I moved python2 above python3 - seems to have solved the issue for me, check the screenshots below :) |
Python 2 is mandatory |
Hi Varath, I have installed NodeJS 16.13.0 and the Visual Studio Build Tools 2017. I still get Error Code 1 when I run the npm ci. |
I got it fixed. I had to edit the path variable as the final fix. Python 2 is required. |
This issue is for a: (mark with an
x
)Minimal steps to reproduce
Any log messages given by the failure
Log files attached
2022-11-24T11_00_43_412Z-debug-0.log
2022-11-24T11_00_43_740Z-debug-0.log
Expected/desired behavior
OS and Version?
Versions
The text was updated successfully, but these errors were encountered: