<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] new fmtpipe updates



On Tuesday, 13 March 2007 at 13:45, Brendan Cully wrote:
> I can't figure out the right syntax for this. Both approaches seem to
> have defects. There are comments in line below.

On further reflection, I think your fmtpipe.3 patch should work well
enough -- people will just have to get used to putting their string in
extra quoting. But the one you submitted seems to have some garbage in
it ('/tmp/ass', and an #if 0 block), and doesn't really quote things
properly for shell (embedded single quotes would break it). Can you
clean these things up, and maybe add the \| check?

Attachment: pgpeI35VyiM6D.pgp
Description: PGP signature