Fix support for using system supplied md4c library
Add FindWrapSystemMd4c.cmake so that the old md4c target can be used as well as the new one and set WrapSystemMd4c_FOUND. Link to the imported target WrapSystemMd4c::WrapSystemMd4c if the system library is used. Add qt_find_package line to find the package in configure.cmake. Fix the condition for enabling system-textmarkdownreader, it includes testing for textmarkdownreader because even if the code would compile correctly without it, it looks strange when the output says "textmarkdownreader no" and under "using system libmd4c yes" even if libmd4c is not used. Use system include when system-markdownreader is enabled. Add library mapping for libmd4c. Change-Id: Id5d5b13d6691a8c1cdf627238887977c847c1e67 Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io> Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
This commit is contained in:
parent
5cc3105807
commit
7920c03ff1
24
cmake/FindWrapSystemMd4c.cmake
Normal file
24
cmake/FindWrapSystemMd4c.cmake
Normal file
@ -0,0 +1,24 @@
|
||||
if(TARGET WrapSystemMd4c::WrapSystemMd4c)
|
||||
set(WrapSystemMd4c_FOUND TRUE)
|
||||
return()
|
||||
endif()
|
||||
|
||||
find_package(md4c CONFIG)
|
||||
|
||||
# md4c provides a md4c::md4c target but
|
||||
# older versions create a md4c target without
|
||||
# namespace. If we find the old variant create
|
||||
# a namespaced target out of the md4c target.
|
||||
if(TARGET md4c AND NOT TARGET md4c::md4c)
|
||||
add_library(md4c::md4c INTERFACE IMPORTED)
|
||||
target_link_libraries(md4c::md4c INTERFACE md4c)
|
||||
endif()
|
||||
|
||||
if(TARGET md4c::md4c)
|
||||
add_library(WrapSystemMd4c::WrapSystemMd4c INTERFACE IMPORTED)
|
||||
target_link_libraries(WrapSystemMd4c::WrapSystemMd4c INTERFACE md4c::md4c)
|
||||
endif()
|
||||
|
||||
if(TARGET WrapSystemMd4c::WrapSystemMd4c)
|
||||
set(WrapSystemMd4c_FOUND TRUE)
|
||||
endif()
|
@ -682,7 +682,7 @@ qt_internal_extend_target(Gui CONDITION QT_FEATURE_textmarkdownreader
|
||||
|
||||
qt_internal_extend_target(Gui CONDITION QT_FEATURE_system_textmarkdownreader AND QT_FEATURE_textmarkdownreader
|
||||
LIBRARIES
|
||||
libmd4c
|
||||
WrapSystemMd4c::WrapSystemMd4c
|
||||
)
|
||||
|
||||
qt_internal_extend_target(Gui CONDITION QT_FEATURE_textmarkdownreader AND NOT QT_FEATURE_system_textmarkdownreader
|
||||
|
@ -41,6 +41,7 @@ qt_find_package(gbm PROVIDED_TARGETS gbm::gbm MODULE_NAME gui QMAKE_LIB gbm)
|
||||
qt_find_package(WrapSystemHarfbuzz 2.6.0 PROVIDED_TARGETS WrapSystemHarfbuzz::WrapSystemHarfbuzz MODULE_NAME gui QMAKE_LIB harfbuzz)
|
||||
qt_find_package(Libinput PROVIDED_TARGETS Libinput::Libinput MODULE_NAME gui QMAKE_LIB libinput)
|
||||
qt_find_package(JPEG PROVIDED_TARGETS JPEG::JPEG MODULE_NAME gui QMAKE_LIB libjpeg)
|
||||
qt_find_package(WrapSystemMd4c PROVIDED_TARGETS WrapSystemMd4c::WrapSystemMd4c MODULE_NAME gui QMAKE_LIB libmd4c)
|
||||
qt_find_package(WrapSystemPNG PROVIDED_TARGETS WrapSystemPNG::WrapSystemPNG MODULE_NAME gui QMAKE_LIB libpng)
|
||||
if(QT_FEATURE_system_zlib)
|
||||
qt_add_qmake_lib_dependency(libpng zlib)
|
||||
@ -945,7 +946,7 @@ qt_feature("textmarkdownreader" PUBLIC
|
||||
qt_feature("system-textmarkdownreader" PUBLIC
|
||||
SECTION "Kernel"
|
||||
LABEL " Using system libmd4c"
|
||||
CONDITION libs.libmd4c OR FIXME
|
||||
CONDITION QT_FEATURE_textmarkdownreader AND WrapSystemMd4c_FOUND
|
||||
ENABLE INPUT_libmd4c STREQUAL 'system'
|
||||
DISABLE INPUT_libmd4c STREQUAL 'qt'
|
||||
)
|
||||
|
@ -1452,7 +1452,7 @@
|
||||
"disable": "input.libmd4c == 'qt'",
|
||||
"enable": "input.libmd4c == 'system'",
|
||||
"section": "Kernel",
|
||||
"condition": "libs.libmd4c",
|
||||
"condition": "features.textmarkdownreader && libs.libmd4c",
|
||||
"output": [ "publicFeature" ]
|
||||
},
|
||||
"textmarkdownwriter": {
|
||||
|
@ -48,7 +48,11 @@
|
||||
#include <QTextDocumentFragment>
|
||||
#include <QTextList>
|
||||
#include <QTextTable>
|
||||
#if QT_CONFIG(system_textmarkdownreader)
|
||||
#include <md4c.h>
|
||||
#else
|
||||
#include "../../3rdparty/md4c/md4c.h"
|
||||
#endif
|
||||
|
||||
QT_BEGIN_NAMESPACE
|
||||
|
||||
|
@ -457,6 +457,7 @@ _library_map = [
|
||||
LibraryMapping("librt", "WrapRt", "WrapRt::WrapRt"),
|
||||
LibraryMapping("libudev", "Libudev", "PkgConfig::Libudev"),
|
||||
LibraryMapping("lttng-ust", "LTTngUST", "LTTng::UST", resultVariable="LTTNGUST"),
|
||||
LibraryMapping("libmd4c", "WrapMd4c", "WrapMd4c::WrapMd4c", is_bundled_with_qt=True),
|
||||
LibraryMapping("mtdev", "Mtdev", "PkgConfig::Mtdev"),
|
||||
LibraryMapping("mysql", "MySQL", "MySQL::MySQL"),
|
||||
LibraryMapping("odbc", "ODBC", "ODBC::ODBC"),
|
||||
|
Loading…
Reference in New Issue
Block a user