Hi On Sun, May 5, 2024 at 12:29 AM Dmitry Osipenko <dmitry.osipe...@collabora.com> wrote: > > New virglrerenderer features were stabilized with release of v1.0.0. > Presence of symbols in virglrenderer.h doesn't guarantee ABI compatibility > with pre-release development versions of libvirglerender. Use virglrenderer > version to decide reliably which virgl features are available. > > Signed-off-by: Dmitry Osipenko <dmitry.osipe...@collabora.com> > --- > meson.build | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/meson.build b/meson.build > index 5db2dbc12ec7..f4a4d71c1978 100644 > --- a/meson.build > +++ b/meson.build > @@ -2286,11 +2286,8 @@ config_host_data.set('CONFIG_PNG', png.found()) > config_host_data.set('CONFIG_VNC', vnc.found()) > config_host_data.set('CONFIG_VNC_JPEG', jpeg.found()) > config_host_data.set('CONFIG_VNC_SASL', sasl.found()) > -if virgl.found() > - config_host_data.set('HAVE_VIRGL_D3D_INFO_EXT', > - cc.has_member('struct > virgl_renderer_resource_info_ext', 'd3d_tex2d', > - prefix: '#include <virglrenderer.h>', > - dependencies: virgl)) > +if virgl.version().version_compare('>=1.0.0') > + config_host_data.set('HAVE_VIRGL_D3D_INFO_EXT', 1)
If it checks for version >=1.0, we may as well depend on >=1.0.1 imho, and then simply use VIRGL_CHECK_VERSION in the code. > endif > config_host_data.set('CONFIG_VIRTFS', have_virtfs) > config_host_data.set('CONFIG_VTE', vte.found()) > -- > 2.44.0 > -- Marc-André Lureau