Re: [RESEND] [PATCH] Mark ofono/*.h symbolink links as built sources

[Marcel Holtmann <marcel@holtmann.org>]

[-- Attachment #1: Type: text/plain, Size: 1059 bytes --]

Hi Remi,

> This forces automake/make to build them first if needed (as before).
> But it avoids marking every single header as a dependency of every
> single object. Thus we do not need a bogus full rebuild of the tree
> everytime a header is added.

I have run some tests on this and seems fine to me.

> ---
>  Makefile.am |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/Makefile.am b/Makefile.am
> index a4c47e8..ee7949d 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -326,7 +326,8 @@ src_ofonod_LDADD = $(builtin_libadd) @GLIB_LIBS@ @DBUS_LIBS@ @CAPNG_LIBS@ -ldl
>  src_ofonod_LDFLAGS = -Wl,--export-dynamic \
>  				-Wl,--version-script=$(srcdir)/src/ofono.ver
>  
> -CLEANFILES = src/builtin.h $(local_headers) $(rules_DATA)
> +BUILT_SOURCES = $(local_headers)
> +CLEANFILES = src/builtin.h $(BUILT_SOURCES) $(rules_DATA)

But can you please add an empty line between these two variable
assignments. Just for consistency with the rest of the Makefile.am code.

Regards

Marcel