[darcs-devel] darcs patch: Allow escaped quotes in `quoted' for match text.

Dave Love fx at gnu.org
Sun Nov 5 06:03:06 PST 2006


"Eric Y. Kow" <eric.kow at gmail.com> writes:

> I'll have a second look tomorrow morning.

Yes, please check it, though I have tested it.

> My preference would be for
> this new matching syntax not to require people to escape things -- they
> already have regex escaping to deal with -- and for the syntax to be as
> consistent as possible.

Right, but that's basically inescapable if you're going to be able to
match things generally.

As background, I found the issue in a real case with Emacs support.
It was selecting a patch automatically from the logged name and barfed
on one which included double quotes.  I was expecting quoting them to
work.  [That was meant to be included with the original patch, but
wasn't due to issue 315...]




More information about the darcs-devel mailing list