[PATCH weston 2/2] automake: add missing git-version.h dependency

Quentin Glidic sardemff7+wayland at sardemff7.net
Tue Jul 5 12:18:16 UTC 2016


On 04/07/2016 15:57, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
>
> compositor/main.c depends on the header, while the dependency isn't
> specified. Thus depending on the order of how things are build we could
> get a build failure.
>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Strictly speaking one could resolve this (and future) issue, by not
> pushing everything from BUILT_SOURCES into CLEANFILES, but I'll leave
> that to someone more familiar with why things are as is.

The better way is to remove stuff from BUILT_SOURCES and add specific 
dependencies (as this patch does).
BUILT_SOURCES has the (big, IMO) downside that you cannot build only one 
target in a clean tree.

I’d prefer dependencies for the actual .o instead of the source file, 
but I guess that’ll do it for now.

Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Cheers,

-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list