.github/workflows/desktop-build.yml

@@ -458,17 +458,6 @@ jobs:
           VCPKG_FEATURE_FLAGS: dependencygraph
         run: .ci/compile.sh --server --test --vcpkg
 
-      # CPack reports a failed makensis run as "Problem running NSIS command"
-      # plus a path to NSISOutput.log, and that log lives only on the runner, so
-      # the actual compile error is otherwise nowhere in the workflow output.
-      # Dump it, or a broken installer script is undiagnosable from CI alone.
-      # shell: bash matters here - the default pwsh resolves `find` to Windows'
-      # text-search find.exe, which errors instead of searching.
-      - name: "Dump NSIS compile log"
-        if: failure() && matrix.os == 'Windows'
-        shell: bash
-        run: find . -name NSISOutput.log -exec cat {} +
-
       # Delete used cache to emulate a ccache update. See https://github.com/actions/cache/issues/342
       - name: "[macOS] Delete remote compiler cache (ccache)"
         if: matrix.os == 'macOS' && matrix.use_ccache == 1 && github.ref == 'refs/heads/master' && steps.restore_ccache.outputs.cache-hit

cmake/NSIS.template.in

@@ -15,6 +15,7 @@ Var ReinstallMode
 Var AllowForceClose
 Var GuardDir
 Var OldUninstallString
+Var StaleFindHandle
 Var StaleFindName
 Var IsUpgrade
 Var AbortReason
@@ -94,7 +95,7 @@ ${IfNot} ${Errors}
       /S : Silent install$\n\
       /R : Silent upgrade$\n\
       /PID=%pid% : Process to wait for before replacing files (used by the in-app updater)$\n\
-      /D=%directory% : Specify destination directory$\n"
+      /D=%directory% : Specify destination directory$\n\"
     Quit
 ${EndIf}
 
@@ -518,9 +519,9 @@ Function WaitForLauncherToExit
     ; do with the upgrade.
     ClearErrors
     System::Call 'kernel32::OpenProcess(i 0x00100000, i 0, i $WaitPid) p.r0'
-    ${If} ${Errors}
-    ; The launcher is gone - OpenProcess cannot open a dead process - which is
-    ; exactly the state the wait was for.
+    ${If} $Errors
+        ; The launcher is gone - OpenProcess cannot open a dead process - which is
+        ; exactly the state the wait was for.
         DetailPrint "The application that started this installer has already exited."
         StrCpy $LauncherExited 1
         Return
@@ -685,16 +686,10 @@ Function CheckStaleRuntimeMask
     ${If} $R0 = 1
         Return
     ${EndIf}
-    ; FindFirst takes the pattern and the name variable only, and leaves the
-    ; search handle in $0 - there is no handle argument and no handle variable
-    ; to keep, which is why FindNext does not take one either. $R3 and $R0 are
-    ; untouched by it, so only $0 is consumed here.
-    ; One match is enough: $R0 is a yes/no flag, not a list, so there is no
-    ; FindNext and nothing to iterate.
-    FindFirst "$INSTDIR\$R3" $StaleFindName
-    ${If} $0 != ""
+    FindFirst $StaleFindHandle $StaleFindName "$INSTDIR\$R3"
+    ${If} $StaleFindHandle != ""
         StrCpy $R0 1
-        FindClose $0
+        FindClose $StaleFindHandle
     ${EndIf}
 FunctionEnd