[PATCH] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
From: Gary V. Vaughan g...@gnu.org Following a recent discussion on bug-gnulib, Jim tells me not to use the ugly as_echo setting boilerplate in my scripts anymore, since printf has been available on every shell we care about for at least 5 if not 10 years. That affords us the opportunity to

Re: [PATCH] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Paul Eggert
This change would break gnulib's gl_CACHE_VAL_SILENT. Is there some way you can fix that? It's OK if we have to modify gl_CACHE_VAL_SILENT, it's just that we need to modify it in a way that'll work with both old and new Autoconf. Also, any such patch to Autoconf should update the documentation a

Re: [PATCH] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
Hi Paul, On 29 Jan 2013, at 06:13, Paul Eggert egg...@cs.ucla.edu wrote: This change would break gnulib's gl_CACHE_VAL_SILENT. Is there some way you can fix that? Sure, since gl_CACHE_VAL_SILENT is already using undocumented internals of Autoconf, how about: m4_pushdef([_AS_ECHO_N], [: ]) ...

[PATCHv2] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
Added NEWS entry, and updated texinfo documentation for echo. Okay to push? printf '%s\n' ... has been a fine replacement for plain echo for at least 5 years (probably more like 10), even with most museum-piece shells. * lib/m4sugar/m4sh.m4 (_AS_ECHO_PREPARE): Remove. (_AS_SHELL_SANITIZE): Keep

Re: [PATCHv2] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
On 29 Jan 2013, at 09:24, Gary V. Vaughan g...@gnu.org wrote: Added NEWS entry, and updated texinfo documentation for echo. Okay to push? [[snip]] +** Use of 'printf' is now recommended instead of working around bugs in + 'echo'. Macros AS_ECHO and AS_ECHO_N now expand unconditionally to

Re: [PATCHv2] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Peter Rosin
On 2013-01-29 03:24, Gary V. Vaughan wrote: * doc/autoconf.text (Limitations of Shell Builtins): Document s/text/texi/ Cheers, Peter

[PATCHv3] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
Incorporating feedback from Paul and Paul. Thank you both :) Okay to push? printf '%s\n' ... has been a fine replacement for plain echo for at least 5 years (probably more like 10), even with most museum-piece shells. * lib/m4sugar/m4sh.m4 (_AS_ECHO_PREPARE): Remove. (_AS_SHELL_SANITIZE): Keep

Re: [PATCHv3] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Gary V. Vaughan
On 29 Jan 2013, at 13:17, Gary V. Vaughan g...@gnu.org wrote: Incorporating feedback from Paul and Paul. Of course that should say Paul and *Peter*. Apologies. Typos on parade. Time for a break methinks. Cheers, -- Gary V. Vaughan (gary AT gnu DOT org)

Re: [PATCHv3] m4sugar: factor away _AS_ECHO_PREPARE.

2013-01-28 Thread Paul Eggert
Thanks, that looks good to me; please push.