[PATCH] test: use LDFLAGS in test/Makefile.local

Tomi Ollila tomi.ollila at iki.fi
Tue Oct 28 14:32:11 PDT 2014


On Mon, Oct 20 2014, Jani Nikula wrote:

> Apparently the test binaries are built with minimal LDFLAGS, only
> adding dependency specific LDFLAGS as needed. However because some of
> the test binaries incorporate notmuch object files, it is necessary to
> use the same link flags as notmuch. For example user provided
> CFLAGS/CXXFLAGS/LDFLAGS with -fsanitize=undefined fails to build the
> test binaries if the flags differ.
>
> ---
>
> With this I could do
>
> CFLAGS="-fsanitize=undefined -g" CXXFLAGS="-fsanitize=undefined -g" LDFLAGS="-fsanitize=undefined -g" ./configure
>
> to try ubsan with gcc 4.9. No problems found while running make test
> (and it did find an UB I planted to ensure I had it all set up right).
> ---
>  test/Makefile.local | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/test/Makefile.local b/test/Makefile.local
> index a2d58fc1a1b0..2331ceb15e33 100644
> --- a/test/Makefile.local
> +++ b/test/Makefile.local
> @@ -13,10 +13,10 @@ smtp_dummy_srcs =		\
>  smtp_dummy_modules = $(smtp_dummy_srcs:.c=.o)
>  
>  $(dir)/arg-test: $(dir)/arg-test.o command-line-arguments.o util/libutil.a
> -	$(call quiet,CC) $^ -o $@
> +	$(call quiet,CC) $^ -o $@ $(LDFLAGS)
>  
>  $(dir)/hex-xcode: $(dir)/hex-xcode.o command-line-arguments.o util/libutil.a
> -	$(call quiet,CC) $^ $(TALLOC_LDFLAGS) -o $@
> +	$(call quiet,CC) $^ -o $@ $(LDFLAGS) $(TALLOC_LDFLAGS)

LGTM; especially the above change which moves all *LDFLAGS after -o $@...

Tomi

>  
>  random_corpus_deps =  $(dir)/random-corpus.o  $(dir)/database-test.o \
>  			notmuch-config.o command-line-arguments.o \
> @@ -24,19 +24,19 @@ random_corpus_deps =  $(dir)/random-corpus.o  $(dir)/database-test.o \
>  			parse-time-string/libparse-time-string.a
>  
>  $(dir)/random-corpus: $(random_corpus_deps)
> -	$(call quiet,CXX) $(CFLAGS_FINAL) $^ -o $@ $(CONFIGURE_LDFLAGS)
> +	$(call quiet,CXX) $^ -o $@ $(LDFLAGS) $(CONFIGURE_LDFLAGS)
>  
>  $(dir)/smtp-dummy: $(smtp_dummy_modules)
> -	$(call quiet,CC) $^ -o $@
> +	$(call quiet,CC) $^ -o $@ $(LDFLAGS)
>  
>  $(dir)/symbol-test: $(dir)/symbol-test.o lib/$(LINKER_NAME)
> -	$(call quiet,CXX) $^ -o $@ -Llib -lnotmuch $(XAPIAN_LDFLAGS)
> +	$(call quiet,CXX) $^ -o $@ $(LDFLAGS) -Llib -lnotmuch $(XAPIAN_LDFLAGS)
>  
>  $(dir)/parse-time: $(dir)/parse-time.o parse-time-string/parse-time-string.o
> -	$(call quiet,CC) $^ -o $@
> +	$(call quiet,CC) $^ -o $@ $(LDFLAGS)
>  
>  $(dir)/make-db-version: $(dir)/make-db-version.o
> -	$(call quiet,CXX) $^ -o $@ $(XAPIAN_LDFLAGS)
> +	$(call quiet,CXX) $^ -o $@ $(LDFLAGS) $(XAPIAN_LDFLAGS)
>  
>  .PHONY: test check
>  
> -- 
> 2.1.1
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


More information about the notmuch mailing list