I felt I should cross-post since I'm trying to infect core python developers with ReviewBoard goodness. Original and related threads are at:
http://groups.google.com/group/the-fellowship-of-the-packaging
Cheers,
--
~ Dan
----- Forwarded message from Dan Buch <daniel.b...@gmail.com> -----
Date: Sat, 10 Jul 2010 13:25:32 -0400
From: Dan Buch <daniel.b...@gmail.com>
To: the-fellowship-of-the-packag...@googlegroups.com
Subject: RBTools 0.2.1alpha needed for Mercurial repos (was Re: reviewboard)
> The names given in your ``[paths]`` section are significant. This is a
> recent addition to RBTools and deserves some documentation, but I'll
> summarize here. If, when trying to determine the repository type and
> metadata, ``post-review`` determines the repo to be of type "hg", remote
> paths are checked with the following names (in this order):
>
> * "reviewboard"
> * "origin"
> * "parent"
> * "default"
CORRECTION:
This behavior is only present in 0.2.1, which hasn't *yet* been released.
You'll be wanting to install the alpha 0.2.1 to get desired behavior when
working with Mercurial. Source distributions available here:
http://github.com/meatballhat/rbtools/downloads
> At one point the debug output of ``post-review`` included information
> about this heuristic, but such output was deemed too verbose and backed
> out.
>
> That said, this is an example ``[paths]`` section (for your local
> repository's ``.hg/hgrc``) that *should* work with ``post-review``::
>
> [paths]
> default = http://bitbucket.org/meatballhat/distutils2
> origin = http://bitbucket.org/tarek/distutils2
> ...
> zubin = http://bitbucket.org/zubin71/distutils2
> merwok = http://bitbucket.org/Merwok/distutils2
> ... other remotes ...
>
> >
> > > Also: if you upload the diff by hand, make sure you're not using the
> > > git diff format, as this would cause the server to 500.
> >
> > This is very bad. Non-git diff are really limited, they can’t describe
> > executable bit change, changes to binary files, renames, whereas git
> > diffs are equivalent to a native changeset and thus much more useful.
>
> I fully agree. git-style diffs are waaaay better. There is a
> server-side fix for this. I will update the ReviewBoard instance now
> that 1.5RC1 is out, which *supposedly* includes the fix.
----- End forwarded message -----
pgpllVVblwosW.pgp
Description: PGP signature
