Skip to content

Commit

Permalink
[cloudbuild] Fix CHEF builds for esp32. (#28295)
Browse files Browse the repository at this point in the history
Adds the workaround for gdbgui for chef builds as well.

Chef failures were missed because we had a 'keep going' flag to builds.
Removed that one as we should actually fix errors not ignore them.
  • Loading branch information
andy31415 authored and pull[bot] committed Nov 7, 2023
1 parent e62e8cd commit 3248827
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions integrations/cloudbuild/chef.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,9 @@ steps:
env:
- PW_ENVIRONMENT_ROOT=/pwenv
args:
- ./examples/chef/chef.py --build_all --keep_going --build_exclude
noip
- >-
perl -i -pe 's/^gdbgui==/# gdbgui==/' /opt/espressif/esp-idf/requirements.txt &&
./examples/chef/chef.py --build_all --build_exclude noip
id: CompileAll
waitFor:
- Bootstrap
Expand All @@ -30,7 +31,7 @@ steps:
env:
- PW_ENVIRONMENT_ROOT=/pwenv
args:
- ./examples/chef/chef.py --build_all --keep_going --build_include
- ./examples/chef/chef.py --build_all --build_include
linux_arm64_ipv6only.*noip
id: CompileNoip
waitFor:
Expand Down

0 comments on commit 3248827

Please sign in to comment.