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

Fusionado
SlavekB fusionados 1 commits de feat/cmakeCommonTests en master hace 5 años
Propietario

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 cerró este pull request hace 5 años
SlavekB rama eliminada feat/cmakeCommonTests hace 5 años
SlavekB ha añadido esto al hito R14.0.6 release hace 5 años
El Pull Request se ha fusionado como 68b71ec34c.
Inicie sesión para unirse a esta conversación.
No hay revisores
Sin Milestone
No asignados
1 participantes
Notificaciones
Fecha de vencimiento

Sin fecha de vencimiento.

Dependencias

No se han establecido dependencias.

Referencia: TDE/akode#1
Cargando…
Aún no existe contenido.