cmake conversion #1
Zusammengeführt
SlavekB
hat 1 Commits von feat/cmakeConv nach master vor 5 Jahren zusammengeführt
Keine Reviewer
Kein Label
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: TDE/tdeio-sword#1
Laden…
In neuem Issue referenzieren
Hier gibt es bis jetzt noch keinen Inhalt.
Branch „feat/cmakeConv“ löschen
Das Löschen eines Branches ist permanent. Es KANN NICHT rückgängig gemacht werden. Fortfahren?
The code looks good, but the search sword includes and the library following the
pkg_search_moduleseems unnecessary.I tested the code without this search and with the modifications mentioned below in the comments and everything was fine. Please test on your machine and make the adjustments according to the result.
)set( LIBSWORD_LIBRARIES ${LIBSWORD_LIBRARY} )set( LIBSWORD_INCLUDE_DIRS ${LIBSWORD_INCLUDE_DIR} )Why there is a separate search for
LIBSWORD_INCLUDE_DIRandLIBSWORD_LIBRARYwhenLIBSWORD_INCLUDE_DIRSandLIBSWORD_LIBRARIESare already set by?
${CMAKE_CURRENT_SOURCE_DIR}${TDE_INCLUDE_DIR}${TQT_INCLUDE_DIRS}${LIBSWORD_INCLUDE_DIR}Here you can use
LIBSWORD_INCLUDE_DIRS, which is defined bypkg_search_module(…)LINKtdecore-sharedtdeio-shared${LIBSWORD_LIBRARY}Here you can use
LIBSWORD_LIBRARIES, which is defined bypkg_search_module(…)After the final edits, the code is ready. Thank you for your efforts.
8a9f67e94agemergt.