MQTT Client should compile with only Ethernet enabled

sparkfun / SparkFun_RTK_Everywhere_Firmware

Centimeter precision GPS/GNSS using L1/L2/L5 signals broadcast over Bluetooth, WiFi, and Ethernet.

http://docs.sparkfun.com/SparkFun_RTK_Everywhere_Firmware/

Other

14 stars 9 forks source link

MQTT Client should compile with only Ethernet enabled #357

Closed nseidle closed 2 months ago

nseidle commented 3 months ago

Using the following compile guard setup

  // #define COMPILE_BT       // Comment out to remove Bluetooth functionality
  // #define COMPILE_WIFI     // Comment out to remove WiFi functionality
  #define COMPILE_ETHERNET // Comment out to remove Ethernet (W5500) support

We should be able to compile but we cannot. the MQTT_Client.ino throws errors because it requires WiFi. Does ESP32 really require the adding of the WiFi library just so we can use the built in MQTT client over ethernet?

nseidle commented 2 months ago

This is in release v1.3.