Avoid potential FTBFS in tde_setup_gcc_visibility #24

Злито
SlavekB злито 1 комітів з feat/setup-gcc-visibility-kdemacros до master 5 роки тому
SlavekB прокоментував(ла) 5 роки тому
Власник
  1. If a user builds in an environment that is not clean, while building dependencies that do not need tdelibs, the previously installed tdelibs can be searched for and gcc visibility tested. In that case, kdemacros.h that is not on CMAKE_INCLUDE_DIRS, can lead to a test failure. Therefore, the full path to the detected kdemacros.h is now used.

  2. If a user builds in an environment that is not clean, and the module that is currently being built does not use TDE and / or TQt but these are found on the disk, they are tested on GCC visibility, which is unwanted and misleading. Therefore, tests are now conditioned by whether the module has already detected TDE and / or TQt.

1. If a user builds in an environment that is not clean, while building dependencies that do not need tdelibs, the previously installed tdelibs can be searched for and gcc visibility tested. In that case, kdemacros.h that is not on CMAKE_INCLUDE_DIRS, can lead to a test failure. Therefore, the full path to the detected kdemacros.h is now used. 2. If a user builds in an environment that is not clean, and the module that is currently being built does not use TDE and / or TQt but these are found on the disk, they are tested on GCC visibility, which is unwanted and misleading. Therefore, tests are now conditioned by whether the module has already detected TDE and / or TQt.
SlavekB додав(ла) до R14.0.6 release етапу 5 роки тому
MicheleC зміни затверджено 5 роки тому
MicheleC додав коментар
Власник

LGTM

LGTM
SlavekB закрив цей запит на злиття 5 роки тому
SlavekB видалена гілка feat/setup-gcc-visibility-kdemacros 5 роки тому
Запит на злиття був влитиий як f8254e1561.
Підпишіться щоб приєднатися до обговорення.
Немає рецензентів
Етап відсутній
Немає виконавця
2 учасників
Сповіщення
Дата завершення

Термін виконання не встановлений.

Залежності

No dependencies set.

Reference: TDE/tde-cmake#24
Завантаження…
Тут ще немає жодного змісту.