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

Merged
SlavekB merged 2 commits from feat/cmakeCommonTests into master 3 months ago
SlavekB commented 3 months ago

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 deleted branch feat/cmakeCommonTests 3 months ago
SlavekB added this to the R14.0.6 release milestone 3 months ago
The pull request has been merged.
Sign in to join this conversation.
Loading…
Cancel
Save
There is no content yet.