Re: [PATCH 7/7] syntax-check: fix violations and implement sc_prohibit_sed_s_comma.

[Eric Blake]

On 11/21/2011 07:47 AM, Gary V. Vaughan wrote:
> I like to name temporary directories that I will remove shortly
> with two leading commas so that they sort lexicographically at
> the top of `ls' output.  Now, `./configure
> --prefix=`pwd`/,,inst' works again, for the first time in
> several years.

> Try to use `|' as the default separator wherever possible,
> otherwise something else that doesn't occur in the substitution
> expression.

I'm in favor of this one.  In particular, one of the reasons that
autoconf documents | as superior to , is that | has to be shell-quoted
to be used, while , does not, which means a user is more likely to have
a filename containing comma than they are to have a filename containing
a pipe.

-- 
Eric Blake   address@hidden    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature