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

Windows release job failing on vs2015-x86 #669

Closed
mhdawson opened this issue Mar 30, 2017 · 5 comments
Closed

Windows release job failing on vs2015-x86 #669

mhdawson opened this issue Mar 30, 2017 · 5 comments

Comments

@mhdawson
Copy link
Member

This seems to have failed the most recent couple of nightly builds. Looks like it passed on an earlier one (Mar 23)

https://ci-release.nodejs.org/job/iojs+release/

       [
              _Ty=v8::CpuProfileDeoptInfo
          ]
  c:\ws\deps\v8\include/v8-profiler.h(44): note: see reference to class template instantiation 'std::vector<v8::CpuProfileDeoptInfo,std::allocator<_Ty>>' being compiled
          with
          [
              _Ty=v8::CpuProfileDeoptInfo
          ]
     Creating library c:\ws\Release\mksnapshot.lib and object c:\ws\Release\mksnapshot.exp
v8_base_0.lib(parser.obj) : error LNK2001: unresolved external symbol "public: static struct v8::internal::Runtime::Function const * __cdecl v8::internal::Runtime::FunctionForId(enum v8::internal::Runtime::FunctionId)" (?FunctionForId@Runtime@internal@v8@@SAPBUFunction@123@W4FunctionId@123@@Z) [c:\ws\deps\v8\src\mksnapshot.vcxproj]
v8_base_0.lib(parser.obj) : error LNK2001: unresolved external symbol "public: static struct v8::internal::Runtime::Function const * __cdecl v8::internal::Runtime::FunctionForName(unsigned char const *,int)" (?FunctionForName@Runtime@internal@v8@@SAPBUFunction@123@PBEH@Z) [c:\ws\deps\v8\src\mksnapshot.vcxproj]
v8_base_3.lib(assembler.obj) : error LNK2001: unresolved external symbol "public: static struct v8::internal::Runtime::Function const * __cdecl v8::internal::Runtime::RuntimeFunctionTable(class v8::internal::Isolate *)" (?RuntimeFunctionTable@Runtime@internal@v8@@SAPBUFunction@123@PAVIsolate@23@@Z) [c:\ws\deps\v8\src\mksnapshot.vcxproj]
v8_base_3.lib(assembler.obj) : error LNK2001: unresolved external symbol "public: static struct v8::internal::Runtime::Function const * __cdecl v8::internal::Runtime::FunctionForEntry(unsigned char *)" (?FunctionForEntry@Runtime@internal@v8@@SAPBUFunction@123@PAE@Z) [c:\ws\deps\v8\src\mksnapshot.vcxproj]
v8_base_0.lib(js-operator.obj) : error LNK2001: unresolved external symbol "class std::basic_ostream<char,struct std::char_traits<char> > & __cdecl v8::internal::operator<<(class std::basic_ostream<char,struct std::char_traits<char> > &,enum v8::internal::Runtime::FunctionId)" (??6internal@v8@@YAAAV?$basic_ostream@DU?$char_traits@D@std@@@std@@AAV23@W4FunctionId@Runtime@01@@Z) [c:\ws\deps\v8\src\mksnapshot.vcxproj]
c:\ws\Release\mksnapshot.exe : fatal error LNK1120: 5 unresolved externals [c:\ws\deps\v8\src\mksnapshot.vcxproj]
Build step 'Conditional steps (multiple)' marked build as failure
Skipped archiving because build is not successful
Sending e-mails to: [email protected] [email protected]
Notifying upstream projects of job completion
Finished: FAILURE
@mhdawson
Copy link
Member Author

@joaocgreis

@joaocgreis
Copy link
Member

This was tracked and fixed for x64 in nodejs/v8#4 .

We don't have 32 bit builds in the test CI because we don't have real 32 bit VMs, but this issue makes a very good case for adding WoW64 somewhere in the matrix.

@kunalspathak
Copy link
Member

We don't have 32 bit builds in the test CI

Is it possible to pass x86 to vcbuild.bat and that should trigger 32 bit builds or is it that it won't simulate the behavior we would get while running on Wow64?

@joaocgreis
Copy link
Member

Yes, that's even how we do the x86 releases. But we don't have 32 bit machines to test a 32 bit node.exe natively, WoW64 will have to do.

@joaocgreis
Copy link
Member

Added a x86 build to the test matrix to node-compile-windows on VS2015. The resulting 32 bit binary is tested on the 64 bit Windows 2008 machines. Test runs:

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

No branches or pull requests

3 participants