Skip to content

Commit 094b3b3

Browse files
jeplerdhalbert
authored andcommitted
include python executable path in the idf key
I noticed that the latest time we needed to switch the idf key was when the environment changed from python 3.10.4 to 3.10.5. I don't know if this is the only cause of needing to change the key, but it seems to be a good idea to make the key include it. I think this is what's going on because when I replaced a virtualenv's "python" with a dangling symlink (same as would occur when python's version gets bumped), I got a similar error.
1 parent 4525bc7 commit 094b3b3

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/build.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -382,6 +382,7 @@ jobs:
382382
if: ${{ needs.test.outputs.boards-espressif != '[]' }}
383383
steps:
384384
- name: Set up Python 3
385+
id: py3
385386
uses: actions/setup-python@v2
386387
with:
387388
python-version: "3.x"
@@ -400,7 +401,7 @@ jobs:
400401
id: idf-cache
401402
with:
402403
path: ${{ github.workspace }}/.idf_tools
403-
key: ${{ runner.os }}-idf-tools-${{ hashFiles('.git/modules/ports/espressif/esp-idf/HEAD') }}-20220404
404+
key: ${{ runner.os }}-idf-tools-${{ hashFiles('.git/modules/ports/espressif/esp-idf/HEAD') }}-${{ steps.py3.outputs.python-path }}-20220404
404405
- name: Clone IDF submodules
405406
run: |
406407
(cd $IDF_PATH && git submodule update --init)

0 commit comments

Comments
 (0)