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

Cannot open include file: 'stddef.h': No such file or directory on win2016-vs2015-COMPILED_BY-vs2019-x86 for Node.js 18 builds #3554

Closed
richardlau opened this issue Nov 2, 2023 · 5 comments

Comments

@richardlau
Copy link
Member

Node.js 18 builds are consistently failing on Windows in node-test-binary-windows-native-suites for win2016-vs2015-COMPILED_BY-vs2019-x86
e.g. https://ci.nodejs.org/job/node-test-binary-windows-native-suites/20206/nodes=win2016-vs2015-COMPILED_BY-vs2019-x86/console (from https://ci.nodejs.org/job/node-test-commit-windows-fanned/58999/):

04:21:39     "C:\\workspace\\node-test-binary-windows-native-suites\\node\\deps\\v8\\include\\v8.h(18): fatal error C1083: Cannot open include file: 'stddef.h': No such file or directory [C:\\workspace\\node-test-binary-windows-native-suites\\node\\test\\addons\\04_object_factory\\build\\addon.vcxproj]\r\n",
04:21:39   stderr: 'gyp info it worked if it ends with ok\n' +

@StefanStojanovic

@targos
Copy link
Member

targos commented Nov 2, 2023

This is relatively new. Here is a successful run I had two days ago on the staging branch: https://ci.nodejs.org/view/All/job/node-test-commit/66202/

@richardlau
Copy link
Member Author

This is relatively new. Here is a successful run I had two days ago on the staging branch: https://ci.nodejs.org/view/All/job/node-test-commit/66202/

win2016-vs2015-COMPILED_BY-vs2019-x86 looks to be recently added. From the successful https://ci.nodejs.org/job/node-test-binary-windows-native-suites/20177/console we can see that it ran on win2012r2 labels rather than the newer win2016 ones for the x86 builds.

09:51:26 Skipping win11-vs2022-arm64-COMPILED_BY-vs2019-arm64 for Node.js 18
09:51:26 Skipping win2022-vs2019-COMPILED_BY-vs2022 for Node.js 18
09:51:26 Skipping win11-vs2022-arm64-COMPILED_BY-vs2022-arm64 for Node.js 18
09:51:26 Skipping win2016-vs2017-COMPILED_BY-vs2022-x86 for Node.js 18
09:51:26 Skipping win2019-vs2019-COMPILED_BY-vs2022 for Node.js 18
09:51:26 Running nodes
09:51:26 Triggering [node-test-binary-windows-native-suites » win2012r2-vs2015-COMPILED_BY-vs2019-x86](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2012r2-vs2015-COMPILED_BY-vs2019-x86/)
09:51:26 Triggering [node-test-binary-windows-native-suites » win2012r2-vs2019-COMPILED_BY-vs2019-x86](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2012r2-vs2019-COMPILED_BY-vs2019-x86/)
09:51:26 Triggering [node-test-binary-windows-native-suites » win2022-vs2019-COMPILED_BY-vs2019](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2022-vs2019-COMPILED_BY-vs2019/)
09:56:20 Completed [node-test-binary-windows-native-suites » win2012r2-vs2015-COMPILED_BY-vs2019-x86](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2012r2-vs2015-COMPILED_BY-vs2019-x86/) SUCCESS
09:56:44 Completed [node-test-binary-windows-native-suites » win2012r2-vs2019-COMPILED_BY-vs2019-x86](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2012r2-vs2019-COMPILED_BY-vs2019-x86/) SUCCESS
09:56:52 Completed [node-test-binary-windows-native-suites » win2022-vs2019-COMPILED_BY-vs2019](https://ci.nodejs.org/job/node-test-binary-windows-native-suites/nodes=win2022-vs2019-COMPILED_BY-vs2019/) SUCCESS

@StefanStojanovic
Copy link
Contributor

Thanks for bringing this to my attention @richardlau, yes Win2016_VS2015 machines are new and I had some difficulties setting them up, mostly related to installing VS2015. For now, I've reverted back to using Win2012r2 machines until I fix this. I assume it is missing something from the VS although I'll have to investigate it further.

@StefanStojanovic
Copy link
Contributor

I made changes and tried running vcbuild.bat release noprojgen nobuild ignore-flaky test-ci-native. Since it passed I reenabled those machines for node-test-binary-windows-native-suites but also left Win2012r2 machines active just in case. After I see it passing in the CI as well, I'll exclude the old machines and close this issue

@StefanStojanovic
Copy link
Contributor

Closing this, since the job has been passing for the last few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants