Fix compile issue on new environments

[ascillato]

This PR fixes a dependency version for pyparsing needed for compiling ESP32.

There are some reports of this and the fix was being done manually:

• https://www.esp32.com/viewtopic.php?f=13&p=108741
• With v6 (ESP-IDF v5): ImportError: cannot import name 'rest_of_line' from 'pyparsing' #1007
• Feature Request: Support for IDF 5.0 #979

[Jason2866]

Hi Adrian, with the change CI fails https://github.com/Jason2866/platform-espressif32/actions/runs/4004743730

[ascillato]

Hi @Jason2866 ,

I see from that link that it is failing only for the examples of:

examples/espidf-arduino-blink
examples/espidf-arduino-wifiscan
examples/espidf-aws-iot

The example of espidf-aws-iot is forcing to use the previous SDK so, it won't work either way with lastest version of esp-idf.

The 2 examples of espidf-arduino-XX has framework = arduino, espidf in platformio.ini. Just changing that to framework = arduino it compiles without any issue.

So, seems that those examples needs some reviewing perhaps.

@valeros What do you think?

[Jason2866]

@ascillato agreed aws IoT example is out. But the other two should work. There is nothing special with. The examples are there to check the use of Arduino as a component of IDF.
The do work fine without the change of this PR. https://github.com/Jason2866/platform-espressif32/actions/runs/4006615412

[ascillato]

@Jason2866 Yes, when using

framework = arduino, espidf

it gives the error:

Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/rmt_periph.o
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/rtc_io_periph.o
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/sdio_slave_periph.o
linker script generation failed for /home/user/.platformio/packages/framework-espidf@3.40403.0/components/esp_system/ld/esp32/sections.ld.in
ERROR: failed to parse /home/user/.platformio/packages/framework-espidf@3.40403.0/components/esp_event/linker.lf
required keys ['entries'] for fragment not found, found '['  (at char 41), (line:2, col:5)
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/sdmmc_periph.o
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/sigmadelta_periph.o
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/spi_periph.o
Compiling .pio/build/esp-wrover-kit/bootloader/soc/esp32/timer_periph.o
*** [.pio/build/esp-wrover-kit/sections.ld] Error 1

but those 2 examples work if changing the framework to:
framework = arduino
or using the old version of pyparsing.

So, without updating pyparsing, a blank project on ESP32 with esp-idf 5.0 can't compile showing the error explained in the linked issues at the OP.

How can we fix the compilation for those 2 examples which use Arduino as a component of IDF? Which one is the script who do the compilation for framework = arduino, espidf?

[Jason2866]

The only ugly way (don't like), i see is to install the old pyparsing version when using Arduino as a component of IDF. For all other cases the new pyparsing version.
Noticed that there was already an update of pyparsing in a commit for IDF5 support which was later removed.

[PetteriAimonen]

I'm also seeing this problem in nanopb CI for two weeks now ( https://github.com/nanopb/nanopb/actions/runs/4039824116/jobs/6944916384 )

with config using only espidf:

[env:pio_esp32_idf]
platform = espressif32
board = firebeetle32
framework = espidf
lib_deps = Nanopb

and build in a clean, new directory with pio run.

[Jason2866]

Probably the best to merge this PR and remove Arduino as IDF component support. Users needing Arduino as a Component of IDF can use release v5.3.0.

[ascillato]

Agree.

@valeros Please, can you take a look into this? Thanks.

[Itaiboss]

I have run into the same issue. I have been manually replacing the pyparsing version in my penv. This would reset on clears and I would have to repeat the process. Editing the version in the espidf.py file has solved this for me. Please review this pr.

[ascillato]

The changes in the commit 8507dd8 work perfectly for me. Thank you so much!!!!

If in platformio.ini I use:
platform = https://github.com/platformio/platform-espressif32.git ; Latest Development
it works

But if I use
platform = espressif32 ; Latest Stable
or
platform = espressif32@6.0.0 ; Includes ESP-IDF v5.0
I obviously still having the same old compiling issue.

Are you planning on launching a new release for espressif32?

[ascillato]

https://github.com/platformio/platform-espressif32/releases/tag/v6.0.1

Thanks!!!! 👍