diff options
author | Alexandru Croitor <alexandru.croitor@qt.io> | 2021-08-19 11:42:12 +0200 |
---|---|---|
committer | Alexandru Croitor <alexandru.croitor@qt.io> | 2021-08-24 16:10:31 +0200 |
commit | 3c1125d9fe41332087ab6213a5514426109ead08 (patch) | |
tree | 5fe8b70215d3b6caad6e6b6cfa30eb67bc607c43 /examples/widgets/tools/plugandpaint/plugins | |
parent | 511bbbdfc0c048936336e0e7923cfd002ea05d38 (diff) |
CMake: Create plugin initializers for static user plugins
Previously we only created object library static plugin initializers
for Qt plugins only, not user-project plugins.
The reason was that if a user tried to install the plugin target via
an export set, CMake would error out saying that the _init library is
not part of the same export set.
Introduce an OUTPUT_TARGETS option that would allow projects to get
the name of the generated _init target, so they can install it if
needed.
This was already done for qt6_add_qml_module, so we just introduce the
same option for qt6_add_plugin.
Now user static plugins will have an _init target created, which will
be propagated to consumers whenever the consumers link against the
plugin itself.
We also need an internal option to disable this propagation, because
it's handled a bit differently for Qt plugins which can be linked
either via finalizers or via usage requirements.
Amends 91c65dd80cdd2de666448c14202c0c63718152b6
As a result of the implementation change, cleanup example projects
to ensure that they build successfully (the important part is
specifying the CLASS_NAME).
Only plugandpaint works properly with both shared and static Qt
builds.
echoplugin works with a shared Qt build, but not a static one due to
some assumptions in the C++ code about shared plugins.
styleplugin doesn't seem to work properly neither with shared Qt
builds nor static Qt builds, at least on macOS. But it builds fine.
For some reason even if the plugin is found, the style is not applied.
Amends 4caac1feea025b0ad496141e8f16ab88c04c2caa
Pick-to: 6.2
Task-number: QTBUG-80863
Task-number: QTBUG-92933
Change-Id: I6f631cda9566229b7a63992b23d7d7fa50303eeb
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Alexey Edelev <alexey.edelev@qt.io>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
Diffstat (limited to 'examples/widgets/tools/plugandpaint/plugins')
3 files changed, 9 insertions, 17 deletions
diff --git a/examples/widgets/tools/plugandpaint/plugins/CMakeLists.txt b/examples/widgets/tools/plugandpaint/plugins/CMakeLists.txt index 1e63da97ca..9a2d66d99e 100644 --- a/examples/widgets/tools/plugandpaint/plugins/CMakeLists.txt +++ b/examples/widgets/tools/plugandpaint/plugins/CMakeLists.txt @@ -1,4 +1,2 @@ -# Generated from plugins.pro. - add_subdirectory(basictools) add_subdirectory(extrafilters) diff --git a/examples/widgets/tools/plugandpaint/plugins/basictools/CMakeLists.txt b/examples/widgets/tools/plugandpaint/plugins/basictools/CMakeLists.txt index c66f83ce62..0056205319 100644 --- a/examples/widgets/tools/plugandpaint/plugins/basictools/CMakeLists.txt +++ b/examples/widgets/tools/plugandpaint/plugins/basictools/CMakeLists.txt @@ -1,19 +1,15 @@ -# Generated from basictools.pro. -# special case skip regeneration - -qt_add_plugin(pnp_basictools STATIC) +qt_add_plugin(pnp_basictools + STATIC + CLASS_NAME BasicToolsPlugin +) target_sources(pnp_basictools PRIVATE basictoolsplugin.cpp basictoolsplugin.h ) -set_target_properties(pnp_basictools PROPERTIES - WIN32_EXECUTABLE TRUE - MACOSX_BUNDLE TRUE -) target_include_directories(pnp_basictools PUBLIC ../../app ) -target_link_libraries(pnp_basictools PUBLIC +target_link_libraries(pnp_basictools PRIVATE Qt::Core Qt::Gui Qt::Widgets diff --git a/examples/widgets/tools/plugandpaint/plugins/extrafilters/CMakeLists.txt b/examples/widgets/tools/plugandpaint/plugins/extrafilters/CMakeLists.txt index 9227dd0537..2332f84399 100644 --- a/examples/widgets/tools/plugandpaint/plugins/extrafilters/CMakeLists.txt +++ b/examples/widgets/tools/plugandpaint/plugins/extrafilters/CMakeLists.txt @@ -1,19 +1,17 @@ -# Generated from extrafilters.pro. -# special case skip regeneration - -qt_add_plugin(pnp_extrafilters) +qt_add_plugin(pnp_extrafilters + CLASS_NAME ExtraFiltersPlugin +) target_sources(pnp_extrafilters PRIVATE extrafiltersplugin.cpp extrafiltersplugin.h ) set_target_properties(pnp_extrafilters PROPERTIES - MACOSX_BUNDLE TRUE LIBRARY_OUTPUT_DIRECTORY "${PROJECT_BINARY_DIR}/app" ) target_include_directories(pnp_extrafilters PUBLIC ../../app ) -target_link_libraries(pnp_extrafilters PUBLIC +target_link_libraries(pnp_extrafilters PRIVATE Qt::Core Qt::Gui Qt::Widgets |