Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

"/ZW" (C++/CX) is incompatible with "/std:c++20" #6

Open
xan105 opened this issue Aug 21, 2024 · 0 comments
Open

"/ZW" (C++/CX) is incompatible with "/std:c++20" #6

xan105 opened this issue Aug 21, 2024 · 0 comments
Labels
help wanted Extra attention is needed wontfix This will not be worked on

Comments

@xan105
Copy link
Owner

xan105 commented Aug 21, 2024

NodeRT package(s) are very old. They were made in the Win8 UWP era.
They require the /ZW flag (C++/CX | Windows Runtime Compilation).

Node v22 started to use /std:c++20.
That's an issue because it's incompatible with /ZW (C++/CX).

command line error D8016: '/ZW' and '/std:c++20' command-line options are incompatible

And this is unlikely to change because Microsoft has abandoned UWP/WinRT and recommends to move to C++/WinRT from C++/CX.

Altho forcing /std:c++17 worked before when building native addons for Node v22; Electron v32 requires it.

...\prebuildify\electron\32.0.0\include\node\v8config.h(13,1): error C1189: #error:  "C++20 or later required." 
[...\node-nodeRT\node_modules\@nodert-win10-rs4\windows.devices.lights\build\binding.vcxproj]
  (compiling source file '../_nodert_generated.cpp')
@xan105 xan105 added help wanted Extra attention is needed wontfix This will not be worked on labels Aug 21, 2024
@xan105 xan105 pinned this issue Aug 22, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
help wanted Extra attention is needed wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant