meson: make libplacebo a required dependency

Make it not possible to build mpv without the latest libplacebo anymore. This will allow for less code duplication between mpv and libplacebo, and in the future also let us delete legacy ifdefs and track libplacebo better.
mpv-player · Sep 26, 2023 · 3224717 · 3224717
1 parent ebdf7c9
commit 3224717
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 33 deletions.
diff --git a/ci/build-macos.sh b/ci/build-macos.sh
@@ -14,7 +14,7 @@ PKG_CONFIG_PATH="${FFMPEG_SYSROOT}/lib/pkgconfig/" CC="${CC}" CXX="${CXX}" \
 meson setup build \
     -Dprefix="${MPV_INSTALL_PREFIX}" \
     -D{libmpv,tests}=true \
-    -D{gl,iconv,lcms2,libplacebo,lua,jpeg,plain-gl,zlib}=enabled \
+    -D{gl,iconv,lcms2,lua,jpeg,plain-gl,zlib}=enabled \
     -D{cocoa,coreaudio,gl-cocoa,macos-cocoa-cb,macos-touchbar,videotoolbox-gl}=enabled
 
 meson compile -C build -j4

diff --git a/ci/build-mingw64.sh b/ci/build-mingw64.sh
@@ -271,7 +271,7 @@ rm -rf $build
 meson setup $build --cross-file "$prefix_dir/crossfile" \
     --buildtype debugoptimized \
     -Dlibmpv=true -Dlua=luajit \
-    -D{shaderc,spirv-cross,d3d11,libplacebo}=enabled
+    -D{shaderc,spirv-cross,d3d11}=enabled
 
 meson compile -C $build
 

diff --git a/meson.build b/meson.build
@@ -23,6 +23,9 @@ libavutil = dependency('libavutil', version: '>= 56.70.100')
 libswresample = dependency('libswresample', version: '>= 3.9.100')
 libswscale = dependency('libswscale', version: '>= 5.9.100')
 
+libplacebo = dependency('libplacebo', version: '>=6.338.0', fallback: ['libplacebo', 'libplacebo'],
+    default_options: ['default_library=static', 'demos=false'])
+
 libass = dependency('libass', version: '>= 0.12.2')
 pthreads = dependency('threads')
 
@@ -33,6 +36,7 @@ dependencies = [libass,
                 libavfilter,
                 libavformat,
                 libavutil,
+                libplacebo,
                 libswresample,
                 libswscale,
                 pthreads]
@@ -46,6 +50,7 @@ features = {
     'avif-muxer': libavformat.version().version_compare('>= 59.24.100'),
     'libass': true,
     'threads': true,
+    'libplacebo': true,
 }
 
 
