On Sat, Jun 26, 2010 at 09:17:48PM -0700, Christian Hammond wrote: > Not bitter at all! ;) Sadly, this is what happens when $DAYJOB and > real life get in the way for a little while. > > I'll be doing a new RBTools release with those fixes very soon.
Objectively Awesome! I can relate wrt ye olde $DAYJOB. I speak for my coworkers when I say that the many hours you've sunk into ReviewBoard and RBTools are very much appreciated. Cheers and thanks again, -- ~Dan > > Christian > > > On Saturday, June 26, 2010, Dan Buch <daniel.b...@gmail.com> wrote: > > > > On Fri, Jun 25, 2010 at 09:41:02AM -0700, Gordon Morehouse wrote: > >> On Jun 24, 9:12 pm, Dan Buch <daniel.b...@gmail.com> wrote: > >> > IIRC there's some naive logic in the remote Mercurial repo client inside > >> > Reviewboard that screws up the number/presence of some '/' chars. I > >> > think > >> > a recently accepted patch fixed the logic, but you can get it working > >> > with > >> > older versions of ReviewBoard by doing something like this: > >> > > >> > for the "Path" value: > >> > > >> > http://example.com/hgweb/AwesomeTown > >> > > >> > for the "Mirror path" value: > >> > > >> > http://example.com/hgweb/AwesomeTown/ > >> > > >> > And yes, it really is the trailing slash. No, this is not 1994. > >> > >> ...Unbelievable. Banging my head for days over a single slash (and no > >> documentation of the issue anywhere). Thank goodness there's a patch > >> in. > >> > >> I've submitted a documentation bug about this: > >> http://code.google.com/p/reviewboard/issues/detail?id=1699 > >> > >> My prior solution had been to mount the repo itself from our repo > >> server via sshfs, but after putting the HTTPS URL back in minus the > >> trailing slash it still worked. Thanks for that, because an > >> automounted sshfs was not exactly the greatest 'solution.' > >> > >> > If you haven't already tried, I can warn you that the current public > >> > release > >> > of RBTools doesn't *actually* support plain ol' Mercurial usage, > >> > although it > >> > *does* have good support for hgsubversion. Another recently-accepted > >> > patch > >> > fixes this (for the most part,) and a dist which includes it is available > >> > here: > >> > > >> > http://github.com/meatballhat/rbtools/downloads > >> > >> Thanks! I will investigate that. Any idea if it will be accepted > >> upstream? > > > > It has! It *only* took 3 months! ;-) > > http://reviews.reviewboard.org/r/1464/ > > I'm not bitter. I swear. > > > > When it will actually ship with canonical RBTools? not sure... > > chipx86 or trowbrds willing to weigh in? :) > > > >> > >> -- > >> Want to help the Review Board project? Donate today at > >> http://www.reviewboard.org/donate/ > >> Happy user? Let us know at http://www.reviewboard.org/users/ > >> -~----------~----~----~----~------~----~------~--~--- > >> To unsubscribe from this group, send email to > >> reviewboard+unsubscr...@googlegroups.com > >> For more options, visit this group at > >> http://groups.google.com/group/reviewboard?hl=en > > > > > > -- > > ~Dan > > -- > -- > Christian Hammond - chip...@chipx86.com > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > -- > Want to help the Review Board project? Donate today at > http://www.reviewboard.org/donate/ > Happy user? Let us know at http://www.reviewboard.org/users/ > -~----------~----~----~----~------~----~------~--~--- > To unsubscribe from this group, send email to > reviewboard+unsubscr...@googlegroups.com > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en
Description: PGP signature