Skip to content

Commit 6dfcd81

Browse files
committed
ci(tools): Fix python tools build
1 parent fbf94b2 commit 6dfcd81

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

.github/workflows/build_py_tools.yml

+4-5
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ on:
1111
jobs:
1212
find-changed-tools:
1313
name: Check if tools have been changed
14-
runs-on: ubuntu-20.04
14+
runs-on: ubuntu-latest
1515
outputs:
1616
any_changed: ${{ steps.verify-changed-files.outputs.any_changed }}
1717
all_changed_files: ${{ steps.verify-changed-files.outputs.all_changed_files }}
@@ -47,7 +47,7 @@ jobs:
4747
strategy:
4848
fail-fast: false
4949
matrix:
50-
os: [windows-latest, macos-latest, ubuntu-20.04, ARM, ARM64]
50+
os: [windows-latest, macos-latest, ubuntu-latest, ARM, ARM64]
5151
include:
5252
- os: windows-latest
5353
TARGET: win64
@@ -56,7 +56,7 @@ jobs:
5656
- os: macos-latest
5757
TARGET: macos
5858
SEPARATOR: ':'
59-
- os: ubuntu-20.04
59+
- os: ubuntu-latest
6060
TARGET: linux-amd64
6161
SEPARATOR: ':'
6262
- os: ARM
@@ -89,7 +89,6 @@ jobs:
8989
- name: Checkout repository
9090
uses: actions/checkout@v4
9191
with:
92-
token: ${{ secrets.TOOLS_UPLOAD_PAT }}
9392
ref: ${{ github.event.pull_request.head.ref }}
9493
- name: Set up Python 3.8
9594
# Skip setting python on ARM because of missing compatibility: https://github.com/actions/setup-python/issues/108
@@ -108,7 +107,7 @@ jobs:
108107
pyinstaller --distpath ./${{ env.DISTPATH }} -F --icon=.github/pytools/espressif.ico tools/$tool.py
109108
done
110109
- name: Sign binaries
111-
if: matrix.os == 'windows-latest' && env.CERTIFICATE != '' && env.CERTIFICATE_PASSWORD != ''
110+
if: matrix.os == 'windows-latest' && env.CERTIFICATE != ''
112111
env:
113112
CERTIFICATE: ${{ secrets.CERTIFICATE }}
114113
CERTIFICATE_PASSWORD: ${{ secrets.CERTIFICATE_PASSWORD }}

0 commit comments

Comments
 (0)