From f5169abde257a8085a6b0bcffd0d1fee793abadb Mon Sep 17 00:00:00 2001
From: dvermd <315743+dvermd@users.noreply.github.com>
Date: Mon, 8 May 2023 19:36:12 +0200
Subject: [PATCH] Remove unsupported.hpp (#2956)

* Remove unsupported.hpp

* Rename names.hpp to types.hpp

* Replace macro by function

* Add missing cassert include in controller.cpp
---
 include/modules/alsa.hpp        |  3 +-
 include/modules/backlight.hpp   |  3 +-
 include/modules/battery.hpp     |  3 +-
 include/modules/bspwm.hpp       |  3 +-
 include/modules/counter.hpp     |  3 +-
 include/modules/cpu.hpp         |  3 +-
 include/modules/date.hpp        |  3 +-
 include/modules/fs.hpp          |  3 +-
 include/modules/github.hpp      |  3 +-
 include/modules/i3.hpp          |  3 +-
 include/modules/ipc.hpp         |  3 +-
 include/modules/memory.hpp      |  3 +-
 include/modules/menu.hpp        |  3 +-
 include/modules/meta/all.hpp    |  1 -
 include/modules/meta/types.hpp  | 37 ++++++++++++++++++
 include/modules/mpd.hpp         |  3 +-
 include/modules/network.hpp     |  3 +-
 include/modules/pulseaudio.hpp  |  3 +-
 include/modules/script.hpp      |  3 +-
 include/modules/temperature.hpp |  3 +-
 include/modules/text.hpp        |  3 +-
 include/modules/tray.hpp        |  3 +-
 include/modules/unsupported.hpp | 68 ---------------------------------
 include/modules/xbacklight.hpp  |  3 +-
 include/modules/xkeyboard.hpp   |  3 +-
 include/modules/xwindow.hpp     |  3 +-
 include/modules/xworkspaces.hpp |  3 +-
 src/components/controller.cpp   |  1 +
 src/modules/meta/factory.cpp    | 42 +++++++++++++++++++-
 29 files changed, 126 insertions(+), 95 deletions(-)
 create mode 100644 include/modules/meta/types.hpp
 delete mode 100644 include/modules/unsupported.hpp

diff --git a/include/modules/alsa.hpp b/include/modules/alsa.hpp
index 3a86da85..cb4e851f 100644
--- a/include/modules/alsa.hpp
+++ b/include/modules/alsa.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/event_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -29,7 +30,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/alsa";
+    static constexpr auto TYPE = ALSA_TYPE;
 
     static constexpr auto EVENT_INC = "inc";
     static constexpr auto EVENT_DEC = "dec";
diff --git a/include/modules/backlight.hpp b/include/modules/backlight.hpp
index f1db0fe7..02542505 100644
--- a/include/modules/backlight.hpp
+++ b/include/modules/backlight.hpp
@@ -2,6 +2,7 @@
 
 #include "components/config.hpp"
 #include "modules/meta/inotify_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -26,7 +27,7 @@ namespace modules {
     bool on_event(const inotify_event& event);
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/backlight";
+    static constexpr auto TYPE = BACKLIGHT_TYPE;
 
     static constexpr const char* EVENT_INC = "inc";
     static constexpr const char* EVENT_DEC = "dec";
diff --git a/include/modules/battery.hpp b/include/modules/battery.hpp
index bd6ef788..0ee1fc31 100644
--- a/include/modules/battery.hpp
+++ b/include/modules/battery.hpp
@@ -2,6 +2,7 @@
 
 #include "common.hpp"
 #include "modules/meta/inotify_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -55,7 +56,7 @@ namespace modules {
     string get_format() const;
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/battery";
+    static constexpr auto TYPE = BATTERY_TYPE;
 
    protected:
     state current_state();
diff --git a/include/modules/bspwm.hpp b/include/modules/bspwm.hpp
index 9fb20ba6..f228b990 100644
--- a/include/modules/bspwm.hpp
+++ b/include/modules/bspwm.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/event_module.hpp"
+#include "modules/meta/types.hpp"
 #include "utils/bspwm.hpp"
 
 POLYBAR_NS
@@ -47,7 +48,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/bspwm";
+    static constexpr auto TYPE = BSPWM_TYPE;
 
     static constexpr auto EVENT_FOCUS = "focus";
     static constexpr auto EVENT_NEXT = "next";
diff --git a/include/modules/counter.hpp b/include/modules/counter.hpp
index a92d98f6..a074282d 100644
--- a/include/modules/counter.hpp
+++ b/include/modules/counter.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -12,7 +13,7 @@ namespace modules {
     bool update();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/counter";
+    static constexpr auto TYPE = COUNTER_TYPE;
 
    private:
     static constexpr auto TAG_COUNTER = "<counter>";
diff --git a/include/modules/cpu.hpp b/include/modules/cpu.hpp
index fff25da0..8f11bacf 100644
--- a/include/modules/cpu.hpp
+++ b/include/modules/cpu.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -26,7 +27,7 @@ namespace modules {
     string get_format() const;
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/cpu";
+    static constexpr auto TYPE = CPU_TYPE;
 
    protected:
     bool read_values();
diff --git a/include/modules/date.hpp b/include/modules/date.hpp
index cd49d643..4d63a487 100644
--- a/include/modules/date.hpp
+++ b/include/modules/date.hpp
@@ -6,6 +6,7 @@
 #include <iostream>
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -17,7 +18,7 @@ namespace modules {
     bool update();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/date";
+    static constexpr auto TYPE = DATE_TYPE;
 
     static constexpr auto EVENT_TOGGLE = "toggle";
 
diff --git a/include/modules/fs.hpp b/include/modules/fs.hpp
index 5fb880d2..a97661e7 100644
--- a/include/modules/fs.hpp
+++ b/include/modules/fs.hpp
@@ -2,6 +2,7 @@
 
 #include "components/config.hpp"
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -42,7 +43,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/fs";
+    static constexpr auto TYPE = FS_TYPE;
 
    private:
     static constexpr auto FORMAT_MOUNTED = "format-mounted";
diff --git a/include/modules/github.hpp b/include/modules/github.hpp
index e9a8d4c0..eeeeae7c 100644
--- a/include/modules/github.hpp
+++ b/include/modules/github.hpp
@@ -3,6 +3,7 @@
 #include <atomic>
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 #include "utils/http.hpp"
 
@@ -20,7 +21,7 @@ namespace modules {
     bool build(builder* builder, const string& tag) const;
     string get_format() const;
 
-    static constexpr auto TYPE = "internal/github";
+    static constexpr auto TYPE = GITHUB_TYPE;
 
    private:
     void update_label(int);
diff --git a/include/modules/i3.hpp b/include/modules/i3.hpp
index 8cfb7170..480967ab 100644
--- a/include/modules/i3.hpp
+++ b/include/modules/i3.hpp
@@ -4,6 +4,7 @@
 
 #include "components/config.hpp"
 #include "modules/meta/event_module.hpp"
+#include "modules/meta/types.hpp"
 #include "utils/i3.hpp"
 #include "utils/io.hpp"
 
@@ -51,7 +52,7 @@ namespace modules {
     bool update();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/i3";
+    static constexpr auto TYPE = I3_TYPE;
 
     static constexpr auto EVENT_FOCUS = "focus";
     static constexpr auto EVENT_NEXT = "next";
diff --git a/include/modules/ipc.hpp b/include/modules/ipc.hpp
index ce2f63a4..69c373af 100644
--- a/include/modules/ipc.hpp
+++ b/include/modules/ipc.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "utils/command.hpp"
 
 POLYBAR_NS
@@ -33,7 +34,7 @@ namespace modules {
     bool build(builder* builder, const string& tag) const;
     void on_message(const string& message);
 
-    static constexpr auto TYPE = "custom/ipc";
+    static constexpr auto TYPE = IPC_TYPE;
 
     static constexpr auto EVENT_SEND = "send";
     static constexpr auto EVENT_HOOK = "hook";
diff --git a/include/modules/memory.hpp b/include/modules/memory.hpp
index 90160b8e..6cca10d0 100644
--- a/include/modules/memory.hpp
+++ b/include/modules/memory.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -16,7 +17,7 @@ namespace modules {
     string get_format() const;
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/memory";
+    static constexpr auto TYPE = MEMORY_TYPE;
 
    private:
     static constexpr const char* TAG_LABEL{"<label>"};
diff --git a/include/modules/menu.hpp b/include/modules/menu.hpp
index 8255e2b5..8219e199 100644
--- a/include/modules/menu.hpp
+++ b/include/modules/menu.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -22,7 +23,7 @@ namespace modules {
     bool build(builder* builder, const string& tag) const;
     void update() {}
 
-    static constexpr auto TYPE = "custom/menu";
+    static constexpr auto TYPE = MENU_TYPE;
 
     static constexpr auto EVENT_OPEN = "open";
     static constexpr auto EVENT_CLOSE = "close";
diff --git a/include/modules/meta/all.hpp b/include/modules/meta/all.hpp
index 38e92b42..33ed4621 100644
--- a/include/modules/meta/all.hpp
+++ b/include/modules/meta/all.hpp
@@ -43,4 +43,3 @@
 #if ENABLE_XKEYBOARD
 #include "modules/xkeyboard.hpp"
 #endif
-#include "modules/unsupported.hpp"
diff --git a/include/modules/meta/types.hpp b/include/modules/meta/types.hpp
new file mode 100644
index 00000000..5c9009f8
--- /dev/null
+++ b/include/modules/meta/types.hpp
@@ -0,0 +1,37 @@
+#pragma once
+
+#include "common.hpp"
+
+POLYBAR_NS
+
+namespace modules {
+
+  static constexpr const char ALSA_TYPE[] = "internal/alsa";
+  static constexpr const char BACKLIGHT_TYPE[] = "internal/backlight";
+  static constexpr const char BATTERY_TYPE[] = "internal/battery";
+  static constexpr const char BSPWM_TYPE[] = "internal/bspwm";
+  static constexpr const char COUNTER_TYPE[] = "internal/counter";
+  static constexpr const char CPU_TYPE[] = "internal/cpu";
+  static constexpr const char DATE_TYPE[] = "internal/date";
+  static constexpr const char FS_TYPE[] = "internal/fs";
+  static constexpr const char GITHUB_TYPE[] = "internal/github";
+  static constexpr const char I3_TYPE[] = "internal/i3";
+  static constexpr const char MEMORY_TYPE[] = "internal/memory";
+  static constexpr const char MPD_TYPE[] = "internal/mpd";
+  static constexpr const char NETWORK_TYPE[] = "internal/network";
+  static constexpr const char PULSEAUDIO_TYPE[] = "internal/pulseaudio";
+  static constexpr const char TEMPERATURE_TYPE[] = "internal/temperature";
+  static constexpr const char TRAY_TYPE[] = "internal/tray";
+  static constexpr const char XBACKLIGHT_TYPE[] = "internal/xbacklight";
+  static constexpr const char XKEYBOARD_TYPE[] = "internal/xkeyboard";
+  static constexpr const char XWINDOW_TYPE[] = "internal/xwindow";
+  static constexpr const char XWORKSPACES_TYPE[] = "internal/xworkspaces";
+
+  static constexpr const char IPC_TYPE[] = "custom/ipc";
+  static constexpr const char MENU_TYPE[] = "custom/menu";
+  static constexpr const char SCRIPT_TYPE[] = "custom/script";
+  static constexpr const char TEXT_TYPE[] = "custom/text";
+
+} // namespace modules
+
+POLYBAR_NS_END
\ No newline at end of file
diff --git a/include/modules/mpd.hpp b/include/modules/mpd.hpp
index c11e0857..8f381b9b 100644
--- a/include/modules/mpd.hpp
+++ b/include/modules/mpd.hpp
@@ -4,6 +4,7 @@
 
 #include "adapters/mpd.hpp"
 #include "modules/meta/event_module.hpp"
+#include "modules/meta/types.hpp"
 #include "utils/env.hpp"
 
 POLYBAR_NS
@@ -24,7 +25,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/mpd";
+    static constexpr auto TYPE = MPD_TYPE;
 
     static constexpr const char* EVENT_PLAY = "play";
     static constexpr const char* EVENT_PAUSE = "pause";
diff --git a/include/modules/network.hpp b/include/modules/network.hpp
index c69e1559..bf56a196 100644
--- a/include/modules/network.hpp
+++ b/include/modules/network.hpp
@@ -3,6 +3,7 @@
 #include "adapters/net.hpp"
 #include "components/config.hpp"
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -18,7 +19,7 @@ namespace modules {
     string get_format() const;
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/network";
+    static constexpr auto TYPE = NETWORK_TYPE;
 
    protected:
     void subthread_routine();
diff --git a/include/modules/pulseaudio.hpp b/include/modules/pulseaudio.hpp
index 897f4d73..fdc73bde 100644
--- a/include/modules/pulseaudio.hpp
+++ b/include/modules/pulseaudio.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/event_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -22,7 +23,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/pulseaudio";
+    static constexpr auto TYPE = PULSEAUDIO_TYPE;
 
     static constexpr auto EVENT_INC = "inc";
     static constexpr auto EVENT_DEC = "dec";
diff --git a/include/modules/script.hpp b/include/modules/script.hpp
index 6f8e7859..dcd25ee7 100644
--- a/include/modules/script.hpp
+++ b/include/modules/script.hpp
@@ -2,6 +2,7 @@
 
 #include "adapters/script_runner.hpp"
 #include "modules/meta/base.hpp"
+#include "modules/meta/types.hpp"
 #include "utils/command.hpp"
 #include "utils/io.hpp"
 
@@ -20,7 +21,7 @@ namespace modules {
 
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "custom/script";
+    static constexpr auto TYPE = SCRIPT_TYPE;
 
    protected:
     bool check_condition();
diff --git a/include/modules/temperature.hpp b/include/modules/temperature.hpp
index c7fc3e4c..cd7039d6 100644
--- a/include/modules/temperature.hpp
+++ b/include/modules/temperature.hpp
@@ -3,6 +3,7 @@
 #include <istream>
 
 #include "modules/meta/timer_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 
 POLYBAR_NS
@@ -18,7 +19,7 @@ namespace modules {
     string get_format() const;
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/temperature";
+    static constexpr auto TYPE = TEMPERATURE_TYPE;
 
    private:
     static constexpr auto TAG_LABEL = "<label>";
diff --git a/include/modules/text.hpp b/include/modules/text.hpp
index fba16f63..163dc675 100644
--- a/include/modules/text.hpp
+++ b/include/modules/text.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 
 POLYBAR_NS
 
@@ -14,7 +15,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "custom/text";
+    static constexpr auto TYPE = TEXT_TYPE;
 
    private:
     static constexpr const char* TAG_LABEL{"<label>"};
diff --git a/include/modules/tray.hpp b/include/modules/tray.hpp
index 7cf60401..add0a845 100644
--- a/include/modules/tray.hpp
+++ b/include/modules/tray.hpp
@@ -3,6 +3,7 @@
 #include "common.hpp"
 #include "components/bar.hpp"
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "x11/tray_manager.hpp"
 
 POLYBAR_NS
@@ -17,7 +18,7 @@ class tray_module : public static_module<tray_module> {
   bool build(builder* builder, const string& tag) const;
   void update() {}
 
-  static constexpr auto TYPE = "internal/tray";
+  static constexpr auto TYPE = TRAY_TYPE;
 
  private:
   static constexpr const char* TAG_TRAY{"<tray>"};
diff --git a/include/modules/unsupported.hpp b/include/modules/unsupported.hpp
deleted file mode 100644
index 788b4f4f..00000000
--- a/include/modules/unsupported.hpp
+++ /dev/null
@@ -1,68 +0,0 @@
-#pragma once
-
-#include "modules/meta/base.hpp"
-#include "modules/meta/base.inl"
-
-POLYBAR_NS
-
-namespace modules {
-  struct module_interface;
-
-#define DEFINE_UNSUPPORTED_MODULE(MODULE_NAME, MODULE_TYPE)                             \
-  class MODULE_NAME : public module_interface {                                         \
-   public:                                                                              \
-    MODULE_NAME(const bar_settings, string, const config&) {                            \
-      throw application_error("No built-in support for '" + string{MODULE_TYPE} + "'"); \
-    }                                                                                   \
-    static constexpr auto TYPE = MODULE_TYPE;                                           \
-    string type() const override {                                                      \
-      return "";                                                                        \
-    }                                                                                   \
-    string name_raw() const override {                                                  \
-      return "";                                                                        \
-    }                                                                                   \
-    string name() const override {                                                      \
-      return "";                                                                        \
-    }                                                                                   \
-    bool running() const override {                                                     \
-      return false;                                                                     \
-    }                                                                                   \
-    bool visible() const override {                                                     \
-      return false;                                                                     \
-    }                                                                                   \
-    void start() override {}                                                            \
-    void stop() override {}                                                             \
-    void join() override {}                                                             \
-    void halt(string) override {}                                                       \
-    string contents() override {                                                        \
-      return "";                                                                        \
-    }                                                                                   \
-    bool input(const string&, const string&) override {                                 \
-      return false;                                                                     \
-    }                                                                                   \
-  }
-
-#if not ENABLE_I3
-  DEFINE_UNSUPPORTED_MODULE(i3_module, "internal/i3");
-#endif
-#if not ENABLE_MPD
-  DEFINE_UNSUPPORTED_MODULE(mpd_module, "internal/mpd");
-#endif
-#if not ENABLE_NETWORK
-  DEFINE_UNSUPPORTED_MODULE(network_module, "internal/network");
-#endif
-#if not ENABLE_ALSA
-  DEFINE_UNSUPPORTED_MODULE(alsa_module, "internal/alsa");
-#endif
-#if not ENABLE_PULSEAUDIO
-  DEFINE_UNSUPPORTED_MODULE(pulseaudio_module, "internal/pulseaudio");
-#endif
-#if not ENABLE_CURL
-  DEFINE_UNSUPPORTED_MODULE(github_module, "internal/github");
-#endif
-#if not ENABLE_XKEYBOARD
-  DEFINE_UNSUPPORTED_MODULE(xkeyboard_module, "internal/xkeyboard");
-#endif
-}  // namespace modules
-
-POLYBAR_NS_END
diff --git a/include/modules/xbacklight.hpp b/include/modules/xbacklight.hpp
index e18295bf..75d9ab3a 100644
--- a/include/modules/xbacklight.hpp
+++ b/include/modules/xbacklight.hpp
@@ -3,6 +3,7 @@
 #include "components/config.hpp"
 #include "modules/meta/event_handler.hpp"
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "settings.hpp"
 #include "x11/extensions/randr.hpp"
 
@@ -30,7 +31,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/xbacklight";
+    static constexpr auto TYPE = XBACKLIGHT_TYPE;
 
     static constexpr const char* EVENT_INC = "inc";
     static constexpr const char* EVENT_DEC = "dec";
diff --git a/include/modules/xkeyboard.hpp b/include/modules/xkeyboard.hpp
index 75b83083..d52fc7c7 100644
--- a/include/modules/xkeyboard.hpp
+++ b/include/modules/xkeyboard.hpp
@@ -6,6 +6,7 @@
 #include "drawtypes/layouticonset.hpp"
 #include "modules/meta/event_handler.hpp"
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "x11/extensions/xkb.hpp"
 #include "x11/window.hpp"
 
@@ -27,7 +28,7 @@ namespace modules {
     void update();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/xkeyboard";
+    static constexpr auto TYPE = XKEYBOARD_TYPE;
 
     static constexpr const char* EVENT_SWITCH = "switch";
 
diff --git a/include/modules/xwindow.hpp b/include/modules/xwindow.hpp
index 185ac611..5c3f3423 100644
--- a/include/modules/xwindow.hpp
+++ b/include/modules/xwindow.hpp
@@ -2,6 +2,7 @@
 
 #include "modules/meta/event_handler.hpp"
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "x11/ewmh.hpp"
 #include "x11/icccm.hpp"
 #include "x11/window.hpp"
@@ -38,7 +39,7 @@ namespace modules {
     void update();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/xwindow";
+    static constexpr auto TYPE = XWINDOW_TYPE;
 
    protected:
     void handle(const evt::property_notify& evt) override;
diff --git a/include/modules/xworkspaces.hpp b/include/modules/xworkspaces.hpp
index 605f9af6..578f5337 100644
--- a/include/modules/xworkspaces.hpp
+++ b/include/modules/xworkspaces.hpp
@@ -2,6 +2,7 @@
 
 #include "modules/meta/event_handler.hpp"
 #include "modules/meta/static_module.hpp"
+#include "modules/meta/types.hpp"
 #include "x11/ewmh.hpp"
 
 POLYBAR_NS
@@ -50,7 +51,7 @@ namespace modules {
     string get_output();
     bool build(builder* builder, const string& tag) const;
 
-    static constexpr auto TYPE = "internal/xworkspaces";
+    static constexpr auto TYPE = XWORKSPACES_TYPE;
 
     static constexpr auto EVENT_FOCUS = "focus";
     static constexpr auto EVENT_NEXT = "next";
diff --git a/src/components/controller.cpp b/src/components/controller.cpp
index 5548d3de..8fa78120 100644
--- a/src/components/controller.cpp
+++ b/src/components/controller.cpp
@@ -1,6 +1,7 @@
 #include "components/controller.hpp"
 
 #include <csignal>
+#include <cassert>
 #include <utility>
 
 #include "components/bar.hpp"
diff --git a/src/modules/meta/factory.cpp b/src/modules/meta/factory.cpp
index 203475fc..dddefe73 100644
--- a/src/modules/meta/factory.cpp
+++ b/src/modules/meta/factory.cpp
@@ -36,6 +36,16 @@ namespace modules {
     return std::make_pair(std::string(M::TYPE), get_factory<M>());
   }
 
+  template<const char* module_type>
+  static factory_map::value_type map_entry_unsupported() {
+    return {
+      module_type, 
+      [](const bar_settings&, string&&, const config&) -> module_t {
+        throw application_error("No built-in support for '" + string(module_type) + "'");
+      }
+    };
+  }
+
   /**
    * Factory function for each module type.
    */
@@ -46,17 +56,45 @@ namespace modules {
       map_entry<bspwm_module>(),
       map_entry<cpu_module>(),
       map_entry<date_module>(),
+#if ENABLE_CURL
       map_entry<github_module>(),
+#else
+      map_entry_unsupported<GITHUB_TYPE>(),
+#endif
       map_entry<fs_module>(),
       map_entry<memory_module>(),
+#if ENABLE_I3
       map_entry<i3_module>(),
+#else
+      map_entry_unsupported<I3_TYPE>(),
+#endif
+#if ENABLE_MPD
       map_entry<mpd_module>(),
+#else
+      map_entry_unsupported<MPD_TYPE>(),
+#endif
+#if ENABLE_ALSA
       map_entry<alsa_module>(),
+#else
+      map_entry_unsupported<ALSA_TYPE>(),
+#endif
+#if ENABLE_PULSEAUDIO
       map_entry<pulseaudio_module>(),
+#else
+      map_entry_unsupported<PULSEAUDIO_TYPE>(),
+#endif
+#if ENABLE_NETWORK
       map_entry<network_module>(),
+#else
+      map_entry_unsupported<NETWORK_TYPE>(),
+#endif
       map_entry<temperature_module>(),
       map_entry<xbacklight_module>(),
+#if ENABLE_XKEYBOARD
       map_entry<xkeyboard_module>(),
+#else
+      map_entry_unsupported<XKEYBOARD_TYPE>(),
+#endif
       map_entry<xwindow_module>(),
       map_entry<xworkspaces_module>(),
       map_entry<tray_module>(),
@@ -70,8 +108,8 @@ namespace modules {
     string actual_type = type;
 
     if (type == "internal/volume") {
-      log.warn("internal/volume is deprecated, use %s instead", string(alsa_module::TYPE));
-      actual_type = alsa_module::TYPE;
+      log.warn("internal/volume is deprecated, use %s instead", ALSA_TYPE);
+      actual_type = ALSA_TYPE;
     }
 
     auto it = factories.find(actual_type);