build break -- optional header doesn't exist

Instead, use platformio_user.ini to allow users to override these settings.
joe714 · Apr 25, 2022 · f754202 · f754202
1 parent 52a07ae
commit f754202
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/ESPixelStick/src/network/WiFiDriver.cpp b/ESPixelStick/src/network/WiFiDriver.cpp
@@ -31,13 +31,15 @@
 #include "../FileMgr.hpp"
 
 //-----------------------------------------------------------------------------
-// Create secrets.h with a #define for SECRETS_SSID and SECRETS_PASS
+// Use platformio_user.ini to define these values.
+// Platformio_user.ini defines these in the [env] section.
 // or delete the #include and enter the strings directly below.
-#include "secrets.h"
-#ifndef SECRETS_SSID
+#if !defined(SECRETS_SSID)
 #   define SECRETS_SSID "DEFAULT_SSID_NOT_SET"
+#endif // SECRETS_SSID
+#if !defined(SECRETS_PASS)
 #   define SECRETS_PASS "DEFAULT_PASSPHRASE_NOT_SET"
-#endif // ndef SECRETS_SSID
+#endif // SECRETS_SSID
 
 /* Fallback configuration if config.json is empty or fails */
 const String default_ssid       = SECRETS_SSID;

diff --git a/platformio_user.ini.sample b/platformio_user.ini.sample
@@ -30,6 +30,9 @@
 ;build_type = debug
 ;monitor_port = COM3
 ;upload_port = COM3
+;build_flags =
+;    -D SECRETS_SSID="DEFAULT_SSID_NOT_SET"
+;    -D SECRETS_PASS="DEFAULT_PASSPHRASE_NOT_SET"
 
 ;[esp8266]
 ;monitor_port = COM4