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 b

Re: [PATCH] Add optional silent-rules support

2009-05-23 Thread Jim Meyering
Sergey Poznyakoff wrote: > Ralf Wildenhues 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 working on th

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

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Ralf Wildenhues 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} should do to help in thi

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 functio

Re: [PATCH] Add optional silent-rules support

2009-05-21 Thread Sergey Poznyakoff
Ralf Wildenhues 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 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. > Sho

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

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 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 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. > +si

[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 + 1