Closed burtonkent closed 1 year ago
Also I did try searching back through the commit history to see if total
's declaration was somehow deleted, causing the bug. I used:
git log -Gtotal components/sys/drivers/net.c
And it was added at the one commit and never changed.
Oh, that's sad. This probably has to do with the discussion around https://github.com/whitecatboard/Lua-RTOS-ESP32/pull/238 that has lead to not merging it yet. Despite that, it shouldn't have happened of cause.
Please try to cherry-pick/merge the corresponding commits into your source tree. Will it build afterwards?
Alternatively, you could just comment out the two lines:
int pct = (int)(((float)(address-update_partition->address))/total*100);
if (0 == pct%10) esp_task_wdt_reset();
Fixed with 6b455df4965c2242754a80896562e254ba0ee528 - closing. Thanks for reporting the issue!
Steps to reproduce if you already have Lua-RTOS-ESP32 set up to 'make':
make clean
to clear out past makes.make
components/sys/drivers/net.c
because the variabletotal
isn't declared before use on line 513. The error given is:It's strange because git blame shows: b22ddc939 components/sys/drivers/net.c (the0ne 2020-01-09 07:43:20 +0100 513) int pct = (int)(((float)(address-update_partition->address))/*100); Note the 2020 commit date. I'm surprised this wasn't found as a bug before. I have to wonder if this is my error somehow.