Fix GCC visibility, use common tests in CMake rules #1

Sammanfogat
SlavekB sammanfogade 1 incheckningar från feat/cmakeCommonTests in i master 5 år sedan
SlavekB kommenterad 5 år sedan
Ägare

In Akode I added the usual option WITH_GCC_VISIBILITY and the new common test tde_setup_gcc_visibility. The previous code only set __KDE_HAVE_GCC_VISIBILITY but did not set the compiler options at all. GCC visibility has therefore not been used properly. That's why I checked and added the use of AKODE_EXPORT to the exported functions in each plug-in.

Please test it.

Note: For test you need to build with WITH_ALL_OPTIONS or WITH_GCC_VISIBILITY.

In Akode I added the usual option WITH_GCC_VISIBILITY and the new common test tde_setup_gcc_visibility. The previous code only set __KDE_HAVE_GCC_VISIBILITY but did not set the compiler options at all. GCC visibility has therefore not been used properly. That's why I checked and added the use of AKODE_EXPORT to the exported functions in each plug-in. Please test it. Note: For test you need to build with WITH_ALL_OPTIONS or WITH_GCC_VISIBILITY.
SlavekB closed this pull request 5 år sedan
SlavekB tog bort grenen feat/cmakeCommonTests 5 år sedan
SlavekB lade till denna till milstolpe R14.0.6 release 5 år sedan
Pull-förfrågan har sammanfogats som 68b71ec34c.
Logga in för att delta i denna konversation.
Inga granskare
Ingen Milsten
Ingen tilldelad
1 Deltagare
Notiser
Förfallodatum

Inget förfallodatum satt.

Beroenden

No dependencies set.

Reference: TDE/akode#1
Laddar…
Det finns inget innehåll än.