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                       [email protected]
>  Paul Lunau                      [email protected]
>  Paul Martinolich                [email protected]
>  Paul Thomas                     [email protected]
> +Pavel Raiskup                   [email protected]

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