Re: [PATCH] Add optional silent-rules support

2009-05-23 Thread Jim Meyering
Sergey Poznyakoff wrote: Ralf Wildenhues ralf.wildenh...@gmx.de ha escrit: - the awk script won't detect if there are already $(AM_V_...) variables present in the rules, Yes, that's intended. It was supposed that silent-rules variables can not appear in the input. Hi Sergey, Thanks for

Re: [PATCH] Add optional silent-rules support

2009-05-23 Thread Bruno Haible
Eric Blake wrote: Would it be possible to change gnulib-tool to generate the appropriate $(AM_V_GEN) output itself, rather than making bootstrap do a post-process pass? After all, not all projects use the bootstrap script. Seconded, because - the silencing that you have in mind can also be

[PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Hello, How about the following patch, which adds support for `silent-rules' mode introduced in Automake 1.11: * build-aux/bootstrap (slurp): Add silent rule support to $gnulib_mk, if required by the configure.ac. --- build-aux/bootstrap | 49 +

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Ralf Wildenhues
Hi Sergey, * Sergey Poznyakoff wrote on Thu, May 21, 2009 at 11:07:53AM CEST: How about the following patch, which adds support for `silent-rules' mode introduced in Automake 1.11: Neat idea. :-) +# AWK program to convert a Makefile(.am) file rules to Automake 1.11 +# silent mode.

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Ralf Wildenhues
A couple more data points: - the awk script won't detect if there are already $(AM_V_...) variables present in the rules, - the script won't cope with some of the more elaborate GNU make-specific constructs; this is probably totally irrelevant for gnulib, but not for: - this could be done

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Jim Meyering
Sergey Poznyakoff wrote: How about the following patch, which adds support for `silent-rules' mode introduced in Automake 1.11: * build-aux/bootstrap (slurp): Add silent rule support to $gnulib_mk, if required by the configure.ac. Hi Sergey, Nice. I did something similar using perl, but

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Jim Meyering
Ralf Wildenhues wrote: A couple more data points: - the awk script won't detect if there are already $(AM_V_...) variables present in the rules, - the script won't cope with some of the more elaborate GNU make-specific constructs; this is probably totally irrelevant for gnulib, but

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Hi Ralf, Can gnulib-tool create target1 target2 ... \ targetN : prereq1 ... \ prereqN ; rule-command rules? Well, quick grep through modules/* shows that so far no module generates such rules. But in theory it is possible. I'll take it into account, then. Shouldn't

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Ralf Wildenhues ralf.wildenh...@gmx.de ha escrit: - the awk script won't detect if there are already $(AM_V_...) variables present in the rules, Yes, that's intended. It was supposed that silent-rules variables can not appear in the input. Regards, Sergey

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Ralf Wildenhues
* Sergey Poznyakoff wrote on Thu, May 21, 2009 at 11:58:15AM CEST: FWIW the awk script is not portable to ancient awk, such as found on Solaris. Hmm, but why? I didn't use any GNU awk extensions, did I? Not as far as I can see. Solaris /bin/awk doesn't have the match and sub functions,

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Ralf Wildenhues ralf.wildenh...@gmx.de ha escrit: Not as far as I can see. Solaris /bin/awk doesn't have the match and sub functions, and has several other limitations over POSIX awk. Ah, I see. On Solaris, you should be able to use nawk or /usr/xpg4/bin/awk. OK, then using ${AWK-awk}

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Sergey Poznyakoff on 5/21/2009 3:07 AM: Hello, How about the following patch, which adds support for `silent-rules' mode introduced in Automake 1.11: * build-aux/bootstrap (slurp): Add silent rule support to $gnulib_mk, if