[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Eric Blake
Subject: Re: [PATCH 7/7] syntax-check: fix violations and implement sc_prohibit_sed_s_comma.
Date: Mon, 21 Nov 2011 13:19:17 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20110930 Thunderbird/7.0.1

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

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]