Update MacQetDeploy_arm64.sh

misc/MacQetDeploy_arm64.sh

@@ -16,7 +16,6 @@
     # along with QElectroTech.  If not, see <http://www.gnu.org/licenses/>.
  # Need homebrew and coreutils installed see <http://brew.sh>.
 
- 
 #Force MacOSX12.3.sdk
 #see: https://www.downtowndougbrown.com/2023/08/how-to-create-a-qt-5-arm-intel-universal-binary-for-mac/
 
@@ -27,13 +26,13 @@ APPNAME='qelectrotech'
 BUNDLE=$APPNAME.app
 APPBIN="$BUNDLE/Contents/MacOS/$APPNAME"
 
-# Emplacement du script
+# Script location
 current_dir=$(dirname "$0")
 
-# On se remet au depart 
+# Go back to repo root
 cd "${current_dir}/../"
 
-# Emplacement courant
+# Current directory
 current_dir=$(PWD)
 
 
@@ -46,11 +45,11 @@ echo "Please see the \"Deploying an Application on Qt/Mac\""
 echo "page in the Qt documentation for more information."
 echo
 echo "This script :"
-echo "\t - up date the git depot"
+echo "\t - update the git depot"
-echo "\t - built the application bundle,"
+echo "\t - build the application bundle,"
 echo "\t - copy over required Qt frameworks,"
 echo "\t - copy additional files: translations, titleblocks and elements,"
-echo "\t - create image disk."
+echo "\t - create DMG disk image."
 echo
 echo "Enjoy ;-)"
 echo
@@ -70,24 +69,23 @@ echo
 echo "______________________________________________________________"
 echo "Run GIT:"
 
-# Fait une mise à jour
 git submodule init
 git submodule update
 git pull --recurse-submodules
 git pull
-#git checkout foliolist_position
-
-# recupere le numero de la nouvelle revision
 
+# Get revision number and version
 GITCOMMIT=$(git rev-parse --short HEAD)
 A=$(git rev-list HEAD --count)
 HEAD=$(($A+473))
 
-
 VERSION=$(cat sources/qetversion.cpp | grep "return QVersionNumber{"| head -n 1| awk -F "{" '{ print $2 }' | awk -F "}" '{ print $1 }' | sed -e 's/,/./g' -e 's/ //g')
 
-# Tarball de la dernière revision déjà créé
+DMG_NAME="${APPNAME}-$VERSION-r$HEAD-arm64.dmg"
-if [ -e "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip" ] ; then
+DMG_PATH="build-aux/mac-osx/$DMG_NAME"
+
+# Check if already built
+if [ -e "$DMG_PATH" ] ; then
     echo "There are not new updates, make disk image can"
     echo "take a lot of time (5 min). Can you continu?"
     echo  "[y/n]"
@@ -107,26 +105,25 @@ echo
 echo "______________________________________________________________"
 echo "Run make install:"
 
-# pour effacer l'ancienne compilation
+# Remove old bundle
 if [ -d $BUNDLE ] ; then
-    echo "Removing hold bundle..."
+    echo "Removing old bundle..."
     rm -rf $BUNDLE
 fi
 if [ -e Makefile ] ; then
-    echo "Removing hold Makefile..."
+    echo "Removing old Makefile..."
     rm .qmake.stash
     make clean
 fi
 
-# genere le Makefile
+# Generate Makefile
 echo "Generating new makefile..."
 qmake -spec macx-clang
 
-# compilation
+# Compile
 if [ -e Makefile.Release ] ; then
-	START_TIME=$SECONDS
+    START_TIME=$SECONDS
 
-    # arret du script si erreur de compilation
     testSuccessBuild () {
         if [ $? -ne 0 ]; then
             cleanVerionTag
@@ -137,7 +134,6 @@ if [ -e Makefile.Release ] ; then
         fi
     }
 
-    # utilise tout les coeurs pour une compilation plus rapide
     coeur=$(sysctl hw.ncpu | awk '{print $2}')
     if [ $? -ne 0 ]; then
         make -f Makefile.Release
@@ -148,8 +144,8 @@ if [ -e Makefile.Release ] ; then
     fi
 
     ELAPSED_TIME=$(($SECONDS - $START_TIME))
