On Monday, 2018-10-29 10:11:53 -0700, Dylan Baker wrote: > Quoting Eric Engestrom (2018-10-29 09:46:46) > > On Monday, 2018-10-29 09:07:30 -0700, Dylan Baker wrote: > > > Quoting Eric Engestrom (2018-10-28 06:45:05) > > > > Meson already automatically tracks included headers, so there's no need > > > > to add them everywhere; cleans up the code a bit. > > > > > > > > Signed-off-by: Eric Engestrom <eric.engest...@intel.com> > > > > --- > > > > src/vulkan/wsi/meson.build | 23 +++++------------------ > > > > 1 file changed, 5 insertions(+), 18 deletions(-) > > > > > > > > diff --git a/src/vulkan/wsi/meson.build b/src/vulkan/wsi/meson.build > > > > index d073b23dc2586a3fcc98..e9812b663e44dc82b896 100644 > > > > --- a/src/vulkan/wsi/meson.build > > > > +++ b/src/vulkan/wsi/meson.build > > > > @@ -21,12 +21,8 @@ > > > > vulkan_wsi_args = [] > > > > vulkan_wsi_deps = [] > > > > > > > > -files_vulkan_wsi = files( > > > > - 'wsi_common.c', > > > > - 'wsi_common.h', > > > > - 'wsi_common_private.h', > > > > - 'wsi_common_queue.h', > > > > -) > > > > +files_vulkan_wsi = files('wsi_common.c') > > > > + > > > > if with_platform_x11 > > > > vulkan_wsi_args += ['-DVK_USE_PLATFORM_XCB_KHR', > > > > '-DVK_USE_PLATFORM_XLIB_KHR'] > > > > vulkan_wsi_deps += [ > > > > @@ -38,19 +34,13 @@ if with_platform_x11 > > > > dep_xcb_sync, > > > > dep_xshmfence, > > > > ] > > > > - files_vulkan_wsi += files( > > > > - 'wsi_common_x11.c', > > > > - 'wsi_common_x11.h', > > > > - ) > > > > + files_vulkan_wsi += files('wsi_common_x11.c') > > > > endif > > > > > > > > if with_platform_wayland > > > > vulkan_wsi_deps += dep_wayland_client > > > > vulkan_wsi_args += ['-DVK_USE_PLATFORM_WAYLAND_KHR'] > > > > - files_vulkan_wsi += files( > > > > - 'wsi_common_wayland.c', > > > > - 'wsi_common_wayland.h', > > > > - ) > > > > + files_vulkan_wsi += files('wsi_common_wayland.c') > > > > files_vulkan_wsi += [ > > > > wayland_drm_client_protocol_h, > > > > wayland_drm_protocol_c, > > > > @@ -61,10 +51,7 @@ endif > > > > > > > > if with_platform_drm > > > > vulkan_wsi_args += '-DVK_USE_PLATFORM_DISPLAY_KHR' > > > > - files_vulkan_wsi += files( > > > > - 'wsi_common_display.c', > > > > - 'wsi_common_display.h', > > > > - ) > > > > + files_vulkan_wsi += files('wsi_common_display.c') > > > > endif > > > > > > > > if with_xlib_lease > > > > -- > > > > Cheers, > > > > Eric > > > > > > > > > > For this patch: > > > Reviewed-by: Dylan Baker <dy...@pnwbakers.com> > > > > Thanks; pushed. > > > > > > > > I've got a todo on my list for a boring day to go through and remove all > > > .h > > > files from the meson lists, just haven't been bored enough yet :) > > > > Same. I actually started looking at it once but got bored of that too :P > > > > I'll need to write a script to figure out if a given file is already > > included by a file in the list (or a file included by a file...) and > > only if so remove it. Or maybe you'll beat me to it and save me the > > trouble ;) > > You're plan is much smarter than mine. I was just going to run it through > a fast 72 core system and see if I could catch any races :)
Heh, this isn't a bad idea; it could catch other races :P _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev