Merge pull request #36 from CSCfi/fix/windows-build-temporary-compromise

compromise fix for windows build: display cmd behind gui app
CSCfi · Apr 27, 2023 · f78e226 · f78e226
2 parents e77d901 + 5803352
commit f78e226
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -78,12 +78,12 @@ jobs:
         run: |
           echo '${{ secrets.DOCKER_PASSWORD }}' | docker login -u '${{ secrets.DOCKER_USERNAME }}' --password-stdin
           docker pull cscfi/pyinstaller
-          docker run --rm -v "${PWD}:/builder" cscfi/pyinstaller --noconsole --onefile sdagui.py --name ${{ matrix.asset_name }}
+          docker run --rm -v "${PWD}:/builder" cscfi/pyinstaller --onefile sdagui.py --name ${{ matrix.asset_name }}
           sudo chown -R $USER:$GROUP dist/
       - name: Build GUI artifact
         if: matrix.os != 'ubuntu-latest'
         run: |
-          pyinstaller --noconsole --onefile sdagui.py --name ${{ matrix.asset_name }}
+          pyinstaller --onefile sdagui.py --name ${{ matrix.asset_name }}
       - name: Build GUI Asset
         run: |
           cd ./dist