Merge pull request #415 from phst-randomizer/tox-pyinstaller

Run pyinstaller build in isolated env using `tox`

.github/workflows/build-test-release.yml

@@ -483,11 +483,8 @@ jobs:
       - name: Upgrade `pip` and install `wheel`
         run: python -m pip install --upgrade pip wheel
 
-      - name: Install randomizer dependencies
-        run: pip install .
-
-      - name: Install pyinstaller
-        run: pip install -U pyinstaller
+      - name: Install tox
+        run: pip install tox
 
       - name: Download base patches
         uses: actions/download-artifact@v3
@@ -498,7 +495,7 @@ jobs:
           mv ./**/*.bps base/out
 
       - name: Build randomizer executable
-        run: pyinstaller ph_rando.spec
+        run: tox -e build
 
       - name: Get current commit hash
         id: commit-hash

pyproject.toml

@@ -33,6 +33,7 @@ types = [
     "types-setuptools~=68.2.0.0",
     "types-tqdm~=4.66.0.1",
 ]
+pyinstaller = ["pyinstaller==6.1.0"]
 
 [project.scripts]
 ph_rando = "ph_rando.ui.main:main"

tox.ini

@@ -30,5 +30,11 @@ extras =
 commands =
     pytest --numprocesses auto {posargs} tests/desmume
 
+[testenv:build]
+extras =
+    pyinstaller
+commands =
+    pyinstaller ph_rando.spec
+
 [pytest]
 addopts = --strict-markers --showlocals --verbose