Workflow: Switch to burningtyger Docker image to fix Windows path errors
This commit is contained in:
@@ -40,27 +40,17 @@ jobs:
|
|||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
- name: Build Windows executable via Docker (Unique Names)
|
- name: Build Windows executable via Docker (Modern & Robust)
|
||||||
run: |
|
run: |
|
||||||
# Ensure unique container name for this run
|
|
||||||
CONTAINER_NAME="builder-${{ github.run_id }}"
|
CONTAINER_NAME="builder-${{ github.run_id }}"
|
||||||
|
|
||||||
# Clean up any existing container with this name just in case
|
|
||||||
docker rm -f $CONTAINER_NAME || true
|
docker rm -f $CONTAINER_NAME || true
|
||||||
|
|
||||||
# Create container
|
# Using a more modern Python 3.10 image that handles paths better
|
||||||
docker create --name $CONTAINER_NAME cdrx/pyinstaller-windows
|
docker create --name $CONTAINER_NAME burningtyger/pyinstaller-windows "pip install -r requirements.txt && pyinstaller --onefile --name tamigo-cli tamigo.py"
|
||||||
|
|
||||||
# Copy source code
|
|
||||||
docker cp . $CONTAINER_NAME:/src
|
docker cp . $CONTAINER_NAME:/src
|
||||||
|
|
||||||
# Run build
|
|
||||||
docker start -a $CONTAINER_NAME
|
docker start -a $CONTAINER_NAME
|
||||||
|
|
||||||
# Copy results
|
|
||||||
docker cp $CONTAINER_NAME:/src/dist .
|
docker cp $CONTAINER_NAME:/src/dist .
|
||||||
|
|
||||||
# Clean up
|
|
||||||
docker rm $CONTAINER_NAME
|
docker rm $CONTAINER_NAME
|
||||||
|
|
||||||
- name: Debug - List output files
|
- name: Debug - List output files
|
||||||
|
|||||||
@@ -34,15 +34,19 @@ jobs:
|
|||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
- name: Build Windows executable via Docker (Unique Names)
|
- name: Build Windows executable via Docker (Modern & Robust)
|
||||||
run: |
|
run: |
|
||||||
CONTAINER_NAME="rel-builder-${{ github.run_id }}"
|
CONTAINER_NAME="rel-builder-${{ github.run_id }}"
|
||||||
docker rm -f $CONTAINER_NAME || true
|
docker rm -f $CONTAINER_NAME || true
|
||||||
docker create --name $CONTAINER_NAME cdrx/pyinstaller-windows
|
|
||||||
|
# Using burningtyger/pyinstaller-windows for better path handling
|
||||||
|
docker create --name $CONTAINER_NAME burningtyger/pyinstaller-windows "pip install -r requirements.txt && pyinstaller --onefile --name tamigo-cli tamigo.py"
|
||||||
|
|
||||||
docker cp . $CONTAINER_NAME:/src
|
docker cp . $CONTAINER_NAME:/src
|
||||||
docker start -a $CONTAINER_NAME
|
docker start -a $CONTAINER_NAME
|
||||||
docker cp $CONTAINER_NAME:/src/dist .
|
docker cp $CONTAINER_NAME:/src/dist .
|
||||||
docker rm $CONTAINER_NAME
|
docker rm $CONTAINER_NAME
|
||||||
|
|
||||||
- name: Debug - List output files
|
- name: Debug - List output files
|
||||||
run: ls -R dist/
|
run: ls -R dist/
|
||||||
- name: Rename for release
|
- name: Rename for release
|
||||||
|
|||||||
Reference in New Issue
Block a user