From b5eb3f494b89edf56274f79688b285a3c2470c7f Mon Sep 17 00:00:00 2001
From: Simon Michael <simon@joyful.com>
Date: Tue, 26 Jul 2022 11:00:02 +0100
Subject: [PATCH] ci: test-linux-x64: preserve executable perms on zipped
 binaries

---
 .github/workflows/test-linux-x64.yml | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/test-linux-x64.yml b/.github/workflows/test-linux-x64.yml
index e66cb650d..20d0b710e 100644
--- a/.github/workflows/test-linux-x64.yml
+++ b/.github/workflows/test-linux-x64.yml
@@ -270,22 +270,25 @@ jobs:
         cd tmp
         mkdir hledger-linux-x64
         cd hledger-linux-x64
-        cp ~/.local/bin/hledger .
-        cp ~/.local/bin/hledger-ui .
-        cp ~/.local/bin/hledger-web .
+        cp -P ~/.local/bin/hledger .
+        cp -P ~/.local/bin/hledger-ui .
+        cp -P ~/.local/bin/hledger-web .
         strip hledger
         strip hledger-ui
         strip hledger-web
+        zip hledger-linux-x64.zip hledger hledger-ui hledger-web
+      # ^ create zip file ourselves, upload-artifact's zipping drops executable permissions
+      if: env.CONTINUE
+
       # how to set a context variable, and an attempt to make a nice artifact version suffix:
       # echo "::set-output name=version::$(git branch --show-current | sed 's/-.*//')-$(git rev-parse --short HEAD)"
-      if: env.CONTINUE
 
     # XXX intermittent upload failures
     - name: Upload executables artifact
-      uses: actions/upload-artifact@v2
+      uses: actions/upload-artifact@v3
       with:
         name: hledger-linux-x64
-        path: tmp/hledger-linux-x64
+        path: tmp/hledger-linux-x64.zip
       if: env.CONTINUE