Re: braced variable expansion in here documents

[Stepan Kasal]

Hello Paul,

On Fri, Jun 02, 2006 at 03:45:33PM -0700, Paul Eggert wrote:
> Stepan Kasal <address@hidden> writes:
> >     * lib/m4sugar/m4sh.m4 (_AS_HERE_DOC_WORKS): Check for the
> >     ksh93f bug with long here-documents.
> >     (AS_INIT): Require this check.
> 
> I have some qualms about this, on efficiency grounds.

actually, I found out that my patch was totally bogus: it used
as_shell which either have not been set (first expansion) or have not
been exported (second expansion).

The check would have to write conftest.sh and source it (using the
`.' builtin).

> It will slow down startup, for the sake only of (as far as we know)
> Unixware 7.1.1 which is a minority system.

Well, computer time is cheap, people's attention is expensive.

Your workaround and Ralf's review of the rest of the Autoconf code
made me feel bounded to keep this limitation in my mind when I write
future code.  I refuse to do that.

So I'm going to forget about the bug right now.  It's in the best
interest of the upcoming release.  But next time I stumble over it,
I will again try to write a startup time check again.  ;-)

> The documentation change looks good, though.  Thanks.  I tweaked it a
> bit and installed this:

Thank you for finishing it!

Stepan Kasal