On Thursday, 2018-06-21 08:38:05 -0700, Dylan Baker wrote:
> Currently if vdpau is set to auto, it will be disabled only in cases
> where gallium is disabled or the host OS is not supported (mac, haiku,
> windows). However on (for example) Linux if libvdpau is not installed
> then the build will error because of the unmet dependency. This corrects
> auto to do the right thing, and not error if libvdpau is not installed.
>
> Fixes: 992af0a4b8224bdb4809e01c2f00d2f32546aee5
> ("meson: dedup gallium-vdpau logic")
> Signed-off-by: Dylan Baker <dylan.c.ba...@intel.com>
Oops, that was me :]
Series is
Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
> ---
> meson.build | 17 +++++++++--------
> 1 file changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/meson.build b/meson.build
> index a2d59776c05..429865c9442 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -398,16 +398,17 @@ elif not (with_gallium_r300 or with_gallium_r600 or
> with_gallium_radeonsi or
> else
> _vdpau = 'false'
> endif
> -elif _vdpau == 'auto'
> - _vdpau = 'true'
> endif
> -with_gallium_vdpau = _vdpau == 'true'
> dep_vdpau = null_dep
> -if with_gallium_vdpau
> - dep_vdpau = dependency('vdpau', version : '>= 1.1')
> - dep_vdpau = declare_dependency(
> - compile_args : run_command(prog_pkgconfig, ['vdpau',
> '--cflags']).stdout().split()
> - )
> +with_gallium_vdpau = false
> +if _vdpau != 'false'
> + dep_vdpau = dependency('vdpau', version : '>= 1.1', required : _vdpau ==
> 'true')
> + if dep_vdpau.found()
> + dep_vdpau = declare_dependency(
> + compile_args : run_command(prog_pkgconfig, ['vdpau',
> '--cflags']).stdout().split()
> + )
> + with_gallium_vdpau = true
> + endif
> endif
>
> if with_gallium_vdpau
> --
> 2.17.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
