Re: remote Mercurial repo support?
Thanks both of you. I see the patch has been added as of RBTools 0.2.1 alpha 0, which I'm now using. Might want to take a peek at http://code.google.com/p/reviewboard/issues/detail?id=1708 though ;) -- 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
Re: remote Mercurial repo support?
On Thu, Jul 01, 2010 at 01:54:54PM -0700, Gordon Morehouse wrote: Thanks both of you. I see the patch has been added as of RBTools 0.2.1 alpha 0, which I'm now using. Might want to take a peek at http://code.google.com/p/reviewboard/issues/detail?id=1708 though ;) Clearly the Mercurial features still need ... more work ;) /me invokes patches welcome, runs away -- ~Dan -- 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 pgprCHIQKG3wD.pgp Description: PGP signature
Re: remote Mercurial repo support?
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 pgp2JCf1X6JRU.pgp Description: PGP signature
Re: remote Mercurial repo support?
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 pgp8SHnjWClK9.pgp Description: PGP signature
Re: remote Mercurial repo support?
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. 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
Re: remote Mercurial repo support?
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? -- 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
remote Mercurial repo support?
Hello, The manual page about repositories contains no information whatsoever about Mercurial. Is it possible to configure a remote Mercurial repo either via HTTP(S) or SSH, or does Review Board require disk access to the repository? If it's possible via HTTP, what is the path to use? The path to the human-browseable repo results in Review Board failing to apply diffs because it's attempting to apply them to HTML output from the browseable repository. -- 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
Re: remote Mercurial repo support?
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. 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 Cheers, -- ~Dan On Thu, Jun 24, 2010 at 02:06:08PM -0700, Gordon Morehouse wrote: Hello, The manual page about repositories contains no information whatsoever about Mercurial. Is it possible to configure a remote Mercurial repo either via HTTP(S) or SSH, or does Review Board require disk access to the repository? If it's possible via HTTP, what is the path to use? The path to the human-browseable repo results in Review Board failing to apply diffs because it's attempting to apply them to HTML output from the browseable repository. -- 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 pgpprfG3YMtVe.pgp Description: PGP signature