cmake conversion #1

Samengevoegd
SlavekB heeft 1 commits samengevoegd van feat/cmakeConv naar master 5 jaren geleden
Er is nog geen inhoud.
SlavekB beoordeeld 5 jaren geleden
SlavekB heeft een reactie achtergelaten
Eigenaar

The code looks good, but the search sword includes and the library following the pkg_search_module seems 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.

The code looks good, but the search sword includes and the library following the `pkg_search_module` seems 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} )
Eigenaar

Why there is a separate search for LIBSWORD_INCLUDE_DIR and LIBSWORD_LIBRARY when LIBSWORD_INCLUDE_DIRS and LIBSWORD_LIBRARIES are already set by

pkg_search_module (LIBSWORD sword)

?

Why there is a separate search for `LIBSWORD_INCLUDE_DIR` and `LIBSWORD_LIBRARY` when `LIBSWORD_INCLUDE_DIRS` and `LIBSWORD_LIBRARIES` are already set by ``` pkg_search_module (LIBSWORD sword) ``` ?
${CMAKE_CURRENT_SOURCE_DIR}
${TDE_INCLUDE_DIR}
${TQT_INCLUDE_DIRS}
${LIBSWORD_INCLUDE_DIR}
Eigenaar

Here you can use LIBSWORD_INCLUDE_DIRS, which is defined by pkg_search_module(…)

Here you can use `LIBSWORD_INCLUDE_DIRS`, which is defined by `pkg_search_module(…)`
LINK
tdecore-shared
tdeio-shared
${LIBSWORD_LIBRARY}
Eigenaar

Here you can use LIBSWORD_LIBRARIES, which is defined by pkg_search_module(…)

Here you can use `LIBSWORD_LIBRARIES`, which is defined by `pkg_search_module(…)`
SlavekB voegde het PR/rfc label 5 jaren geleden toe
SlavekB heeft deze veranderingen 5 jaren geleden goedgekeurd
SlavekB heeft een reactie achtergelaten
Eigenaar

After the final edits, the code is ready. Thank you for your efforts.

After the final edits, the code is ready. Thank you for your efforts.
SlavekB verwijderde het PR/rfc label 5 jaren geleden
SlavekB heeft deze pull request gesloten 5 jaren geleden
SlavekB heeft 5 jaren geleden de branch feat/cmakeConv verwijderd.
SlavekB heeft dit 5 jaren geleden aan de mijlpaal R14.0.6 release toegevoegd
De pull request is samengevoegd als 8a9f67e94a.
Log in om deel te nemen aan deze discussie.
Geen beoordelaars
Geen mijlpaal
Niet toegewezen
2 deelnemers
Notificaties
Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Referentie: TDE/tdeio-sword#1
Laden…
Er is nog geen inhoud.