Quoting Emil Velikov (2018-02-08 15:17:16)
> On 8 February 2018 at 23:11, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> > On 2 February 2018 at 19:30, Dylan Baker <dy...@pnwbakers.com> wrote:
> >> The VDPAU state tracker needs to be linked with whole-archive (autotools
> >> does this). Because we are linking the whole archive we alos need to
> >> link with libswdri and libswkmsdri if those have been enabled.
> >>
> >> Fixes: 68076b87474e7959 ("meson: build gallium vdpau state tracker")
> >> Signed-off-by: Dylan Baker <dylan.c.ba...@intel.com>
> >> ---
> >
> >> +if with_dri
> >> + vdpau_link_with += libswdri
> >> +endif
> >> +if with_gallium_drisw_kms
> >> + vdpau_link_with += libswkmsdri
> >> +endif
> >> +
> > This is not specific for vdpau - va/omx/etc also need it.
> > Instead of duplicating the code - one could fold that once into
> > another meson.build file.
> >
> As an example - the rest of the series does copy the exact same hunk
> across the remainder.
> When we already have it in nine and dri targets :-\
>
> Thanks
> EmilYeah, I thought about that. I'm just not sure what the right level to fold that up into is. I also felt like the obvious but stupid thing was better for nominating to stable than a more clever approach. Dylan
signature.asc
Description: signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
