[PATCH 1 of 3] localrepo: add desc parameter to transaction

Matt Mackall mpm at selenic.com
Mon Apr 12 00:56:11 CDT 2010


On Sun, 2010-04-11 at 21:08 -0500, Steve Borho wrote:
> On Sun, Apr 11, 2010 at 3:18 PM, Sune Foldager <cryo at cyanite.org> wrote:
> > On 11-04-2010 21:47, Greg Ward wrote:
> >> Second: some deep gut instinct is telling me that "\n" is a better
> >> delimiter in the undo.desc file than ",".  Not sure why I think that,
> >> but it feels more right to me.  Maybe I want to be able to parse it
> >> with 'head' and 'tail' in a pinch.
> >
> > Yes, I agree with that as well. Unless we have some grand plans about
> > being able to store multiple entries later on, of course. Either that or
> > just space, if the first token is always a word.
> 
> Unfortunately, you're both about an hour late, as I pushed the last
> patches (with test fixes) about an hour before these two emails.  Feel
> free to work on that bike shed some more; at least the basics of it
> are now in crew.

I'll probably implement both of these suggestions, thanks.

-- 
http://selenic.com : development and support for Mercurial and Linux




More information about the Mercurial-devel mailing list