[PATCH 1/2] Build symbol-test with make instead of hardcoding in symbol-hiding.

David Bremner david at tethera.net
Fri Nov 25 05:04:44 PST 2011


On Fri, 25 Nov 2011 11:54:51 +0100, Amadeusz Żołnowski <aidecoe at aidecoe.name> wrote:
>  
> +$(dir)/symbol-test: $(dir)/symbol-test.o
> +	$(call quiet,CC) $^ -o $@ -Llib -lnotmuch -lxapian

I'm a bit surprised no -I is neaded here. Is that taken care in the CC call?

>  .PHONY: test check
> -test:	all $(dir)/smtp-dummy
> +test:	all $(dir)/smtp-dummy $(dir)/symbol-test
>  	@${dir}/notmuch-test $(OPTIONS)

The instructions were already wrong, but maybe they should be updated to
point out that calling the test scripts directly requires smtp-dummy and
symbol-test already be built. Or maybe the tests should check for these
binaries as pre-reqs along with emacs, gpg, etc... 

> +	   -e "/^(emacs.expected-output|smtp-dummy|smtp-dummy.c|test-verbose|symbol-test.cc|symbol-test|symbol-test.o)/d" \

I wonder if this should be shortened (and made a bit sloppier) with the
use of regexes.



More information about the notmuch mailing list