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

[Brendan Cully]

On Saturday, 10 April 2010 at 15:18, Steve Borho wrote:
> On Sat, Apr 10, 2010 at 3:16 PM, Steve Borho <steve at borho.org> wrote:
> > On Fri, Apr 9, 2010 at 11:28 PM, Brendan Cully <brendan at kublai.com> wrote:
> >> I agree that revision numbers are fine for rollback. I do wonder if it
> >> might be nice to capture the transaction source as well as type in the
> >> description:
> >>
> >> rolling back pull from http://foo/
> >> rolling back unbundle from ../foo.bundle
> >>
> >> etc.
> >
> > I tried this, but in many operations the provided url is more noise
> > than information.  Something maybe with a little intelligence could be
> > added after my patch series is pushed.
> 
> For example: Many of the test outputs change like this:
> 
> -rolling back last transaction
> +rolling back push (file:/tmp/hgtests.SwlAx1/test-acl/a) to revision 1
> 
> Which would be a PITA to clean up.

Maybe we change the tests to rollback -q, or we only show the URL for
rollback -n or rollback -v ?