Improve script/package:

Rename `mas.pkg` as `mas-${version}.pkg`. Use `/usr/local/bin` for `--install-location` instead of having executable stored in `.pkg` under `mas_components.pkg/Payload/usr/local/bin`. Do not output SHA-256 of `.pkg`. Move `script/build …` earlier. Rename variables & temporary folders to follow Apple's terminology. Update comments. Partial mas-cli#638 Signed-off-by: Ross Goldberg <[email protected]>
rgoldberg · Dec 20, 2024 · 8994505 · 8994505
1 parent d5f9dba
commit 8994505
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 18 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -50,17 +50,17 @@ jobs:
         run: |
           script/bootstrap
 
-      - name: 📦 Build universal executable & package it in mas.pkg
+      - name: 📦 Build universal executable & package it in an installer
         run: |
           script/package
 
       - name: 📝 Create draft release
         run: |
           gh release create -d --generate-notes ${PRERELEASE} "${TAG_NAME}"
 
-      - name: 📤 Upload mas.pkg
+      - name: 📤 Upload installer
         run: |
-          gh release upload "${TAG_NAME}" .build/mas.pkg
+          gh release upload "${TAG_NAME}" ".build/mas-${TAG_NAME#v}.pkg"
 
       - name: 🚰 Bump mas-cli/tap/mas formula
         run: |

diff --git a/script/package b/script/package
@@ -3,36 +3,29 @@
 # script/package
 # mas
 #
-# Builds macOS installer component and distribution packages.
+# Builds .pkg installer.
 #
 
 . "${0:a:h}/_setup_script"
 
+script/build --arch arm64 --arch x86_64
+
 build_dir=.build
-distribution_package="${build_dir}/mas.pkg"
-distributions_root="${build_dir}/distributions"
+destination_root="${build_dir}/destination"
 version="$(script/version)"
 
-script/build --arch arm64 --arch x86_64
-
 printf $'==> 📦 Assembling installer package for mas %s\n' "${version}"
 
-ditto -v "${build_dir}/apple/Products/Release/mas" "${distributions_root}/usr/local/bin/mas"
+ditto -v "${build_dir}/apple/Products/Release/mas" "${destination_root}/mas"
 
-# Assemble macOS installer component package (aka product archive).
 pkgbuild \
   --identifier com.mphys.mas-cli \
-  --install-location / \
+  --install-location /usr/local/bin \
   --version "${version}" \
-  --root "${distributions_root}" \
+  --root "${destination_root}" \
   "${build_dir}/mas_components.pkg"
 
-# Build distribution package (aka product archive). Not sure why, but this is how Carthage does it.
-# https://github.com/Carthage/Carthage/blob/master/Makefile#L69
 productbuild \
   --distribution Package/Distribution.plist \
   --package-path "${build_dir}" \
-  "${distribution_package}"
-
-printf $'==> 🔢 File Hash\n'
-shasum -a 256 "${distribution_package}"
+  "${build_dir}/mas-${version}.pkg"