[Mesa-dev] [PATCH v2] intel: Move the DRM uapi headers to a non-Intel location.

[Alex Deucher]

On Thu, Jun 29, 2017 at 11:42 AM, Eric Anholt <eric at 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.landwerlin at intel.com>
> Reviewed-by: Daniel Stone <daniels at 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