What the developer wrote
Authored by Jon Griffiths
tools: remove dev_flash.sh
Let's reduce the number of ways devices get flashed in the near future -
ideally down to one script (plus manually via idf.py if desired).
What changed, and why it matters
This commit simply deletes a developer helper script used to flash firmware onto Blockstream Jade devices during development. There is no change to the actual device firmware, no bug fix, and no security-relevant behavior change. It is a routine cleanup to reduce the number of flashing scripts maintained by the project.
No security action needed. Treat as normal repository maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The removed dev_flash.sh was a build/flash convenience script. It optionally created a binary diff patch between an old and new firmware image and invoked jade_ota.py, or fell back to idf.py flash. Removing it does not alter firmware code, cryptographic checks, update mechanisms, or any runtime behavior. The same flashing capability remains available via idf.py and other existing scripts.
Changed components
tools/dev_flash.sh (deleted)Inspect captured patch +0 / −32
diff --git a/dev_flash.sh b/dev_flash.sh
deleted file mode 100755
index e54c429..0000000
--- a/dev_flash.sh
+++ /dev/null
@@ -1,32 +0,0 @@
-#!/bin/bash
-set -eo pipefail
-
-if [[ -f build/old_jade.bin ]]; then
- if [[ ! -f build/bsdiff ]]; then
- gcc -O2 -DBSDIFF_EXECUTABLE -o build/bsdiff components/esp32_bsdiff/bsdiff.c
- fi
- # builds just the app
- idf.py app
- rm -fr build/patch.bin
- ./build/bsdiff build/old_jade.bin build/jade.bin build/patch.bin
- FULL_SIZE_NEW=$(stat --printf="%s" build/jade.bin)
- FULL_SIZE_PATCH=$(stat --printf="%s" build/patch.bin)
- OLD_VERSION=$(strings -n 6 build/old_jade.bin | head -4 | egrep "^[0-9]*\.[0-9]*\.[0-9]*" | head -1 || true)
- NEW_VERSION=$(strings -n 6 build/jade.bin | head -4 | egrep "^[0-9]*\.[0-9]*\.[0-9]*" | head -1 || true)
- OLD_BLD_TYPE="ble"
- if [[ $(strings -n 6 ./build/old_jade.bin | head -4 | grep NORADIO) == "NORADIO" ]]; then
- OLD_BLD_TYPE="noradio"
- fi
- NEW_BLD_TYPE="ble"
- if [[ $(strings -n 6 ./build/jade.bin | head -4 | grep NORADIO) == "NORADIO" ]]; then
- NEW_BLD_TYPE="noradio"
- fi
- PATCH_NAME=build/${NEW_VERSION}_${NEW_BLD_TYPE}_from_${OLD_VERSION}_${OLD_BLD_TYPE}_sizes_${FULL_SIZE_NEW}_${FULL_SIZE_PATCH}_patch.bin
- python -c "import zlib; import sys; open(sys.argv[2], 'wb').write(zlib.compress(open(sys.argv[1], 'rb').read(), 9))" build/patch.bin ${PATCH_NAME}
- ./jade_ota.py --skipble --fwfile=${PATCH_NAME}
-else
- # flashes both bootloader and app
- idf.py flash
-fi
-
-cp build/jade.bin build/old_jade.bin
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.