-
Notifications
You must be signed in to change notification settings - Fork 653
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CC1352P2_CC2652P_launchpad_coordinator_20240710.zip has __MACOSX folder in it #512
Comments
It's a Mac OS thing, you can delete it safely |
Yes, however the script that downloads the file (in the container in the example command line I pasted) is not designed to delete/ignore the folder, and it causes an error. The container I referenced is fairly popular and the command is likely used by lots of people. This limits people to downloading and extracting the .hex file from the zip which is less convenient than being able to specify a URL directly to the zip. |
I've got the same issue. |
Im using Zigstar-Addon in Home-Assist to update firmware, and this Addon will fail too with the maxos-drawer in Zip. Thank you for this thread. Without I had no clue how to solve zip "short read", because I know, that the zip is not "faulty". |
Fixed! |
The latest release's (and master) zip file has a __MACOSX folder (with a file in it) that causes an error when trying to use the following (common flashing) docker command:
docker run --rm --device /dev/ttyUSB0:/dev/ttyUSB0 -e FIRMWARE_URL=https://github.com/Koenkk/Z-Stack-firmware/releases/download/Z-Stack_3.x.0_coordinator_20240710/CC1352P2_CC2652P_launchpad_coordinator_20240710.zip ckware/ti-cc-tool -ewv -p /dev/ttyUSB0 --bootloader-sonoff-usb
The error mentions the unexpected __MACOSX folder and file, as the flashing tool is expecting a single file within the zip.
Previous releases of the zip did not contain any other files. I'm assuming that the inclusion of the __MACOSX folder was unintentional.
The text was updated successfully, but these errors were encountered: