Skip to content

Commit

Permalink
Merge #649: Fix Jade CI build pollution
Browse files Browse the repository at this point in the history
5191d21 jade: clean all jade build artifacts when building jade simulator (Jamie C. Driver)

Pull request description:

  It appears that the Jade CI is being polluted by the output of prior runs.  This needs removing, otherwise we're not getting a true clean build for this commit using the current tools, but rather some cached values (now out of date and not applicable) can interfere with the current build.

Top commit has no ACKs.

Tree-SHA512: 3395d3103b2593231220e30bd448a136d52c9ba225815369aadc627945ff114002b9635af8377bdfe9464680f5840d21dd0ab8382a33230f758a8150f62ea340
  • Loading branch information
achow101 committed Dec 8, 2022
2 parents e7af45d + 5191d21 commit 36ccaf7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/setup_environment.sh
Original file line number Diff line number Diff line change
Expand Up @@ -392,8 +392,8 @@ if [[ -n ${build_jade} ]]; then
# always seem to pick up the locally installed python virtualenv, and instead uses
# the system python/no-virtualenv which fails ...)
# Only install the tools we need (ie. esp32)
rm -fr "${IDF_TOOLS_PATH}"
cd esp-idf
rm -fr ${IDF_TOOLS_PATH}
./install.sh esp32
cd ..

Expand All @@ -404,7 +404,7 @@ if [[ -n ${build_jade} ]]; then
cd jade
rm -fr sdkconfig
cp configs/sdkconfig_qemu.defaults sdkconfig.defaults
idf.py all
idf.py fullclean all

# Make the qemu flash image
esptool.py --chip esp32 merge_bin --fill-flash-size 4MB -o main/qemu/flash_image.bin \
Expand Down

0 comments on commit 36ccaf7

Please sign in to comment.