Skip to content

Commit

Permalink
CI: use venv again, explicitly use brew-installed python3.13 instead …
Browse files Browse the repository at this point in the history
…of system python3
  • Loading branch information
jotelha committed Nov 7, 2024
1 parent 6b09789 commit 6f05cf4
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/build-on-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,25 +48,24 @@ jobs:
- name: Install pythonic dependencies
run: |
# python3 -m venv --system-site-packages venv
# source venv/bin/activate
python3.13 -m venv --system-site-packages venv
source venv/bin/activate
pip install --upgrade pip
pip install wheel setuptools_scm
# pip install -r pyinstaller/macos/requirements.txt
pip install .
pip install -r pyinstaller/macos/requirements.txt
pip install pyinstaller pyinstaller-hooks-contrib
- name: Fix setuptools_scm-generated version
id: fix_version
run: |
# source venv/bin/activate
source venv/bin/activate
version=$(SETUPTOOLS_SCM_DEBUG=1 python -m setuptools_scm) # have version written to file to appear in bundled app as well
echo "version=$version" >> $GITHUB_OUTPUT
- name: Log python package info
run: |
# source venv/bin/activate
source venv/bin/activate
echo "### pip freeze --local ###"
pip freeze --local | tee pip_freeze_local.txt
Expand All @@ -81,7 +80,7 @@ jobs:

- name: Package executable with pyinstaller
run: |
# source venv/bin/activate
source venv/bin/activate
pyinstaller -y ./pyinstaller/macos/dtool-lookup-gui-macos.spec 2>&1 | tee pyinstaller.log
ls -lh dist
mv dist/dtool-lookup-gui.app dtool-lookup-gui.app
Expand Down
4 changes: 0 additions & 4 deletions pyinstaller/macos/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -101,14 +101,10 @@ propcache==0.2.0
# via yarl
pyasn1==0.6.1
# via pysmb
pycairo==1.27.0
# via pygobject
pycparser==2.22
# via cffi
pygments==2.18.0
# via dtool-info
pygobject==3.50.0
# via -r requirements.in
pysmb==1.2.10
# via dtool-smb
python-dateutil==2.9.0.post0
Expand Down

0 comments on commit 6f05cf4

Please sign in to comment.