I see. I was using the "hg postreview" which does not have the --diff-filename parameter.
The "post-review" command does exist but it isn't set up (because we never use it) and so it can't find the mercurial repository: The repository path "http://cm1/hg/sandbox/v01.0/int"; is not in the list of known repositories on the server. Does the mercurial postreview extension need an enhancement? Perhaps a more generic enhancement is to allow different diff utilities to be declared for different filename extensions, e.g. something like this in the .hgrc: [reviewboard] server = http://myserver/reviews difftool = .xml /home/myuserid/bin/xmldiff difftool = .bob /is/your/uncle If the extension is not in the difftool list, the default mercurial diff is used. That way, I still use "hg postreview" and the correct diff utility is invoked automatically for me. No external files, no manual steps, etc. John On Tue, Jan 4, 2011 at 11:48 AM, Christian Hammond <chip...@chipx86.com>wrote: > Hi John, > > You want to use plain post-review with the --diff-filename parameter, > and specify the diff file. > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > > On Mon, Jan 3, 2011 at 5:09 PM, J Arrizza <cppge...@gmail.com> wrote: > > I've searched the archives and google with no luck. > > I have been able to get a mercurial extdiff called 'xmldiff' to generate > a > > nice, clean diff of my xml file. Now I'd like to be able to post the same > > diff to RB. My preference would be to run the same diff script using > > hg postreview <some-magic-occurs> > > but I can live with a manual version: > > hg xmldiff -r nnn some.xml > some.diff > > hg postreview <some-magic-occurs> some.diff > > Is there a way to do either of these? > > John > > > > -- > > 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<reviewboard%2bunsubscr...@googlegroups.com> > > For more options, visit this group at > > http://groups.google.com/group/reviewboard?hl=en > > -- > 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<reviewboard%2bunsubscr...@googlegroups.com> > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en -- 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
