Hi Stefano, thanks for refinements!  I'm ok with these patches.

> [..]
>
> See also:
> <http://lists.gnu.org/archive/html/bug-automake/2013-01/msg00115.html>
> <http://lists.gnu.org/archive/html/automake-patches/2010-02/msg00030.html>
> <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=565663>
> <https://bugzilla.redhat.com/show_bug.cgi?id=901333>
>
> * aclocal.in (SCAN_M4_DIRS_SILENT, SCAN_M4_DIRS_WARN,
> SCAN_M4_DIRS_ERROR): New constants.

This is a nit but the first line should be closed by ')' and second line
opened with '('.

> [..]
> diff --git a/THANKS b/THANKS
> index 66498d4..5c014bf 100644
> --- a/THANKS
> +++ b/THANKS
> @@ -297,6 +297,7 @@ Paul Jarc                       p...@po.cwru.edu
>  Paul Lunau                      t...@lunau.me.uk
>  Paul Martinolich                marti...@datasync.com
>  Paul Thomas                     ptho...@novell.com
> +Pavel Raiskup                   prais...@redhat.com

Thanks :).

> +$ACLOCAL -Wno-error 2>stderr \
> +  && cat stderr >&2 \
> +  && test $(grep -c "couldn't open directory 'not-exist'" stderr) -eq 1 \

Nice check!

Pavel





Reply via email to