Skip to content

src,win: Replacement of unsupported versions of Windows runtime exit. #33108

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.

Sign up for GitHub

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

Closed
xCykrix wants to merge 9 commits into from
Closed

src,win: Replacement of unsupported versions of Windows runtime exit. #33108

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
111226e
src,win,msi: changed exit on unsupported version to extended warning …
Apr 27, 2020
22e91a0
src: fix spacing for deprecation banner
Apr 27, 2020
7869c2f
fix: multiline comment for C++ linter
Apr 27, 2020
6ee8998
msi: adjust message for official support
Apr 27, 2020
55bcf51
src,win: relocate unsupported os warning to node.cc to allow --no-war…
Apr 28, 2020
f6c4c8b
fix: remove the two whitespace linting error...
Apr 28, 2020
a01b275
comment: remove unnecessary comment for windows
xCykrix Apr 28, 2020
63e62c8
comment: corrected grammar mistake
xCykrix Apr 28, 2020
329279f
Update tools/msvs/msi/product.wxs
xCykrix Apr 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
src,win,msi: changed exit on unsupported version to extended warning … 
…of limited support
  • Loading branch information
Samuel J Voeller committed Apr 28, 2020
commit 111226e81041d8f753c2ee925f7fbc1f43e17ffa
8 changes: 4 additions & 4 deletions src/node_main.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,13 @@
#include <WinError.h>

int wmain(int argc, wchar_t* wargv[]) {
// Windows Server 2012 (not R2) is supported until 10/10/2023, so we allow it
// to run in the experimental support tier.
// Display deprecation banner without exiting application for unsupported versions.
if (!IsWindows8Point1OrGreater() &&
!(IsWindowsServer() && IsWindows8OrGreater())) {
fprintf(stderr, "This application is only supported on Windows 8.1, "
"Windows Server 2012 R2, or higher.");
exit(ERROR_EXE_MACHINE_TYPE_MISMATCH);
"Windows Server 2012 R2, or higher. Node.js will not"
"accept bug reports or patches while using unsupported"
"versions of Windows.");
}

// Convert argv to UTF8
Expand Down
4 changes: 2 additions & 2 deletions tools/msvs/msi/product.wxs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
Compressed="yes"
InstallScope="perMachine"/>

<Condition Message="This application is only supported on Windows 8.1, Windows Server 2012 R2, or higher.">
<![CDATA[Installed OR (VersionNT >= 603) OR (VersionNT >= 602 AND MsiNTProductType <> 1)]]>
<Condition Message="This application is only supported on Windows 7, Windows Server 2008 R2, or higher.">
<![CDATA[Installed OR (VersionNT >= 601)]]>
Comment thread
xCykrix marked this conversation as resolved.
Outdated
</Condition>

<Media Id="1" Cabinet="media1.cab" EmbedCab="yes"/>
Expand Down