@@ -236,6 +241,12 @@ sources = files(
     'video/repack.c',
     'video/sws_utils.c',
 
+    ## libplacebo
+    'video/out/placebo/ra_pl.c',
+    'video/out/placebo/utils.c',
+    'video/out/vo_gpu_next.c',
+    'video/out/gpu_next/context.c',
+
     ## osdep
     'osdep/io.c',
     'osdep/semaphore_osx.c',
@@ -934,16 +945,6 @@ if features['jpeg']
     dependencies += jpeg
 endif
 
-libplacebo = dependency('libplacebo', version: '>=6.292.0', required: get_option('libplacebo'))
-features += {'libplacebo': libplacebo.found()}
-if features['libplacebo']
-    dependencies += libplacebo
-    sources += files('video/out/placebo/ra_pl.c',
-                     'video/out/placebo/utils.c',
-                     'video/out/vo_gpu_next.c',
-                     'video/out/gpu_next/context.c')
-endif
-
 sdl2_video = get_option('sdl2-video').require(
     features['sdl2'],
     error_message: 'sdl2 was not found!',
@@ -1261,7 +1262,7 @@ endif
 # vulkan
 vulkan_opt = get_option('vulkan').require(
     libplacebo.get_variable('pl_has_vulkan', default_value: '0') == '1',
-    error_message: 'libplacebo could not be found!',
+    error_message: 'libplacebo compiled without vulkan support!',
 )
 vulkan = dependency('vulkan', version: '>= 1.1.70', required: vulkan_opt)
 features += {'vulkan': vulkan.found()}
@@ -1334,9 +1335,8 @@ endif
 
 vulkan_interop = get_option('vulkan-interop').require(
     features['vulkan'] and vulkan.version().version_compare('>=1.3.238') and
-    features['libplacebo'] and
     libavutil.version().version_compare('>=58.11.100'),
-    error_message: 'Vulkan Interop requires vulkan headers >= 1.3.238, libplacebo, and libavutil >= 58.11.100',
+    error_message: 'Vulkan Interop requires vulkan headers >= 1.3.238, and libavutil >= 58.11.100',
 )
 features += {'vulkan-interop': vulkan_interop.allowed()}
 if vulkan_interop.allowed()
@@ -1430,14 +1430,8 @@ features += {'vaapi': vaapi.allowed()}
 if features['vaapi']
     dependencies += libva
     sources += files('video/filter/vf_vavpp.c',
-                     'video/vaapi.c')
-endif
-
-features += {'vaapi-egl': features['vaapi'] and features['egl'] and features['drm']}
-features += {'vaapi-libplacebo': features['vaapi'] and libplacebo.found()}
-
-if features['vaapi-egl'] or features['vaapi-libplacebo']
-    sources += files('video/out/hwdec/hwdec_vaapi.c')
+                     'video/vaapi.c',
+                     'video/out/hwdec/hwdec_vaapi.c')
 endif
 
 dmabuf_interop_gl = features['egl'] and features['drm']
@@ -1446,7 +1440,7 @@ if features['dmabuf-interop-gl']
     sources += files('video/out/hwdec/dmabuf_interop_gl.c')
 endif
 
-dmabuf_interop_pl = features['vaapi-libplacebo']
+dmabuf_interop_pl = features['vaapi']
 features += {'dmabuf-interop-pl': dmabuf_interop_pl}
 if features['dmabuf-interop-pl']
     sources += files('video/out/hwdec/dmabuf_interop_pl.c')
@@ -1775,7 +1769,6 @@ if get_option('tests')
 endif
 
 summary({'d3d11': features['d3d11'],
-         'gpu-next': features['libplacebo'],
          'javascript': features['javascript'],
          'libmpv': get_option('libmpv'),
          'lua': features['lua'],

diff --git a/meson_options.txt b/meson_options.txt
@@ -75,7 +75,6 @@ option('gl-dxinterop', type: 'feature', value: 'auto', description: 'OpenGL/Dire
 option('gl-win32', type: 'feature', value: 'auto', description: 'OpenGL Win32 Backend')
 option('gl-x11', type: 'feature', value: 'disabled', description: 'OpenGL X11/GLX (deprecated/legacy)')
 option('jpeg', type: 'feature', value: 'auto', description: 'JPEG support')
-option('libplacebo', type: 'feature', value: 'auto', description: 'libplacebo support')
 option('rpi', type: 'feature', value: 'disabled', description: 'Raspberry Pi support')
 option('sdl2-video', type: 'feature', value: 'auto', description: 'SDL2 video output')
 option('shaderc', type: 'feature', value: 'auto', description: 'libshaderc SPIR-V compiler')

diff --git a/player/main.c b/player/main.c
@@ -26,9 +26,7 @@
 
 #include "config.h"
 
-#if HAVE_LIBPLACEBO
 #include <libplacebo/config.h>
-#endif
 
 #include "mpv_talloc.h"
 
@@ -153,9 +151,7 @@ void mp_print_version(struct mp_log *log, int always)
     mp_msg(log, v, "%s %s\n", mpv_version, mpv_copyright);
     if (strcmp(mpv_builddate, "UNKNOWN"))
         mp_msg(log, v, " built on %s\n", mpv_builddate);
-#if HAVE_LIBPLACEBO
     mp_msg(log, v, "libplacebo version: %s\n", PL_VERSION);
-#endif
     check_library_versions(log, v);
     mp_msg(log, v, "\n");
     // Only in verbose mode.

diff --git a/video/out/gpu/hwdec.c b/video/out/gpu/hwdec.c
@@ -41,7 +41,7 @@ extern const struct ra_hwdec_driver ra_hwdec_aimagereader;
 extern const struct ra_hwdec_driver ra_hwdec_vulkan;
 
 const struct ra_hwdec_driver *const ra_hwdec_drivers[] = {
-#if HAVE_VAAPI_EGL || HAVE_VAAPI_LIBPLACEBO
+#if HAVE_VAAPI
     &ra_hwdec_vaapi,
 #endif
 #if HAVE_VIDEOTOOLBOX_GL || HAVE_IOS_GL

diff --git a/video/out/vo.c b/video/out/vo.c
@@ -76,9 +76,7 @@ static const struct vo_driver *const video_out_drivers[] =
     &video_out_mediacodec_embed,
 #endif
     &video_out_gpu,
-#if HAVE_LIBPLACEBO
     &video_out_gpu_next,
-#endif
 #if HAVE_VDPAU
     &video_out_vdpau,
 #endif