From 20e5b4f4607ab07fce6a2164588c0005408cf762 Mon Sep 17 00:00:00 2001
From: Tres Finocchiaro <tres.finocchiaro@gmail.com>
Date: Wed, 29 May 2024 23:01:41 -0400
Subject: [PATCH] Fixes per review

---
 .github/workflows/build.yml |  6 +++---
 cmake/apple/CMakeLists.txt  | 13 +++++--------
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 275fc25f4cc..b8e2cbcf745 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -65,15 +65,16 @@ jobs:
           CCACHE_MAXSIZE: 500M
   macos:
     strategy:
+      fail-fast: false
       matrix:
         arch: [ x86_64, arm64 ]
         include:
           - arch: x86_64
             os: macos-12
-            xcode: 13.1
+            xcode: "13.1"
           - arch: arm64
             os: macos-14
-            xcode: 14.3.1
+            xcode: "14.3.1"
     name: macos-${{ matrix.arch }}
     runs-on: ${{ matrix.os }}
     env:
@@ -125,7 +126,6 @@ jobs:
                 -B build \
                 -DCMAKE_INSTALL_PREFIX="../target" \
                 -DCMAKE_PREFIX_PATH="$(brew --prefix qt@5)" \
-                -DCMAKE_APPLE_SILICON_PROCESSOR=${{ matrix.arch }} \
                 -DCMAKE_OSX_ARCHITECTURES=${{ matrix.arch }} \
                 $CMAKE_OPTS \
                 -DUSE_WERROR=OFF
diff --git a/cmake/apple/CMakeLists.txt b/cmake/apple/CMakeLists.txt
index 844f29fa08a..3fd0a4da457 100644
--- a/cmake/apple/CMakeLists.txt
+++ b/cmake/apple/CMakeLists.txt
@@ -19,21 +19,18 @@ CONFIGURE_FILE("lmms.plist.in"          "${CMAKE_BINARY_DIR}/Info.plist")
 CONFIGURE_FILE("install_apple.sh.in"    "${CMAKE_BINARY_DIR}/install_apple.sh" @ONLY)
 CONFIGURE_FILE("package_apple.json.in"  "${CMAKE_BINARY_DIR}/package_apple.json" @ONLY)
 
-IF(CMAKE_APPLE_SILICON_PROCESSOR)
-	# Target arch is provided to cmake (M-series only)
-	SET(DMG_ARCH "-${CMAKE_APPLE_SILICON_PROCESSOR}")
-ELSEIF(CMAKE_OSX_ARCHITECTURES)
-	SET(DMG_ARCH "-${CMAKE_OSX_ARCHITECTURES}")
+IF(CMAKE_OSX_ARCHITECTURES)
+	SET(DMG_ARCH "${CMAKE_OSX_ARCHITECTURES}")
 ELSEIF(IS_ARM64)
     # Target arch is host arch
-	SET(DMG_ARCH "-arm64")
+	SET(DMG_ARCH "arm64")
 ELSE()
 	# Fallback to Intel
-	SET(DMG_ARCH "-x86_64")
+	SET(DMG_ARCH "x86_64")
 ENDIF()
 
 # DMG creation target
-SET(DMG_FILE "${CMAKE_BINARY_DIR}/${CMAKE_PROJECT_NAME}-${VERSION}-mac${APPLE_OS_VER}${DMG_ARCH}.dmg")
+SET(DMG_FILE "${CMAKE_BINARY_DIR}/${CMAKE_PROJECT_NAME}-${VERSION}-mac${APPLE_OS_VER}-${DMG_ARCH}.dmg")
 
 FILE(REMOVE "${DMG_FILE}")
 ADD_CUSTOM_TARGET(removedmg