use FCITX_DEFINE_STATIC_ADDON_REGISTRY (#214)

CMakeLists.txt

@@ -90,16 +90,17 @@ add_compile_definitions($<$<COMPILE_LANGUAGE:C,CXX>:-DFCITX_GETTEXT_DOMAIN=\"fci
 add_subdirectory(keycode)
 add_subdirectory(macosfrontend)
 add_subdirectory(macosnotifications)
-add_subdirectory(webpanel)
-
-add_subdirectory(src)
-add_subdirectory(assets)
 
 option(BUILD_PREVIEW "" OFF)
 add_subdirectory(fcitx5-webview)
+add_subdirectory(webpanel)
 
-set(ADDON_TYPE "StaticLibrary")
+set(BUILD_SHARED_FCITX_ADDON OFF)
+include(fcitx5/src/lib/fcitx-utils/Fcitx5Macros.cmake)
 add_subdirectory(fcitx5-beast/src)
 
+add_subdirectory(src)
+add_subdirectory(assets)
+
 enable_testing()
 add_subdirectory(tests)

keycode/CMakeLists.txt

@@ -1,3 +1,3 @@
-add_library(Keycode keycode.cpp)
+add_library(Keycode STATIC keycode.cpp)
 target_include_directories(Keycode PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}")
 target_link_libraries(Keycode Fcitx5::Utils)

logging/CMakeLists.txt

@@ -5,4 +5,4 @@ else()
 endif()
 configure_file(debug.swift.in ${CMAKE_CURRENT_SOURCE_DIR}/debug.swift @ONLY)
 
-add_library(Logging logging.swift debug.swift)
+add_library(Logging STATIC logging.swift debug.swift)

macosnotifications/CMakeLists.txt

@@ -12,10 +12,10 @@ _swift_generate_cxx_header(
     DEPENDS Logging
 )
 
-add_library(macosnotifications STATIC macosnotifications.cpp)
-add_dependencies(macosnotifications SwiftNotify)
-target_link_libraries(macosnotifications Fcitx5::Core Fcitx5::Module::Notifications)
-target_include_directories(macosnotifications PUBLIC
+add_library(notifications STATIC macosnotifications.cpp)
+add_dependencies(notifications SwiftNotify)
+target_link_libraries(notifications Fcitx5::Core Fcitx5::Module::Notifications)
+target_include_directories(notifications PUBLIC
   "${CMAKE_CURRENT_BINARY_DIR}/include"
   "${CMAKE_SOURCE_DIR}/src"
 )

scripts/check-validity.sh

@@ -2,6 +2,7 @@ set -e
 
 has_homebrew_deps=0
 has_xcode_rpath=0
+has_extra_dylib=0
 
 cd /Library/Input\ Methods/Fcitx5.app/Contents
 libs=(MacOS/Fcitx5)
@@ -18,6 +19,10 @@ for lib in "${libs[@]}"; do
     otool -l $lib | grep -A2 LC_RPATH
     has_xcode_rpath=2
   fi
+  n_dylib=$(otool -L MacOS/Fcitx5 | grep rpath | wc -l | xargs)
+  if [[ $n_dylib != 3 ]]; then
+    has_extra_dylib=4
+  fi
 done
 
-exit $((has_homebrew_deps + has_xcode_rpath))
+exit $((has_homebrew_deps + has_xcode_rpath + has_extra_dylib))

src/CMakeLists.txt

@@ -25,7 +25,7 @@ target_link_libraries(Fcitx5Objs
     webpanel
     beast
     macosfrontend
-    macosnotifications
+    notifications
     WebviewCandidateWindow
     Logging
 )
@@ -46,6 +46,11 @@ target_link_libraries(Fcitx5
     SwiftNotify
 )
 
+fcitx5_import_addons(Fcitx5
+    REGISTRY_VARNAME getStaticAddon
+    ADDONS beast keyboard webpanel macosfrontend notifications
+)
+
 set(APP_PATH "${CMAKE_CURRENT_BINARY_DIR}/Fcitx5.app/Contents")
 
 set(BINARY_LIB_PATH "${PROJECT_BINARY_DIR}/fcitx5/src/lib")

src/fcitx.cpp

@@ -25,14 +25,7 @@ namespace fs = std::filesystem;
 
 #define APP_CONTENTS_PATH "/Library/Input Methods/Fcitx5.app/Contents"
 
-fcitx::BeastFactory beastFactory;
-fcitx::StaticAddonRegistry staticAddons = {
-    std::make_pair<std::string, fcitx::AddonFactory *>("beast", &beastFactory)};
-
-FCITX_IMPORT_ADDON_FACTORY(staticAddons, keyboard);
-FCITX_IMPORT_ADDON_FACTORY(staticAddons, webpanel);
-FCITX_IMPORT_ADDON_FACTORY(staticAddons, macosfrontend);
-FCITX_IMPORT_ADDON_FACTORY(staticAddons, notifications);
+FCITX_DEFINE_STATIC_ADDON_REGISTRY(getStaticAddon)
 
 static std::string join_paths(const std::vector<fs::path> &paths,
                               char sep = ':');
@@ -139,7 +132,7 @@ void Fcitx::setupInstance() {
     instance_ = std::make_unique<fcitx::Instance>(0, nullptr);
     dispatcher_ = std::make_unique<fcitx::EventDispatcher>();
     auto &addonMgr = instance_->addonManager();
-    addonMgr.registerDefaultLoader(&staticAddons);
+    addonMgr.registerDefaultLoader(&getStaticAddon());
     instance_->initialize();
     dispatcher_->attach(&instance_->eventLoop());
 }