-
Notifications
You must be signed in to change notification settings - Fork 284
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
Building v8/node/vee-eight-lkgr fails in Windows 7 x64 #471
Comments
Same issue as nodejs/node#10992 (comment). Whoever finds the fix will help us to land V8 5.6 on master ;) |
@targos The first unfound file seems to be required on this line There is no such file in the |
I'll take a look. My hunch is conflicting uses of PRODUCT_DIR vs. SHARED_INTERMEDIATE_DIR. |
@vsemozhetbyt Do the patches from nodejs/node#10992 (comment) work for you? |
@bnoordhuis Yes, deleting two lines mentioned in the first diff seems to fix the building. Thank you. |
@bnoordhuis However, it seems this line in the BUILDING.md:
should be replaced by
(Currently testing after building fails due to signing attempt failure). Or maybe the team could land the nodejs/node@92ed1ab and nodejs/node@7cd6a7d |
If |
Node.js Version: https://github.com/v8/node/tree/vee-eight-lkgr
OS: Windows 7 x64
Scope: build
Prerequisites: Python 2.7.13; Visual C++ Build Tools 2015 14.0.25420.1; Git for Windows 2.11.0.3-64-bit (Unix tools are included in the global PATH)
Command:
.\vcbuild nosign
in thecmd.exe
(according to BUILDING.mdnosign
is still used there)The last output fragment:
Text (click me):
The content of the
.\build\Release\obj\global_intermediate
folder after the fail (no folders mentioned in the error output, just these files):The content of the
.\Release\obj\global_intermediate\src\inspector\protocol\
folder after the fail:Could the v8/node/vee-eight-lkgr be built in my environment? Should I correct something? Should I provide some more information?
The text was updated successfully, but these errors were encountered: