Re: [Automake-NG] [PATCH 2/2] [ng] all, check, install: move more processing at make runtime

[Akim Demaille]

Le 15 juin 2012 à 00:05, Stefano Lattarini a écrit :

> +.PHONY: all all-am all-local
> +ifdef SUBDIRS

Sometimes you using ifdef, and sometimes ifeq "".  Are they equivalent?

> +ifeq ($(strip %LOCAL-HEADERS% $(BUILT_SOURCES)),)
> +all: $(if $(SUBDIRS),all-recursive,all-am)
> +else
> +all: %LOCAL-HEADERS% $(BUILT_SOURCES)
> +     $(MAKE) $(if $(SUBDIRS),all-recursive,all-am)
> +endif

There's no way to avoid forking another Make here?


> am__mkdir = test -d $1 || $(MKDIR_P) $1

Is the "test" really needed?  "mkdir -p /already/exists" should silently
exit 0, shouldn't it?



Thorough testing, as usual :)