Skip to content

Commit

Permalink
CI: everything in venv in mingw
Browse files Browse the repository at this point in the history
  • Loading branch information
jotelha committed Nov 8, 2024
1 parent 6c2431e commit a93a248
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 0 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/build-installer-on-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,17 @@ jobs:
echo "### libcrypto and libssl ###"
pacman -Ql mingw-w64-x86_64-openssl | grep 'lib[^/]*dll$' | awk '{print $2}' | cygpath -wa --file - | tee openssl_dlls.txt
- name: Create local venv
run: |
# recursive copy_metadata needed local reinstall of
# $ pip install --ignore-installed six
# in minsys2/mingw64, would otherwise fail with
# RuntimeError: No metadata path found for distribution 'six'.
# in pyinstaller/hooks/hook-dtoolcore.py
python -m venv --system-site-packages venv
source venv/bin/activate
pip install --ignore-installed six
# strategy: only install remaining pythonic dependencies via pip
- name: Install remaining pythonic dependencies
run: |
Expand Down
11 changes: 11 additions & 0 deletions .github/workflows/build-on-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,17 @@ jobs:
echo "### libcrypto and libssl ###"
pacman -Ql mingw-w64-x86_64-openssl | grep 'lib[^/]*dll$' | awk '{print $2}' | cygpath -wa --file - | tee openssl_dlls.txt
- name: Create local venv
run: |
# recursive copy_metadata needed local reinstall of
# $ pip install --ignore-installed six
# in minsys2/mingw64, would otherwise fail with
# RuntimeError: No metadata path found for distribution 'six'.
# in pyinstaller/hooks/hook-dtoolcore.py
python -m venv --system-site-packages venv
source venv/bin/activate
pip install --ignore-installed six
# strategy: only install remaining pythonic dependencies via pip
- name: Install remaining pythonic dependencies
run: |
Expand Down

0 comments on commit a93a248

Please sign in to comment.