`esp32` missing from list of compatible architectures in `library.properties`

[ryniker]

Support for the Arduino Nano ESP32 board was added to the library:

#139

However, the esp32 architecture identifier has not been added to the architectures property in the library's library.properties metadata file:

Servo/library.properties

Line 9 in c2936a2

architectures=avr,megaavr,sam,samd,nrf52,stm32f4,mbed,mbed_nano,mbed_portenta,mbed_rp2040,renesas,renesas_portenta,renesas_uno,zephyr

This causes a spurious warning to be displayed when compiling the library for the ESP32 board:

WARNING: library Servo claims to run on avr, megaavr, sam, samd, nrf52, stm32f4, mbed, mbed_nano, mbed_portenta, mbed_rp2040, renesas, renesas_portenta, renesas_uno, zephyr architecture(s) and may be incompatible with your current board which runs on esp32 architecture(s).

It could also result in incorrect library discovery under certain conditions, as architecture matching is a significant factor in the priority system:

https://arduino.github.io/arduino-cli/dev/sketch-build-process/#architecture-matching

Additional context

Additional requests

• esp32 missing from list of compatible architectures in library.properties #152 (comment)

Related

• Library fails to compile with recent "esp32" platform versions #155

[niels747]

insane that the buildin servo library doesn't work on ESP32! Why?