Patchwork D12596: wix: bump the minimum Windows required to 8.1

login
register
mail settings
Submitter phabricator
Date April 28, 2022, 3:44 p.m.
Message ID <differential-rev-PHID-DREV-co6qqfdt3lvs6khn3jp5-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/50945/
State New
Headers show

Comments

phabricator - April 28, 2022, 3:44 p.m.
mharbison72 created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  PyOxidizer binaries are built with py 3.9.6, so not even Windows 7 is supported.
  Unfortunately, there don't appear to be more recent values for newer versions of
  Windows, but at least Windows 8.1 is still in extended support for another 9
  months or so.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D12596

AFFECTED FILES
  contrib/packaging/wix/mercurial.wxs

CHANGE DETAILS




To: mharbison72, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/contrib/packaging/wix/mercurial.wxs b/contrib/packaging/wix/mercurial.wxs
--- a/contrib/packaging/wix/mercurial.wxs
+++ b/contrib/packaging/wix/mercurial.wxs
@@ -33,8 +33,8 @@ 
            CompressionLevel='high' />
     <Property Id='DiskPrompt' Value="Mercurial $(var.Version) Installation [1]" />
 
-    <Condition Message='Mercurial MSI installers require Windows XP or higher'>
-        VersionNT >= 501
+    <Condition Message='Mercurial MSI installers require Windows 8.1 or higher'>
+        VersionNT >= 603
     </Condition>
 
     <Property Id="INSTALLDIR">