meson: rename `glvnd_missing_pc_files` to `not glvnd_has_headers_and_pc_files`
authorEric Engestrom <eric.engestrom@intel.com>
Mon, 30 Sep 2019 09:26:43 +0000 (10:26 +0100)
committerEric Engestrom <eric.engestrom@intel.com>
Thu, 10 Oct 2019 21:18:04 +0000 (22:18 +0100)
This reflects better what is provided by glvnd or not.

Fixes: 93df862b6affb6b8507e ("meson: re-add incorrect pkg-config files with GLVND for backward compatibility")
Signed-off-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
meson.build
src/egl/meson.build
src/meson.build

index 20c40f1d22178f965928564c7a2b12712690765b..28a2e6803c6e1f0c836fad1b939773f9bc08a94f 100644 (file)
@@ -1328,13 +1328,12 @@ else
 endif
 
 dep_glvnd = null_dep
-glvnd_missing_pc_files = false
+glvnd_has_headers_and_pc_files = false
 if with_glvnd
   dep_glvnd = dependency('libglvnd', version : '>= 0.2.0')
-  # GLVND until commit 0dfaea2bcb7cdcc785f9 ("Add pkg-config files for EGL, GL,
-  # GLES, and GLX.") was missing its pkg-config files, forcing every vendor to
-  # provide them and the distro maintainers to resolve the conflict.
-  glvnd_missing_pc_files = dep_glvnd.version().version_compare('< 1.2.0')
+  # GLVND before 1.2 was missing its pkg-config and header files, forcing every
+  # vendor to provide them and the distro maintainers to resolve the conflict.
+  glvnd_has_headers_and_pc_files = dep_glvnd.version().version_compare('>= 1.2.0')
   pre_args += '-DUSE_LIBGLVND=1'
 endif
 
index 7d738e2ebd11fb4e086c1a876d211652cb68b5de..9c2c055c3eb07f8852345e0bf3e58d61920492de 100644 (file)
@@ -176,8 +176,8 @@ libegl = shared_library(
 # If using glvnd the pkg-config header should not point to EGL_mesa, it should
 # point to EGL. glvnd is only available on unix like platforms so adding -l
 # should be safe here
-if not with_glvnd or glvnd_missing_pc_files
-  if glvnd_missing_pc_files
+if not with_glvnd or not glvnd_has_headers_and_pc_files
+  if not glvnd_has_headers_and_pc_files
     _egl = '-L${libdir} -lEGL'
   else
     _egl = libegl
index 10b2772f907d5bbf041353ff965fb901823c6169..15fa52be57c4f84c081e694be8b834de5a8d43ef 100644 (file)
@@ -112,12 +112,12 @@ endif
 
 # This must be after at least mesa, glx, and gallium, since libgl will be
 # defined in one of those subdirs depending on the glx provider.
-if with_glx != 'disabled' and (not with_glvnd or glvnd_missing_pc_files)
+if with_glx != 'disabled' and (not with_glvnd or not glvnd_has_headers_and_pc_files)
   # If using glvnd the pkg-config header should not point to GL_mesa, it should
   # point to GL. glvnd is only available on unix like platforms so adding -l
   # should be safe here
   # TODO: in the glvnd case glvnd itself should really be providing this.
-  if glvnd_missing_pc_files
+  if not glvnd_has_headers_and_pc_files
     _gl = '-L${libdir} -lGL'
   else
     _gl = libgl