Arduino IDE can compile even without LMIC library when LORAWAN is disabled

0cae2988 · Eric Duminil · 44e04833 · 0cae2988 · 0cae2988 · 0cae2988
Commit 0cae2988 authored 4 years ago by Eric Duminil
Hide whitespace changes
Inline Side-by-side

Showing

with 3 additions and 3 deletions
+3 -3
--- a/config.public.h
+++ b/config.public.h
@@ -12,7 +12,7 @@
 #  define AMPEL_HTTP    // Should HTTP web server be started?
 #  define AMPEL_MQTT    // Should data be sent over MQTT?
 #  define AMPEL_CSV     // Should data be logged as CSV, on the ESP flash memory?
-// #  define AMPEL_LORAWAN // Should data be sent over LoRaWAN? (Requires ESP32 + LoRa modem)
+// #  define AMPEL_LORAWAN // Should data be sent over LoRaWAN? (Requires ESP32 + LoRa modem, and "MCCI LoRaWAN LMIC library")
 /**
 * WIFI

--- a/lorawan.cpp
+++ b/lorawan.cpp
 #include "lorawan.h"
-#if defined(ESP32)
+#if defined(AMPEL_LORAWAN) && defined(ESP32)
 namespace config {
  // Values should be defined in config.h

--- a/lorawan.h
+++ b/lorawan.h
 #ifndef AMPEL_LORAWAN_H_
 #define AMPEL_LORAWAN_H_
-#if defined(ESP32)
+#if defined(AMPEL_LORAWAN) && defined(ESP32)
 #include <Arduino.h>
 // Requires "MCCI LoRaWAN LMIC library", which will be automatically used with PlatformIO but should be added in "Arduino IDE".
 // Tested successfully with v3.2.0 and connected to a thethingsnetwork.org app.