From bb9c07d8fe8b90c42ba81fb5bb6f8a9826252660 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niklas=20Hamb=C3=BCchen?= Date: Mon, 11 Nov 2019 16:37:46 +0100 Subject: [PATCH 1/2] build: Don't hardcode shared_library() in meson. Instead, use the standard library(). This is a meson best practice. Fixes -Ddefault_library=static not having any effect. --- gtk/meson.build | 18 +++++++++--------- testsuite/reftests/meson.build | 4 ++-- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/gtk/meson.build b/gtk/meson.build index 65576f92c6..4cdad1feb5 100644 --- a/gtk/meson.build +++ b/gtk/meson.build @@ -1007,15 +1007,15 @@ if cc.has_header('langinfo.h') endif # Library -libgtk = shared_library('gtk-4', - soversion: gtk_soversion, - sources: [typefuncs, gtk_sources, gtkmarshal_h, gtkprivatetypebuiltins_h], - c_args: gtk_cargs + common_cflags, - include_directories: [confinc, gdkinc, gskinc, gtkinc], - dependencies: gtk_deps + [libgtk_css_dep, libgdk_dep, libgsk_dep], - link_with: [libgtk_css, libgdk, libgsk, ], - link_args: common_ldflags, - install: true) +libgtk = library('gtk-4', + soversion: gtk_soversion, + sources: [typefuncs, gtk_sources, gtkmarshal_h, gtkprivatetypebuiltins_h], + c_args: gtk_cargs + common_cflags, + include_directories: [confinc, gdkinc, gskinc, gtkinc], + dependencies: gtk_deps + [libgtk_css_dep, libgdk_dep, libgsk_dep], + link_with: [libgtk_css, libgdk, libgsk, ], + link_args: common_ldflags, + install: true) gtk_dep_sources = [gtkversion, gtktypebuiltins_h] # Introspection diff --git a/testsuite/reftests/meson.build b/testsuite/reftests/meson.build index 632e97ffeb..cf6042ab1a 100644 --- a/testsuite/reftests/meson.build +++ b/testsuite/reftests/meson.build @@ -1,9 +1,9 @@ # FIXME: $(no_undefined) ? -libgtkreftestprivate = shared_library('gtkreftestprivate', +libgtkreftestprivate = library('gtkreftestprivate', 'reftest-compare.c', 'reftest-module.c', 'reftest-snapshot.c', dependencies : libgtk_dep) -libreftest = shared_library('reftest', +libreftest = library('reftest', [ 'expand-expander.c', 'frame-inhibitor.c', 'letter-spacing.c', From 3936412332bb5953a8cb535a80fa76306939aead Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niklas=20Hamb=C3=BCchen?= Date: Mon, 11 Nov 2019 20:11:16 +0100 Subject: [PATCH 2/2] Try building static libs on CI --- .gitlab-ci.yml | 7 ++++++- .gitlab-ci/test-docker.sh | 1 + 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 630497088b..453ca1c100 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -12,7 +12,7 @@ stages: - subprojects/libepoxy/ - subprojects/pango/ -fedora-x86_64: +fedora-x86_64: &fedora-x86_64-defaults image: registry.gitlab.gnome.org/gnome/gtk/master:v7 stage: build script: @@ -33,6 +33,11 @@ fedora-x86_64: key: "$CI_JOB_NAME" <<: *cache-paths +fedora-x86_64-staticlibs: + variables: + EXTRA_MESON_FLAGS: "-Ddefault_library=both" + <<: *fedora-x86_64-defaults + .mingw-defaults: &mingw-defaults stage: build tags: diff --git a/.gitlab-ci/test-docker.sh b/.gitlab-ci/test-docker.sh index 78df6f74bc..6b5a7301a1 100755 --- a/.gitlab-ci/test-docker.sh +++ b/.gitlab-ci/test-docker.sh @@ -17,6 +17,7 @@ meson \ -Dbroadway-backend=true \ -Dvulkan=yes \ --werror \ + ${EXTRA_MESON_FLAGS:-} \ _build $srcdir unset CCACHE_DISABLE