Skip to content

Commit 4e10afa

Browse files
committed
Use addon factory v2
1 parent 254eae8 commit 4e10afa

File tree

4 files changed

+7
-3
lines changed

4 files changed

+7
-3
lines changed

CMakeLists.txt

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ cmake_minimum_required(VERSION 3.6)
22

33
project(fcitx-m17n VERSION 5.1.2)
44

5+
set(REQUIRED_FCITX_VERSION 5.1.12)
56
find_package(ECM REQUIRED 1.0.0)
67
set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH} "${CMAKE_CURRENT_SOURCE_DIR}/cmake" ${CMAKE_MODULE_PATH})
78
include(FeatureSummary)
@@ -12,7 +13,7 @@ option(ENABLE_TEST "Build Test" On)
1213
option(ENABLE_COVERAGE "Build the project with gcov support (Need ENABLE_TEST=On)" Off)
1314
set(GCOV_TOOL "gcov" CACHE STRING "Path to gcov tool used by coverage.")
1415

15-
find_package(Fcitx5Core 5.0.2 REQUIRED)
16+
find_package(Fcitx5Core ${REQUIRED_FCITX_VERSION} REQUIRED)
1617
find_package(Fcitx5Module REQUIRED COMPONENTS TestFrontend)
1718
find_package(Gettext REQUIRED)
1819
find_package(fmt REQUIRED)

im/CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ set(fcitx_m17n_sources
44
keysymname.cpp
55
)
66

7-
add_library(m17n MODULE ${fcitx_m17n_sources})
7+
add_fcitx5_addon(m17n ${fcitx_m17n_sources})
88
target_link_libraries(m17n Fcitx5::Core Fcitx5::Config ${FMT_TARGET} PkgConfig::M17NGui)
99
target_include_directories(m17n PRIVATE ${PROJECT_BINARY_DIR})
1010
set_target_properties(m17n PROPERTIES PREFIX "")

im/engine.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -624,4 +624,4 @@ void M17NState::select(int index) {
624624
}
625625
} // namespace fcitx
626626

627-
FCITX_ADDON_FACTORY(fcitx::M17NEngineFactory)
627+
FCITX_ADDON_FACTORY_V2(m17n, fcitx::M17NEngineFactory)

im/m17n.conf.in.in

+3
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,6 @@ Enabled=True
77
Library=m17n
88
Type=SharedLibrary
99
Configurable=True
10+
11+
[Addon/Dependencies]
12+
0=core:@REQUIRED_FCITX_VERSION@

0 commit comments

Comments
 (0)