Fix up usermod libArchive settings

pio-scripts/load_usermods.py

@@ -2,16 +2,21 @@
 import os.path
 from collections import deque
 from pathlib import Path   # For OS-agnostic path manipulation
+from click import secho
+from SCons.Script import Exit
+from platformio.builder.tools.piolib import LibBuilderBase
 from platformio.package.manager.library import LibraryPackageManager
 
 usermod_dir = Path(env["PROJECT_DIR"]) / "usermods"
-all_usermods = [f for f in usermod_dir.iterdir() if f.is_dir() and f.joinpath('library.json').exists()]
 
+# "usermods" environment: expand list of usermods to everything in the folder
 if env['PIOENV'] == "usermods":
    # Add all usermods
+   all_usermods = [f for f in usermod_dir.iterdir() if f.is_dir() and f.joinpath('library.json').exists()]
    env.GetProjectConfig().set(f"env:usermods", 'custom_usermods', " ".join([f.name for f in all_usermods]))
 
-def find_usermod(mod: str):
+# Utility functions
+def find_usermod(mod: str) -> Path:
   """Locate this library in the usermods folder.
      We do this to avoid needing to rename a bunch of folders;
      this could be removed later
@@ -28,6 +33,13 @@ def find_usermod(mod: str):
     return mp
   raise RuntimeError(f"Couldn't locate module {mod} in usermods directory!")
 
+def is_wled_module(dep: LibBuilderBase) -> bool:
+  """Returns true if the specified library is a wled module
+  """
+  return usermod_dir in Path(dep.src_dir).parents or str(dep.name).startswith("wled-")
+
+## Script starts here
+# Process usermod option
 usermods = env.GetProjectOption("custom_usermods","")
 if usermods:
   # Inject usermods in to project lib_deps
@@ -82,13 +94,6 @@ def cached_add_includes(dep, dep_cache: set, includes: deque):
 
 # Our new wrapper
 def wrapped_ConfigureProjectLibBuilder(xenv):
-  # Update usermod properties
-  # Set libArchive before build actions are added
-  for um in (um for um in xenv.GetLibBuilders() if usermod_dir in Path(um.src_dir).parents):
-    build = um._manifest.get("build", {})
-    build["libArchive"] = False
-    um._manifest["build"] = build
-
   # Call the wrapped function
   result = old_ConfigureProjectLibBuilder.clone(xenv)()
 
@@ -102,12 +107,25 @@ def wrapped_ConfigureProjectLibBuilder(xenv):
   for dep in result.depbuilders:
      cached_add_includes(dep, processed_deps, extra_include_dirs)
 
-  for um in [dep for dep in result.depbuilders if usermod_dir in Path(dep.src_dir).parents]:
-    # Add the wled folder to the include path
-    um.env.PrependUnique(CPPPATH=wled_dir)
-    # Add WLED's own dependencies
-    for dir in extra_include_dirs:
-      um.env.PrependUnique(CPPPATH=dir)
+  broken_usermods = []
+  for dep in result.depbuilders:
+    if is_wled_module(dep):
+      # Add the wled folder to the include path
+      dep.env.PrependUnique(CPPPATH=str(wled_dir))
+      # Add WLED's own dependencies
+      for dir in extra_include_dirs:
+        dep.env.PrependUnique(CPPPATH=str(dir))
+      # Enforce that libArchive is not set; we must link them directly to the executable
+      if dep.lib_archive:
+        broken_usermods.append(dep)
+
+  if broken_usermods:
+    broken_usermods = [usermod.name for usermod in broken_usermods]
+    secho(
+      f"ERROR: libArchive=false is missing on usermod(s) {' '.join(broken_usermods)} -- modules will not compile in correctly",
+      fg="red",
+      err=True)    
+    Exit(1)
 
   return result
 

usermods/ADS1115_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "ADS1115_v2",
+  "build": { "libArchive": false },
   "dependencies": {
     "Adafruit BusIO": "https://github.com/adafruit/Adafruit_BusIO#1.13.2",
     "Adafruit ADS1X15": "https://github.com/adafruit/Adafruit_ADS1X15#2.4.0"

usermods/AHT10_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "AHT10_v2",
+  "build": { "libArchive": false },  
   "dependencies": {
     "enjoyneering/AHT10":"~1.1.0"
   }

usermods/Analog_Clock/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "Analog_Clock"
+  "name": "Analog_Clock",
+  "build": { "libArchive": false }
 }

usermods/Animated_Staircase/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "Animated_Staircase"
+  "name": "Animated_Staircase",
+  "build": { "libArchive": false }
 }

usermods/BH1750_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "BH1750_v2",
+  "build": { "libArchive": false },
   "dependencies": {
     "claws/BH1750":"^1.2.0"
   }

usermods/BME280_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "BME280_v2",
+  "build": { "libArchive": false },
   "dependencies": {
     "finitespace/BME280":"~3.0.0"
   }

usermods/BME68X_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "BME68X",
+  "build": { "libArchive": false },
   "dependencies": {
     "boschsensortec/BSEC Software Library":"^1.8.1492"
   }

usermods/Battery/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "Battery"
+  "name": "Battery",
+  "build": { "libArchive": false }
 }

usermods/Cronixie/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "Cronixie"
+  "name": "Cronixie",
+  "build": { "libArchive": false }
 }

usermods/EXAMPLE/library.json

@@ -1,4 +1,5 @@
 {
   "name": "EXAMPLE",
+  "build": { "libArchive": false },
   "dependencies": {}
 }

usermods/EleksTube_IPS/library.json.disabled

@@ -1,5 +1,6 @@
 {
   "name:": "EleksTube_IPS",
+  "build": { "libArchive": false },
   "dependencies": {
     "TFT_eSPI" : "2.5.33"
   } 

usermods/INA226_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "INA226_v2",
+  "build": { "libArchive": false },
   "dependencies": {
     "wollewald/INA226_WE":"~1.2.9"
   }

usermods/Internal_Temperature_v2/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "Internal_Temperature_v2"
+  "name": "Internal_Temperature_v2",
+  "build": { "libArchive": false }
 }

usermods/LD2410_v2/library.json

@@ -1,5 +1,6 @@
 {
   "name": "LD2410_v2",
+  "build": { "libArchive": false },
   "dependencies": {
     "ncmreynolds/ld2410":"^0.1.3"
   }

usermods/LDR_Dusk_Dawn_v2/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "LDR_Dusk_Dawn_v2"
+  "name": "LDR_Dusk_Dawn_v2",
+  "build": { "libArchive": false }
 }

usermods/MY9291/library.json

@@ -1,4 +1,5 @@
 {
   "name": "MY9291",
+  "build": { "libArchive": false },
   "platforms": ["espressif8266"]
 }

usermods/PIR_sensor_switch/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "PIR_sensor_switch"
+  "name": "PIR_sensor_switch",
+  "build": { "libArchive": false }
 }

usermods/PWM_fan/library.json

@@ -1,6 +1,7 @@
 {
   "name": "PWM_fan",
   "build": {
+    "libArchive": false,
     "extraScript": "setup_deps.py"
   }
 }

usermods/RTC/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "RTC"
+  "name": "RTC",
+  "build": { "libArchive": false }
 }

usermods/SN_Photoresistor/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "SN_Photoresistor"
+  "name": "SN_Photoresistor",
+  "build": { "libArchive": false }
 }

usermods/ST7789_display/library.json.disabled

@@ -1,3 +1,4 @@
 {
-  "name:": "ST7789_display"
+  "name:": "ST7789_display",
+  "build": { "libArchive": false }
 }

usermods/Si7021_MQTT_HA/library.json

@@ -1,5 +1,6 @@
 {
   "name": "Si7021_MQTT_HA",
+  "build": { "libArchive": false },
   "dependencies": {
     "finitespace/BME280":"3.0.0",
     "adafruit/Adafruit Si7021 Library" : "1.5.3"

usermods/TetrisAI_v2/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "TetrisAI_v2"
+  "name": "TetrisAI_v2",
+  "build": { "libArchive": false }
 }

usermods/boblight/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "boblight"
+  "name": "boblight",
+  "build": { "libArchive": false }
 }

usermods/buzzer/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "buzzer"
+  "name": "buzzer",
+  "build": { "libArchive": false }
 }

usermods/deep_sleep/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "deep_sleep"
+  "name": "deep_sleep",
+  "build": { "libArchive": false }
 }

usermods/multi_relay/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "multi_relay"
+  "name": "multi_relay",
+  "build": { "libArchive": false }
 }

usermods/pwm_outputs/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "pwm_outputs"
+  "name": "pwm_outputs",
+  "build": { "libArchive": false }
 }

usermods/sd_card/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "sd_card"
+  "name": "sd_card",
+  "build": { "libArchive": false }
 }

usermods/seven_segment_display/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "seven_segment_display"
+  "name": "seven_segment_display",
+  "build": { "libArchive": false }
 }

usermods/seven_segment_display_reloaded/library.json

@@ -1,6 +1,7 @@
 {
   "name": "seven_segment_display_reloaded",
   "build": {
+    "libArchive": false,
     "extraScript": "setup_deps.py"
-  }  
+  }
 }

usermods/sht/library.json

@@ -1,5 +1,6 @@
 {
   "name": "sht",
+  "build": { "libArchive": false },
   "dependencies": {
     "robtillaart/SHT85": "~0.3.3"
   }

usermods/smartnest/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "smartnest"
+  "name": "smartnest",
+  "build": { "libArchive": false }
 }

usermods/stairway_wipe_basic/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "stairway_wipe_basic"
+  "name": "stairway_wipe_basic",
+  "build": { "libArchive": false }
 }

usermods/usermod_rotary_brightness_color/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "usermod_rotary_brightness_color"
+  "name": "usermod_rotary_brightness_color",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_HttpPullLightControl/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "usermod_v2_HttpPullLightControl"
+  "name": "usermod_v2_HttpPullLightControl",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_RF433/library.json

@@ -1,5 +1,6 @@
 {
   "name": "usermod_v2_RF433",
+  "build": { "libArchive": false },
   "dependencies": {
     "sui77/rc-switch":"2.6.4"
   }  

usermods/usermod_v2_animartrix/library.json

@@ -1,5 +1,6 @@
 {
   "name": "animartrix",
+  "build": { "libArchive": false },
   "dependencies": {
     "Animartrix": "https://github.com/netmindz/animartrix.git#b172586"
   }    

usermods/usermod_v2_auto_save/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "auto_save"
+  "name": "auto_save",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_brightness_follow_sun/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "brightness_follow_sun"
+  "name": "brightness_follow_sun",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_four_line_display_ALT/library.json

@@ -1,5 +1,6 @@
 {
   "name": "four_line_display_ALT",
+  "build": { "libArchive": false },
   "dependencies": {
     "U8g2": "~2.34.4",
     "Wire": ""

usermods/usermod_v2_klipper_percentage/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "usermod_v2_klipper_percentage"
+  "name": "usermod_v2_klipper_percentage",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_ping_pong_clock/library.json

@@ -1,3 +1,4 @@
 {
-  "name": "usermod_v2_ping_pong_clock"
+  "name": "usermod_v2_ping_pong_clock",
+  "build": { "libArchive": false }
 }

usermods/usermod_v2_rotary_encoder_ui_ALT/library.json

@@ -1,6 +1,7 @@
 {
   "name": "rotary_encoder_ui_ALT",
   "build": {
+    "libArchive": false,
     "extraScript": "setup_deps.py"
   }
 }