Skip to content
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

Added ELF downloads for debugging tasmota firmware on exception crashes #370

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

rai68
Copy link

@rai68 rai68 commented Aug 15, 2023

made it so you can download 'FIRMWARE.ELF'

Let me know if there is anything ive messed up

@benzino77
Copy link
Owner

Thanks!
Could you please tell me why we need to add this functionality? I have to admit that I have no clue what for those ELF files would be needed.

@rai68
Copy link
Author

rai68 commented Aug 16, 2023

They're used by espExceptionDecoder to decode the stack trace of a hardware crash.

The program normally uses .elf files created by Arduino complier. But since here we are using Platform.io ESP exception decoder asks for an ELF file since the Tasmota firmware isn't complied on the same Arduino instance.

And the .elf file is created with the firmware and when trying to decode an exception it requires the specific .elf file created with the firmware. Using TasmoCompiler it can be hard to extract the .elf file if you get a exception crash.

So I added the functionally to be able to download it.

@benzino77
Copy link
Owner

I'm not sure that "average" Tasmota user will need to debug stack trace of their devices.I'm sure it is a nice feature to have it available, but I don't want to confuse users with "firmware.elf" button next to the "firmware.bin/firmware.bin.gz" button. Maybe a switch (somewhere in the menu/top AppBar) to turn on "Advanced mode" which will be disabled by default?
After choosing AM, one could reach additional GUI features like download ELF file.

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