On Thu, Jun 29, 2017 at 11:42 AM, Eric Anholt <e...@anholt.net> wrote:
> I want to remove vc4's dependency on headers from libdrm as well, but
> storing multiple copies of drm_fourcc.h in our tree would be silly.
>
> v2: Update Android.mk as well, move distcheck drm*.h references to
> top-level noinst_HEADERS.
>
> Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
> Reviewed-by: Daniel Stone <dani...@collabora.com>
> ---
> Makefile.am | 4 ++++
> {src/intel/drm => include/drm-uapi}/README | 0
> {src/intel/drm => include/drm-uapi}/drm.h | 0
> {src/intel/drm => include/drm-uapi}/drm_fourcc.h | 0
> {src/intel/drm => include/drm-uapi}/drm_mode.h | 0
> {src/intel/drm => include/drm-uapi}/i915_drm.h | 0
> src/intel/Android.vulkan.mk | 2 +-
> src/intel/Makefile.am | 1 -
> src/intel/Makefile.drm.am | 22 ----------------------
> src/intel/Makefile.sources | 6 ------
> src/intel/Makefile.vulkan.am | 2 +-
> src/mesa/drivers/dri/i965/Android.mk | 4 ++--
> src/mesa/drivers/dri/i965/Makefile.am | 2 +-
> 13 files changed, 9 insertions(+), 34 deletions(-)
> rename {src/intel/drm => include/drm-uapi}/README (100%)
> rename {src/intel/drm => include/drm-uapi}/drm.h (100%)
> rename {src/intel/drm => include/drm-uapi}/drm_fourcc.h (100%)
> rename {src/intel/drm => include/drm-uapi}/drm_mode.h (100%)
> rename {src/intel/drm => include/drm-uapi}/i915_drm.h (100%)
> delete mode 100644 src/intel/Makefile.drm.am
I don't mean to pick on this patch specifically, but maybe it would
still make sense to depend on libdrm for the drm headers? If not do
we want similar restrictions on updating these as we have for libdrm?
Alex
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
