Skip to content

ymodem_send: report the files the device did not write - #128

Open
JaxCalder wants to merge 1 commit into
masterfrom
feature/upload-reports-skipped
Open

JaxCalder wants to merge 1 commit into
masterfrom
feature/upload-reports-skipped

Conversation

@JaxCalder

@JaxCalder JaxCalder commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This continues #127, which was merged before this commit was pushed.

With choco-technologies/dmell#64, the device's ymodem recv leaves out a file it cannot open (in use, e.g. the MP3 being played) and goes on with the batch. Its summary names those files. ymodem_send.py now shows them, suggests stopping what uses them, and exits with an error, so that upload-files / install-all do not pass over them silently:

ymodem_send: some files were not written - in use on the device? Stop what uses them (e.g. `service stop carhmi`) and send them again: Alexander.mp3

Test plan

  • STM32F746G-DISCO: the MP3 being played plus a small file. The small file is written, the MP3 is listed, exit 1.

🤖 Generated with Claude Code

The receiver leaves out a file it cannot open (in use - the MP3 being
played) and goes on with the batch (dmell's ymodem). The script shows
them and fails, so that a build target does not pass over them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants