Patchwork D7166: packaging: ship all help .txt files on WiX

login
register
mail settings
Submitter phabricator
Date Oct. 24, 2019, 1:56 a.m.
Message ID <differential-rev-PHID-DREV-etqu5tlrda2jy4jjiwsq-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/42556/
State Superseded
Headers show

Comments

phabricator - Oct. 24, 2019, 1:56 a.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  These are technically not needed. But it is easier to ship
  all files than to cherry-pick. A `make install` will copy
  these files, so the new behavior is consistent with that.
  This also makes WiX consistent with Inno, which is my main
  reason for doing this.
  
  If we don't want to ship the files (which is a valid argument),
  I think we can do that in a follow up.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  contrib/packaging/wix/help.wxs

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel
phabricator - Nov. 9, 2019, 1:46 a.m.
martinvonz added a comment.


  I get this test failure from this commit:
  
    @@ -223,11 +223,6 @@
     
       $ ( testrepohgenv; "$PYTHON" wixxml.py help )
       Not installed:
    -    help/common.txt
    -    help/hg-ssh.8.txt
    -    help/hg.1.txt
    -    help/hgignore.5.txt
    -    help/hgrc.5.txt
       Not tracked:
     
       $ ( testrepohgenv; "$PYTHON" wixxml.py templates )
    
    ERROR: test-install.t output changed

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7166/new/

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

To: indygreg, #hg-reviewers
Cc: martinvonz, mercurial-devel

Patch

diff --git a/contrib/packaging/wix/help.wxs b/contrib/packaging/wix/help.wxs
--- a/contrib/packaging/wix/help.wxs
+++ b/contrib/packaging/wix/help.wxs
@@ -17,6 +17,7 @@ 
         <Component Id="help.root" Guid="$(var.help.root.guid)" Win64='$(var.IsX64)'>
           <File Name="bundlespec.txt" />
           <File Name="color.txt" />
+          <File Name="common.txt" />
           <File Name="config.txt" KeyPath="yes" />
           <File Name="dates.txt" />
           <File Name="deprecated.txt" />
@@ -26,7 +27,11 @@ 
           <File Name="filesets.txt" />
           <File Name="flags.txt" />
           <File Name="glossary.txt" />
+          <File Name="hg-ssh.8.txt" />
+          <File Name="hg.1.txt" />
+          <File Name="hgignore.5.txt" />
           <File Name="hgignore.txt" />
+          <File Name="hgrc.5.txt" />
           <File Name="hgweb.txt" />
           <File Name="merge-tools.txt" />
           <File Name="pager.txt" />