cmake conversion #1

Слито
SlavekB слито 1 коммит(ов) из feat/cmakeConv в master 5 лет назад
Ghost прокомментировал(а) 5 лет назад
Пока нет содержимого.
SlavekB рассмотрел(а) изменения 5 лет назад
SlavekB оставил комментарий
Владелец

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} )
SlavekB прокомментировал(а) 5 лет назад
Владелец

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}
SlavekB прокомментировал(а) 5 лет назад
Владелец

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}
SlavekB прокомментировал(а) 5 лет назад
Владелец

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 добавлен PR/rfc с меткой 5 лет назад
SlavekB одобрил(а) эти изменения 5 лет назад
SlavekB оставил комментарий
Владелец

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 удалён PR/rfc с меткой 5 лет назад
SlavekB закрыл этот запрос на слияние 5 лет назад
SlavekB удалил(а) ветку feat/cmakeConv 5 лет назад
SlavekB добавил(а) к этапу R14.0.6 release 5 лет назад
Запрос на слияние был объединен как 8a9f67e94a.
Войдите, чтобы присоединиться к обсуждению.
Нет рецензентов
Нет этапа
Нет назначенных лиц
2 участников
Уведомления
Срок выполнения

Срок выполнения не установлен.

Зависимости

Зависимостей нет.

Reference: TDE/tdeio-sword#1
Загрузка…
Пока нет содержимого.