From: Jeff King <peff@peff.net>
Date: Mon, 10 Feb 2014 01:19:02 GMT
Subject: Re: [PATCH 0/2] Ignore trailing spaces in .gitignore
Message-ID: <20140210011859.GB12773@sigill.intra.peff.net>
In-Reply-To: <CACsJy8C_Lc+mcJxGc7S5AB47YKDrhmWPwZp2oxJ9Z1tONVT0kQ@mail.gmail.com>

On Sun, Feb 09, 2014 at 06:48:18AM +0700, Duy Nguyen wrote:

> > I guess by quoting you meant:
> >
> >   echo '"trailing  "' >.gitignore
> 
> This makes " special. If we follow shell convention then things
> between ".." should be literal (e.g. "*" is no longer a wildcard). We
> don't support it yet. So I rather go with backslash as it adds less
> code.

For some reason I was thinking that we already handled double-quotes
here (as we do in other places where quoting is optional). But it looks
like we don't currently, so yeah, I don't think it is worth adding due
to the potential confusion.

Backslash-escaping was what I had originally assumed you meant, and it
was, so we are all on the same page (the patch was just broken. ;) ).

-Peff