Browse Source

simplify debug launcher to only run inside a terminal

Armin Schrenk 3 months ago
parent
commit
b72982edd8

+ 1 - 8
.github/workflows/win-exe.yml

@@ -126,13 +126,6 @@ jobs:
           --no-man-pages
           --strip-debug
           --compress zip-0
-      - name: Prepare debug launcher config
-        shell: bash
-        run: envsubst '${SEMVER_STR} ${REVISION_NUM} ${APP_NAME} ${LOOPBACK_ALIAS}' < dist/win/resources/debug-launcher.properties > dist/win/resources/CryptomatorDebug.properties
-        env:
-          SEMVER_STR: ${{ needs.get-version.outputs.semVerStr }}
-          REVISION_NUM: ${{ needs.get-version.outputs.revNum }}
-          APP_NAME: 'Cryptomator'
       - name: Run jpackage
         run: >
           ${JAVA_HOME}/bin/jpackage
@@ -170,7 +163,7 @@ jobs:
           --java-options "-Djavafx.verbose=${{ inputs.isDebug }}"
           --resource-dir dist/win/resources
           --icon dist/win/resources/Cryptomator.ico
-          --add-launcher "CryptomatorDebug=CryptomatorDebug.properties"
+          --add-launcher "CryptomatorDebug=dist/win/debug-launcher.properties"
       - name: Patch Application Directory
         run: |
           cp dist/win/contrib/* appdir/Cryptomator

+ 6 - 8
dist/win/build.ps1

@@ -139,13 +139,6 @@ if ($clean -and (Test-Path -Path $appPath)) {
 	Remove-Item -Path $appPath -Force -Recurse
 }
 
-# prepare additional launcher
-$debugProps = Get-Content -Path $buildDir\resources\debug-launcher.properties
-$debugProps = $debugProps -replace '\${SEM_VER_STR}', "$semVerNo"
-$debugProps = $debugProps -replace '\${REVISION_NUM}', "$revisionNo"
-$debugProps = $debugProps -replace '\${APP_NAME}', "$AppName"
-$debugProps = $debugProps -replace '\${LOOPBACK_ALIAS}', "$LoopbackAlias"
-Set-Content -Path $buildDir\resources\${AppName}Debug.properties -Value $debugProps
 
 
 # create app dir
@@ -182,7 +175,12 @@ Set-Content -Path $buildDir\resources\${AppName}Debug.properties -Value $debugPr
 	--java-options "-Dcryptomator.buildNumber=`"msi-$revisionNo`"" `
 	--resource-dir resources `
 	--icon resources/$AppName.ico `
-	--add-launcher "${AppName}Debug=$buildDir\resources\${AppName}Debug.properties"
+	--add-launcher "${AppName}Debug=$buildDir\debug-launcher.properties" `
+
+if ($LASTEXITCODE -ne 0) {
+    Write-Error "jpackage Appimage failed with exit code $LASTEXITCODE"
+	return 1;
+}
 
 #Create RTF license for msi
 &mvn -B -f $buildDir/../../pom.xml license:add-third-party "-Djavafx.platform=win" `

+ 1 - 0
dist/win/debug-launcher.properties

@@ -0,0 +1 @@
+win-console=true

+ 0 - 18
dist/win/resources/debug-launcher.properties

@@ -1,18 +0,0 @@
-win-console=true
-java-options=--enable-preview \
-             --enable-native-access=org.cryptomator.jfuse.win \
-             -Xss5m \
-             -Xmx256m \
-             -Dcryptomator.appVersion="${SEM_VER_STR}" \
-             -Dfile.encoding="utf-8" \
-             -Dcryptomator.logDir="~/AppData/Roaming/${APP_NAME}" \
-             -Dcryptomator.pluginDir="~/AppData/Roaming/${APP_NAME}/Plugins" \
-             -Dcryptomator.settingsPath="~/AppData/Roaming/${APP_NAME}/settings.json" \
-             -Dcryptomator.ipcSocketPath="~/AppData/Roaming/${APP_NAME}/ipc.socket" \
-             -Dcryptomator.p12Path="~/AppData/Roaming/${APP_NAME}/key.p12" \
-             -Dcryptomator.mountPointsDir="~/${APP_NAME}" \
-             -Dcryptomator.loopbackAlias="${LOOPBACK_ALIAS}" \
-             -Dcryptomator.integrationsWin.autoStartShellLinkName="${APP_NAME}" \
-             -Dcryptomator.integrationsWin.keychainPaths="~/AppData/Roaming/${APP_NAME}/keychain.json" \
-             -Dcryptomator.showTrayIcon=true \
-             -Dcryptomator.buildNumber="msi-${REVISION_NUM}"