-	echo
+    echo
-	echo "The time of compilation is $(($ELAPSED_TIME/60)) min $(($ELAPSED_TIME%60)) sec" 
+    echo "The time of compilation is $(($ELAPSED_TIME/60)) min $(($ELAPSED_TIME%60)) sec"
 else
     echo "ERROR: Makefile not found. This script requires the macx-clang makespec"
     exit
@@ -157,8 +153,7 @@ fi
 
 cp -R ${current_dir}/misc/Info.plist qelectrotech.app/Contents/
 cp -R ${current_dir}/ico/mac_icon/*.icns qelectrotech.app/Contents/Resources/
-# On rajoute le numero de version pour "cmd + i"
+/usr/libexec/PlistBuddy -c "Set :CFBundleShortVersionString $VERSION r$HEAD" "qelectrotech.app/Contents/Info.plist"
-/usr/libexec/PlistBuddy -c "Set :CFBundleShortVersionString $VERSION r$HEAD" "qelectrotech.app/Contents/Info.plist"  # Version number
 
 
 ### copy over frameworks ############################################
@@ -174,28 +169,22 @@ if [ ! -d $BUNDLE ] ; then
 fi
 macdeployqt $BUNDLE
 
-### add file missing #######################################
+### add missing files ###############################################
 
 echo
 echo "______________________________________________________________"
-echo "Copy file missing:"
+echo "Copy missing files:"
 
-# Dossier à ajouter
 QET_ELMT_DIR="${current_dir}/elements/"
 QET_TBT_DIR="${current_dir}/titleblocks/"
 QET_LANG_DIR="${current_dir}/lang/"
 QET_EXAMPLES_DIR="${current_dir}/examples/"
 QET_FONTS_DIR="${current_dir}/fonts/"
 QET_LICENSES_DIR="${current_dir}/licenses/"
-
-
-# Add new folder for Qt dialog translation see
-## see <https://download.tuxfamily.org/qet/Qt_lang/>.
-
 LANG_DIR="${current_dir}/lang1/"
 
 if [ -d "${QET_ELMT_DIR}" ]; then
-    echo "Copying add elements in the bundle..."
+    echo "Copying elements in the bundle..."
     cp -R ${QET_ELMT_DIR} $BUNDLE/Contents/Resources/elements
 fi
 
@@ -205,26 +194,26 @@ if [ -d "${QET_TBT_DIR}" ]; then
 fi
 
 if [ -d "${QET_LANG_DIR}" ]; then
-    echo "Copying translations in the bundle... "
+    echo "Copying translations in the bundle..."
     mkdir $BUNDLE/Contents/Resources/lang
     cp ${current_dir}/lang/*.qm $BUNDLE/Contents/Resources/lang
 fi
 
 if [ -d "${LANG_DIR}" ]; then
-   echo "Copying translations in the bundle... "
+    echo "Copying extra translations in the bundle..."
-   cp ${current_dir}/lang1/*.qm $BUNDLE/Contents/Resources/lang
+    cp ${current_dir}/lang1/*.qm $BUNDLE/Contents/Resources/lang
 fi
 
 if [ -d "${QET_EXAMPLES_DIR}" ]; then
-   echo "Copying examples in the bundle... "
+    echo "Copying examples in the bundle..."
-   mkdir $BUNDLE/Contents/Resources/examples
+    mkdir $BUNDLE/Contents/Resources/examples
-   cp ${current_dir}/examples/*.qet $BUNDLE/Contents/Resources/examples
+    cp ${current_dir}/examples/*.qet $BUNDLE/Contents/Resources/examples
 fi
 
 if [ -d "${QET_FONTS_DIR}" ]; then
-   echo "Copying fonts in the bundle... "
+    echo "Copying fonts in the bundle..."
-   mkdir $BUNDLE/Contents/Resources/fonts
+    mkdir $BUNDLE/Contents/Resources/fonts
-   cp ${current_dir}/fonts/*.ttf $BUNDLE/Contents/Resources/fonts
+    cp ${current_dir}/fonts/*.ttf $BUNDLE/Contents/Resources/fonts
 fi
 
 if [ -d "${QET_LICENSES_DIR}" ]; then
@@ -233,74 +222,141 @@ if [ -d "${QET_LICENSES_DIR}" ]; then
     cp -R -L ${QET_LICENSES_DIR} $BUNDLE/Contents/Resources/licenses
 fi
 
-codesign  --force --deep --sign --timestamp -s "Developer ID Application: Laurent TRINQUES (Y73WZ6WZ5X)" --options=runtime $BUNDLE
+### Sign the bundle #################################################
-
-### create zip tarball for notarization ###############################################
 
 echo
 echo "______________________________________________________________"
-echo "Create zip tarball for notarization:"
+echo "Code signing bundle:"
 
-/usr/bin/ditto -c -k --keepParent $BUNDLE "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip"
+codesign --force --deep --sign --timestamp \
+    -s "Developer ID Application: Laurent TRINQUES (Y73WZ6WZ5X)" \
+    --options=runtime $BUNDLE
 
-### notarize zip tarball ###############################################
+### Create zip for notarization only ################################
-echo  -e "\033[1;31mWould you like to upload for Notarize packages "${APPNAME}"-"$VERSION"-"r$HEAD-arm64.zip", n/Y?.\033[m"
+# This ZIP is temporary — used only to submit to notarytool.
+# The final deliverable will be a DMG (see below).
+
+echo
+echo "______________________________________________________________"
+echo "Create temporary zip for notarization:"
+
+NOTARIZE_ZIP="/tmp/${APPNAME}-$VERSION-r$HEAD-arm64-notarize.zip"
+/usr/bin/ditto -c -k --keepParent $BUNDLE "$NOTARIZE_ZIP"
+
+### Notarize ########################################################
+
+echo -e "\033[1;31mWould you like to upload for notarization \"${APPNAME}-${VERSION}-r${HEAD}-arm64\", n/Y?\033[m"
 read a
 if [[ $a == "Y" || $a == "y" ]]; then
-echo
+    echo
-echo "______________________________________________________________"
+    echo "______________________________________________________________"
-echo "Notarize zip tarball:"
+    echo "Notarizing:"
-
+    xcrun notarytool submit "$NOTARIZE_ZIP" --keychain-profile "org.qelectrotech" --wait
-xcrun notarytool submit "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip" --keychain-profile "org.qelectrotech" --wait 
 else
-echo  -e "\033[1;33mExit.\033[m"
+    echo -e "\033[1;33mExit.\033[m"
 fi
 
-### Clean up zip used for notarization ########################################
+# Clean up temporary notarization zip
-echo 'Cleaning up notarization zip...'
+echo "Cleaning up temporary notarization zip..."
-rm "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip"
+rm -f "$NOTARIZE_ZIP"
 
-### Staple the notarization ticket to the .app ################################
+### Staple ##########################################################
-echo  -e "\033[1;31mWould you like to staple the app MacOS packages "${APPNAME}"-"$VERSION"-"r$HEAD", n/Y?.\033[m"
+
+echo -e "\033[1;31mWould you like to staple the app \"${APPNAME}-${VERSION}-r${HEAD}\", n/Y?\033[m"
 read a
 if [[ $a == "Y" || $a == "y" ]]; then
-xcrun stapler staple -v $BUNDLE
+    xcrun stapler staple -v $BUNDLE
+    # Verify staple is correctly applied
+    echo "Verifying staple..."
+    xcrun stapler validate -v $BUNDLE
+    spctl -a -vv $BUNDLE
 else
-echo  -e "\033[1;33mExit.\033[m"
+    echo -e "\033[1;33mExit.\033[m"
 fi
 
-### Re-create final zip tarball (without --sequesterRsrc so Gatekeeper works with Chrome) ###
+### Create final DMG ################################################
+# A DMG is used instead of a ZIP because it correctly preserves the
+# Gatekeeper staple when downloaded via Chrome or any other browser.
+# ZIP extraction via Archive Utility can strip extended attributes,
+# causing Gatekeeper to block the app with an "unverified developer"
+# warning. A signed and notarized DMG does not have this issue.
+
 echo
 echo "______________________________________________________________"
-echo "Re-create final zip tarball (stapled, Gatekeeper-compatible):"
+echo "Create final DMG (Gatekeeper-compatible with Chrome and Safari):"
 
-# Do NOT use --sequesterRsrc: it breaks extended attributes extraction
+mkdir -p "build-aux/mac-osx"
-# in Chrome / Archive Utility, causing Gatekeeper to block the app.
-# ditto without --sequesterRsrc preserves the staple correctly.
-/usr/bin/ditto -c -k --keepParent $BUNDLE "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip"
 
-### Clean up bundle #######################################################
+# Create a temporary writable DMG from the stapled .app bundle
-echo 'Cleaning up bundle...'
+hdiutil create \
+    -volname "QElectroTech $VERSION" \
+    -srcfolder "$BUNDLE" \
+    -ov \
+    -format UDRW \
+    -fs HFS+ \
+    "/tmp/qet_tmp.dmg"
+
+# Convert to compressed read-only DMG
+hdiutil convert "/tmp/qet_tmp.dmg" \
+    -format UDZO \
+    -o "$DMG_PATH"
+
+rm -f "/tmp/qet_tmp.dmg"
+
+# Sign the DMG itself
+codesign \
+    --sign "Developer ID Application: Laurent TRINQUES (Y73WZ6WZ5X)" \
+    --timestamp \
+    "$DMG_PATH"
+
+echo "DMG created and signed: $DMG_PATH"
+
+### Notarize the DMG ################################################
+
+echo -e "\033[1;31mWould you like to notarize the DMG \"${DMG_NAME}\", n/Y?\033[m"
+read a
+if [[ $a == "Y" || $a == "y" ]]; then
+    echo
+    echo "______________________________________________________________"
+    echo "Notarizing DMG:"
+    xcrun notarytool submit "$DMG_PATH" --keychain-profile "org.qelectrotech" --wait
+
+    echo "Stapling notarization ticket to DMG..."
+    xcrun stapler staple "$DMG_PATH"
+    echo "DMG notarized and stapled OK."
+else
+    echo -e "\033[1;33mExit.\033[m"
+fi
+
+### Clean up bundle #################################################
+
+echo "Cleaning up bundle..."
 rm -rf $BUNDLE
 
-### The end, process is done ##########################################
+### The end #########################################################
 
 echo
 echo "______________________________________________________________"
-echo "The process of creating deployable application zip is done."
+echo "The process is done."
-echo "The zip is in the folder 'build-aux/mac-osx'."
+echo "DMG is in the folder 'build-aux/mac-osx'."
 
-#rsync to TF DMG builds
+### Upload via rsync ################################################
-echo  -e "\033[1;31mWould you like to upload MacOS packages "${APPNAME}"-"$VERSION"-"r$HEAD-arm64.zip", n/Y?.\033[m"
+
+echo -e "\033[1;31mWould you like to upload MacOS package \"${DMG_NAME}\", n/Y?\033[m"
 read a
 if [[ $a == "Y" || $a == "y" ]]; then
-cp -Rf "build-aux/mac-osx/${APPNAME}-$VERSION-r$HEAD-arm64.zip" /Users/laurent/MAC_OS_X/
+    cp -Rf "$DMG_PATH" /Users/laurent/MAC_OS_X/
-rsync -e ssh -av --delete-after --no-owner --no-g --chmod=g+w --progress --exclude='.DS_Store' /Users/laurent/MAC_OS_X/ server:download.qelectrotech.org/qet/builds/MAC_OS_X/arm64/
+    rsync -e ssh -av --delete-after --no-owner --no-g --chmod=g+w \
-if [ $? != 0 ]; then
+        --progress --exclude='.DS_Store' \
-{
+        /Users/laurent/MAC_OS_X/ \
-echo "RSYNC ERROR: problem syncing ${APPNAME}-$VERSION-r$HEAD-arm64.zip"
+        server:download.qelectrotech.org/qet/builds/MAC_OS_X/arm64/
-rsync -e ssh -av --delete-after --no-owner --no-g --chmod=g+w --progress --exclude='.DS_Store' /Users/laurent/MAC_OS_X/ server:download.qelectrotech.org/qet/builds/MAC_OS_X/arm64/
+    if [ $? != 0 ]; then
-} fi
+        echo "RSYNC ERROR: problem syncing ${DMG_NAME}, retrying..."
+        rsync -e ssh -av --delete-after --no-owner --no-g --chmod=g+w \
+            --progress --exclude='.DS_Store' \
+            /Users/laurent/MAC_OS_X/ \
+            server:download.qelectrotech.org/qet/builds/MAC_OS_X/arm64/
+    fi
 else
-echo  -e "\033[1;33mExit.\033[m"
+    echo -e "\033[1;33mExit.\033[m"
 fi