Release 3.3 for esp-idf with Ubuntu 19.10

Same problem here.
Is it some kind of bug or just the binary growed out the partition?
And if i see right in the docs espruino does not support OTA updates. So, could we use the ota partition for the app like this:?

#Name,Type,SubType,Offset,Size
#boot,data,0,0x1000,4K
#reserved,0,0,0x2000,24K
#partition,data,0,0x8000,4K
nvs,data,nvs,0x9000,12K
otadata,data,ota,0xC000,8K
free,data,0x40,0xE000,8K
factory,app,factory,0x10000,2688K
flash,data,0x40,0x2B0000,64K
js_code,data,0,0x2C0000,256K
storage,data,0,0x300000,1024K

And the other problem is i'm not sure how to achieve this. I found one partition csv under:
target/esp32/Changes_V3.1/partitions_espruino.csv
But it looks like it's just a changelog maybe?
As @jumjum123 mentioned there is a EspruinoBuildTools which contains another csv but where i can find it? It looks like the source scripts/provision.sh ESP32 downloads it but here? I'm kind of confused.

No problem thank you for your answer. I was able to build a new partition binary without OTA partition and it works fine.

@davidbozo, can you describe your steps to successfully flash your ESP32? I'm running into the same problem but I'm not sure how to build a partition binary without an OTA partition.

No problem thank you for your answer. I was able to build a new partition binary without OTA partition and it works fine.

@davidbozo would be great if you share your knowledge, because we are short on ESP32 know how, since @wilberforce stoped working for Espruino on ESP32 ( I hope he is fine).

It's maybe a little hacky solution but there is no porblem with it yet.

So, what i could figure out is:
When we run this script: source scripts/provision.sh ESP32
It downloads binarys from this repo: EspruinoBuildTools
This repo contains the partitions_espruino.bin and when building espruino it's just copies this file.
So to change the partition table this EspruinoBuildTools project has to be rebuild and overwrite the old partitions_espruino.bin in the app/ folder with the new.
(This 'app' folder will be created after running provision.sh)

To rebuild the repo:
After cloning it modify this esp32/build/app/partitions_espruino.csv file:

#Name,Type,SubType,Offset,Size
#boot,data,0,0x1000,4K
#reserved,0,0,0x2000,24K
#partition,data,0,0x8000,4K
nvs,data,nvs,0x9000,12K
otadata,data,ota,0xC000,8K
free,data,0x40,0xE000,8K
factory,app,factory,0x10000,1344K <-- change the 1344K size to 2688K
ota_0,app,ota_0,0x160000,1344K <-- delete this line
flash,data,0x40,0x2B0000,64K
js_code,data,0,0x2C0000,256K
storage,data,0,0x300000,1024K

https://github.com/espruino/EspruinoBuildTools/tree/master/esp32
Here is some documentation at the bottom of the page. But instead of source ./build-idf.sh you can run source ./build-partition.sh in the build folder which is enough as far as i remember.

Sorry, this is what i remember and i can't repeat and test the steps only in work.

Ok, let's fix this.

• add firmware size check like we use in ESP8266

• provide a new partitions_espruino.bin

#Name,Type,SubType,Offset,Size
#boot,data,0,0x1000,4K
#reserved,0,0,0x2000,24K
#partition,data,0,0x8000,4K
nvs,data,nvs,0x9000,12K
otadata,data,ota,0xC000,8K
free,data,0x40,0xE000,8K
factory,app,factory,0x10000,1536K
#free2,data,0x40,0x190000,1536K
ota_0,app,ota_0,0x190000,1536K
flash,data,0x40,0x310000,64K
js_code,data,0,0x320000,256K
storage,data,0,0x360000,896K

edit: keep ota_0

How fortunate... Decided I wanted to spent some time on this this evening and I just noticed the comment and commits to EspruinoBuildTools by @MaBecker. Pulled and build everything once more with the new changes and Espruino on my ESP32 is now working.

Thanks a lot for the changes @MaBecker, you rock! ❤️

Cool, so you can build and it works for you, thanks for testing!

there are just two more things missing boards/ESP32.py and make/targets/ESP32.make just working on it.

Yeah, it's building for me without any additional changes (I actually removed everything and cloned the repository again just to check) and the error in the original post that I was running into before is gone on my side. Ran a small script to test if that works as well and that seems to be good too 👍

Just created a pr for the additional required updates

433708f

I guess it will not pull down the new app.tgz version

@gfwilliams any hints how to handle updates of app.tgz ?

Is the app.tgz for a new SDK? If so you could create as new branch (like was done for 3.1)?

Or just create a new file - app.3.1.tgz vs app.tgz

Still the same version , but with a new binary partition file.

Ok, I'd just update it. I know the whole build/version thing is tricky but presumably all it means it it was broken and now it's working?

The issue we are talking about is not the build process. The firmware did not fit into the partition.

To be 100% save, I have to flash a Travis build tonight.

just test, works like never was broken :)

 ____                 _
|  __|___ ___ ___ _ _|_|___ ___
|  __|_ -| . |  _| | | |   | . |
|____|___|  _|_| |___|_|_|_|___|
         |_| espruino.com
 2v04.401 (c) 2019 G.Williams
Espruino is Open Source. Our work is supported
only by sales of official boards and donations:
http://espruino.com/Donate
>process.env
={
  VERSION: "2v04.401",
  GIT_COMMIT: "e568c6c7",
  BOARD: "ESP32",
  FLASH: 0, STORAGE: 262144, RAM: 524288,
  SERIAL: "a4cf1276-64b4",
  CONSOLE: "Serial1",
  MODULES: "Flash,Storage,hea" ... "r,crypto,neopixel",
  EXPTR: 1073484860 }

wifi.save() does not work:

=function () { [native code] }
>wifi.connect('......', {password: '.....'}, function() {
:    console.log('Connected to Wifi.  IP address is:', wifi.getIP().ip);
:    wifi.save(); // Next reboot will auto-connect
:});
ERROR: Wifi: event_handler STA_START: esp_wifi_connect: 12298(SSID is invalid)
=undefined
WARNING: Wifi:startMDNS - espressif
Connected to Wifi.  IP address is: 192.168.1.219
Uncaught Error: File already written with different data
 at line 2 col 15
    wifi.save();
              ^
in function called from system
>wifi.save();
Uncaught Error: File already written with different data
 at line 1 col 11
wifi.save();
          ^
>