ci: drop Debug matrix entries and replace ISLE_DEBUG with ISLE_CITRO3D_DEBUG

The Linux (Debug) and MSVC (x64 Debug) jobs only differed from their
Release counterparts in -DISLE_DEBUG=ON. ISLE_DEBUG was only consulted
in one place: miniwin/CMakeLists.txt for the Nintendo 3DS citro3d
library selection (citro3dd vs citro3d) — which the Nintendo 3DS matrix
row never enabled. So Linux (Debug)/MSVC (x64 Debug) produced binaries
identical to their Release builds and collided with them in the release
artifact merge, which is what corrupted #646.

Drop the two duplicate matrix entries and the now-unused ISLE_DEBUG
plumbing, and reintroduce the 3DS knob as ISLE_CITRO3D_DEBUG scoped to
its actual use site.

Reverts the -debug filename suffix added in the previous commit; with
the duplicates gone there is nothing left to disambiguate.

Fixes #646

.github/workflows/ci.yml

@@ -38,10 +38,8 @@ jobs:
       matrix:
         include:
           - { name: 'Linux',               os: 'ubuntu-latest',  generator: 'Ninja',                 dx5: false, config: true,  linux: true, werror: true,   clang-tidy: true, build-assets: true }
-          - { name: 'Linux (Debug)',       os: 'ubuntu-latest',  generator: 'Ninja',                 dx5: false, config: true,  linux: true, werror: true,   clang-tidy: true,   debug: true }
           - { name: 'MSVC (x86)',          os: 'windows-latest', generator: 'Ninja',                 dx5: true,  config: false, msvc: true,  werror: false,  clang-tidy: false,  vc-arch: 'amd64_x86' }
           - { name: 'MSVC (x64)',          os: 'windows-latest', generator: 'Ninja',                 dx5: false, config: true,  msvc: true,  werror: false,  clang-tidy: false,   vc-arch: 'amd64' }
-          - { name: 'MSVC (x64 Debug)',    os: 'windows-latest', generator: 'Ninja',                 dx5: false, config: true,  msvc: true,  werror: false,  clang-tidy: false,   vc-arch: 'amd64',   debug: true }
           - { name: 'MSVC (arm64)',        os: 'windows-latest', generator: 'Ninja',                 dx5: false, config: false, msvc: true,  werror: false,  clang-tidy: false,  vc-arch: 'amd64_arm64' }
           - { name: 'msys2 mingw64',       os: 'windows-latest', generator: 'Ninja',                 dx5: false, config: true,  mingw: true, werror: true,   clang-tidy: true,   msystem: 'mingw64', msys-env: 'mingw-w64-x86_64', shell: 'msys2 {0}' }
           - { name: 'macOS',               os: 'macos-latest',   generator: 'Ninja',                 dx5: false, config: true,  brew: true,  werror: true,   clang-tidy: false }
@@ -180,7 +178,6 @@ jobs:
               -DISLE_BUILD_CONFIG=${{ !!matrix.config }} \
               -DENABLE_CLANG_TIDY=${{ !!matrix.clang-tidy  }} \
               -DISLE_WERROR=${{ !!matrix.werror }} \
-              -DISLE_DEBUG=${{ matrix.debug || 'OFF' }} \
               -Werror=dev"
 
       - name: Configure (CMake)
@@ -194,7 +191,6 @@ jobs:
             -DISLE_BUILD_ASSETS=${{ !!matrix.build-assets }} \
             -DENABLE_CLANG_TIDY=${{ !!matrix.clang-tidy  }} \
             -DISLE_WERROR=${{ !!matrix.werror }} \
-            -DISLE_DEBUG=${{ matrix.debug || 'OFF' }} \
             -Werror=dev
 
       - name: Build (CMake)
@@ -243,14 +239,6 @@ jobs:
             --output appimage && \
           mv *.AppImage dist/
 
-      - name: Suffix debug AppImage
-        if: ${{ matrix.linux && matrix.debug }}
-        run: |
-          cd build/dist
-          for f in *.AppImage; do
-            mv "$f" "${f%.AppImage}-debug.AppImage"
-          done
-
       - name: Package (3DS)
         if: ${{ matrix.n3ds }}
         run: |

CMakeLists.txt

@@ -1083,9 +1083,6 @@ add_subdirectory(packaging)
 
 set(CPACK_PACKAGE_DIRECTORY "dist")
 set(CPACK_PACKAGE_FILE_NAME "isle-${PROJECT_VERSION}-${ISLE_PACKAGE_NAME}")
-if(ISLE_DEBUG)
-  string(APPEND CPACK_PACKAGE_FILE_NAME "-debug")
-endif()
 if(NINTENDO_3DS)
   find_program(BANNERTOOL bannertool)
   find_program(MAKEROM makerom)

miniwin/CMakeLists.txt

@@ -111,7 +111,8 @@ if(VITA)
 endif()
 
 if(NINTENDO_3DS)
-  if(ISLE_DEBUG)
+  option(ISLE_CITRO3D_DEBUG "Link the debug build of citro3d (citro3dd) on Nintendo 3DS" OFF)
+  if(ISLE_CITRO3D_DEBUG)
     find_library(CITRO3D_LIBRARY NAMES citro3dd)
   else()
     find_library(CITRO3D_LIBRARY NAMES citro3d)