[dropping bug-autoconf]
On 06/14/2011 02:04 AM, Stefano Lattarini wrote:
> Hi Eric. I have some minor issues with this patch ...
>
>> +
>> +@example
>> +$ @kbd{echo 'echo hello >&5\n' >k
>>
> Why the `\n' here?
Oops. I started with printf '#!/bin/sh\necho hello >&5\n', but then
realized that I didn't always want to run /bin/sh, and changed printf to
echo. Incompletely.
>>
> I'd reorder and rework the above examples as follow:
>
> $ @kbd{bash -c 'exec 5>&-; echo -`bash ./k`-'}
> ./k: line 1: 5: Bad file number
> --
> $ @kbd{ksh -c 'exec 5>&-; echo -`ksh ./k`-'}
> ./k[1]: 5: cannot open [Bad file number]
> --
> $ @kbd{sh -c 'exec 5>&-; echo -`sh ./k`-}
> -hello-
>
> $ @kbd{sh -c 'exec 5>t; echo -`sh ./k`-`cat t`-'}
> --hello-
> $ @kbd{bash -c 'exec 5>t; echo -`bash ./k`-`cat t`-'}
> --hello-
> $ @kbd{ksh -c 'exec 5>t; echo -`ksh ./k`-`cat t`-'}
> ./k[1]: 5: cannot open [Bad file number]
> ---
> $ @kbd{ksh -c '(echo -`ksh ./k`-`cat t`-) 5>t'}
> --hello-
> $ @kbd{ksh -c '{ echo -`ksh ./k`-`cat t`-; } 5>t'}
> --hello-
> $ @kbd{ksh -c 'echo -`5>t ksh ./k`-`cat t`-'}
> --hello-
>
> At this point, the above might as well be broken in two distinct
> examples, which can thus be kept nearer to the descripton of the
> issues they demonstrate.
>
> Note that I'm just saying this makes things clearer *for me*, but
> I don't know if it does for other people too; so your call in the
> end.
Yeah, that does seem better; it was two paragraphs previously, so I
don't mind rearranging it to keep it as two examples. v2 coming up.
--
Eric Blake [email protected] +1-801-349-2682
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
