esp-mqtt does not compile with SSL enabled (IDFGH-2620)

espressif / esp-mqtt

ESP32 mqtt component

Apache License 2.0

603 stars 255 forks source link

esp-mqtt does not compile with SSL enabled (IDFGH-2620) #149

Closed eespi closed 4 years ago

eespi commented 4 years ago

Compile error after pulling master

/media/paolo/Data/working/FW/esp32/esp-idf/components/mqtt/esp-mqtt/mqtt_client.c: In function 'esp_mqtt_set_ssl_transport_properties':
/media/paolo/Data/working/FW/esp32/esp-idf/components/mqtt/esp-mqtt/mqtt_client.c:236:9: error: label '_mqtt_set_config_failed' used but not defined
goto _mqtt_set_config_failed;

david-cermak commented 4 years ago

@eespi Thanks for the quick report.

The issue is that it doesn't compile with older IDFs, posting a quick fix: 0001-fix-compilation-issue-on-v4.0.patch.txt before fixing on master.

Will add a CI check to compile the library agains all major releases from v3.2

eespi commented 4 years ago

thanks, it works now