vpx: avoid confusing meson configure output when checking for vpx versions
authorTim-Philipp Müller <tim@centricular.com>
Mon, 19 Aug 2019 10:03:00 +0000 (11:03 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Mon, 19 Aug 2019 10:09:19 +0000 (11:09 +0100)
Used to print:
|Run-time dependency vpx found: YES 1.7.0
|Message: libvpx provides VP8 encoder interface (vpx_codec_vp8_cx_algo)
|Message: libvpx provides VP8 decoder interface (vpx_codec_vp8_dx_algo)
|Message: libvpx provides VP9 encoder interface (vpx_codec_vp9_cx_algo)
|Message: libvpx provides VP9 decoder interface (vpx_codec_vp9_dx_algo)
|Dependency vpx found: YES (cached)
|Dependency vpx found: NO found '1.7.0' but need: '>=1.8.0'
|Run-time dependency vpx found: NO (tried pkgconfig and cmake)

We can check the version of the found dep in a way that
doesn't produce this confusing output.

ext/vpx/meson.build

index b6c94d6..67a713b 100644 (file)
@@ -44,11 +44,13 @@ if vpx_dep.found()
     message('WARNING: libvpx was built without any encoder or decoder features!')
   endif
 
-  if dependency('vpx', version : '>=1.4.0', required : false).found()
+  if vpx_dep.version().version_compare('>= 1.4.0')
+    message('Found vpx >= 1.4.0')
     vpx_args += '-DHAVE_VPX_1_4'
   endif
 
-  if dependency('vpx', version : '>=1.8.0', required : false).found()
+  if vpx_dep.version().version_compare('>= 1.8.0')
+    message('Found vpx >= 1.8.0')
     vpx_args += '-DHAVE_VPX_1_8'
   endif