Re: [Mesa-dev] [PATCH v2] gallium/targets: don't leave an empty target directory(ies)

2017-03-08 Thread Emil Velikov
On 5 March 2017 at 21:56, Damien Grassart wrote: > This breaks make install for me as well. > Just pushed a fix which should work in pretty much any case that I can think of. > Does this patch make sense for XvMC since by default it gets installed to > '${libdir}' (while

Re: [Mesa-dev] [PATCH v2] gallium/targets: don't leave an empty target directory(ies)

2017-03-05 Thread Damien Grassart
This breaks make install for me as well. Does this patch make sense for XvMC since by default it gets installed to '${libdir}' (while VDPAU goes to '${libdir}/vdpau' by default)? Should there be a check to make sure ${dest_dir} isn't ${libdir}? Thanks, -Damien On Sun, Mar 5, 2017 at 3:11 PM

Re: [Mesa-dev] [PATCH v2] gallium/targets: don't leave an empty target directory(ies)

2017-03-05 Thread Andy Furniss
Emil Velikov wrote: Some drivers do not support certain targets - for example nouveau doesn't do VAAPI, while freedreno doesn't do of the video backends. As such if we enter vdpau when building freedreno/ilo/etc, a vdpau/ folder will be created, empty library will be build and almost

Re: [Mesa-dev] [PATCH v2] gallium/targets: don't leave an empty target directory(ies)

2017-03-03 Thread Matt Turner
Reviewed-by: Matt Turner Thanks Emil! ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH v2] gallium/targets: don't leave an empty target directory(ies)

2017-03-03 Thread Emil Velikov
Some drivers do not support certain targets - for example nouveau doesn't do VAAPI, while freedreno doesn't do of the video backends. As such if we enter vdpau when building freedreno/ilo/etc, a vdpau/ folder will be created, empty library will be build and almost immediately removed. Thus