[gtk+] meson: make sure gsk/gdk generated headers exist for libgtk_dep users



commit f67a9d58dbaff870e872a5144645253fe4b524b0
Author: Tim-Philipp Müller <tim centricular com>
Date:   Wed Mar 22 00:45:39 2017 +0000

    meson: make sure gsk/gdk generated headers exist for libgtk_dep users
    
    Add back dependencies on libgdk_dep and libsk_dep which are declared
    dependencies. We removed this before because these declarations had
    link_with: lines that dragged in the static libgdk.a and libgsk.a libs
    which are linked into libgtk-4.so anyway and thus shouldn't be used
    when linking internal exes/tools against libgtk-4. Remove the static
    libs from the declared dependencies and have libgtk link those in
    explicitly, so that the declared deps now just provide all the built
    dependencies and include dirs and such for declared libgtk_dep users
    such as the internal exes/tools, which want all the generated gsk/gdk/gtk
    headers to exist before attempting to compile anything against the
    gtk+ headers.

 gdk/meson.build           |   12 +++++-------
 gsk/meson.build           |   12 +++++++-----
 gtk/meson.build           |   21 ++++++++-------------
 testsuite/gdk/meson.build |    2 +-
 4 files changed, 21 insertions(+), 26 deletions(-)
---
diff --git a/gdk/meson.build b/gdk/meson.build
index e8656f1..fa5b66f 100644
--- a/gdk/meson.build
+++ b/gdk/meson.build
@@ -199,15 +199,13 @@ libgdk = static_library('gdk',
   c_args: ['-DGDK_COMPILATION', '-DG_LOG_DOMAIN="Gdk"'],
   include_directories: [confinc, xinc, wlinc],
   link_with: gdk_backends,
-  dependencies: gdk_deps,
-)
-
+  dependencies: gdk_deps)
 
+# We don't have link_with: to internal static libs here on purpose, just
+# list the dependencies and generated headers and such, for use in the
+# "public" libgtk_dep used by internal executables.
 libgdk_dep = declare_dependency(
   sources: ['gdk.h', gdkconfig, gdkenum_h],
   depends: gdk_sources,
   include_directories: [confinc, xinc, wlinc],
-  dependencies: gdk_deps,
-  link_with: libgdk,
-  link_args: ['-Bsymbolic'],
-)
+  dependencies: gdk_deps)
diff --git a/gsk/meson.build b/gsk/meson.build
index 3bd0590..0b87438 100644
--- a/gsk/meson.build
+++ b/gsk/meson.build
@@ -157,12 +157,14 @@ libgsk = static_library('gsk',
   dependencies: gsk_deps,
   include_directories: [confinc],
   c_args: ['-DGSK_COMPILATION', '-DG_LOG_DOMAIN="Gsk"'],
-)
+  link_with: libgdk,
+  link_args: ['-Bsymbolic'])
 
+# We don't have link_with: to internal static libs here on purpose, just
+# list the dependencies and generated headers and such, for use in the
+# "public" libgtk_dep used by internal executables.
 libgsk_dep = declare_dependency(
   depends: gsk_sources,
-  link_with: libgsk,
-  link_args: ['-Bsymbolic'],
   include_directories: [confinc],
-  sources: [gskenum_h, gskresources]
-)
+  sources: [gskenum_h, gskresources],
+  dependencies: libgdk_dep)
diff --git a/gtk/meson.build b/gtk/meson.build
index d667a6c..2adbe2c 100644
--- a/gtk/meson.build
+++ b/gtk/meson.build
@@ -830,22 +830,17 @@ libgtk = shared_library('gtk-4',
   sources: [typefuncs, gtk_sources, gtkmarshal_h, gtkprivatetypebuiltins_h],
   c_args: gtk_cargs,
   include_directories: [confinc, gdkinc, gtkinc],
-  dependencies: [gtk_deps, libgdk_dep, libgsk_dep],
-  install: true
-)
+  dependencies: gtk_deps + [libgdk_dep, libgsk_dep],
+  link_with: [libgdk, libgsk],
+  link_args: ['-Bsymbolic'],
+  install: true)
 
 libgtk_dep = declare_dependency(
-  sources: [
-    'gtk.h',
-    gtkversion,
-    gtktypebuiltins_h,
-  ],
-  include_directories: [confinc],
-  dependencies: gtk_deps,
+  sources: [gtkversion, gtktypebuiltins_h],
+  include_directories: [confinc, gtkinc],
+  dependencies: gtk_deps + [libgdk_dep, libgsk_dep],
   link_with: libgtk,
-  link_args: ['-Bsymbolic'],
-  include_directories: [confinc, gtkinc]
-)
+  link_args: ['-Bsymbolic'])
 
 # Installed tools
 
diff --git a/testsuite/gdk/meson.build b/testsuite/gdk/meson.build
index 6c85e75..cfe2d07 100644
--- a/testsuite/gdk/meson.build
+++ b/testsuite/gdk/meson.build
@@ -17,7 +17,7 @@ test('gdk/keysyms', test_keysyms)
 test_rectangle = executable('rectangle', 'rectangle.c', dependencies: libgtk_dep)
 test('gdk/rectangle', test_rectangle)
 
-test_rgba = executable('rgba', 'rgba.c', dependencies: libgdk_dep)
+test_rgba = executable('rgba', 'rgba.c', dependencies: libgtk_dep)
 test('gdk/rgba', test_rgba)
 
 test_seat = executable('seat', 'seat.c', dependencies: libgtk_dep)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]