Re: Cannot create webrev

2013-03-10 Thread Dmitry Samersoff
Ben, Try something like this: mkdir webrevs ( cd jdk webrev -N -o ../webrevs/jdk_webrev ) ( cd hotspot webrev -N -o ../webrevs/hotspot_webrev ) etc -Dmitry On 2013-03-10 00:36, Ben Evans wrote: > Thanks Jon. > > I thought forest was deprecated in favour of get_source.sh ? It > certainly

Re: Cannot create webrev

2013-03-10 Thread Ben Evans
OK, fixed. Having forest installed on this version of Mercurial causes webrev to not run correctly & removing it entirely seems to be necessary. Thanks, Ben On Sat, Mar 9, 2013 at 10:17 PM, Jonathan Gibbons wrote: > Generally, yes, forest is deprecated. > > If you're just working within a sing

Re: Cannot create webrev

2013-03-09 Thread Jonathan Gibbons
Generally, yes, forest is deprecated. If you're just working within a single repo, run webrev in that repo. -- Jon On 03/09/2013 12:36 PM, Ben Evans wrote: Thanks Jon. I thought forest was deprecated in favour of get_source.sh ? It certainly won't work with 2.5.1 I have changes to a single

Re: Cannot create webrev

2013-03-09 Thread Ben Evans
Thanks Jon. I thought forest was deprecated in favour of get_source.sh ? It certainly won't work with 2.5.1 I have changes to a single file (plus a new test-ng test for the changes) - how best to persuade webrev.ksh to do the right thing from within the jdk repo? Thanks, Ben On Sat, Mar 9, 20

Re: Cannot create webrev

2013-03-09 Thread Chris Hegarty
On 9 Mar 2013, at 19:22, Jonathan Gibbons wrote: > hg: unknown command 'foutgoing' > > Means you either don't have the forest extension installed or it doesn't work > on Mercurial 2.5.1 Right. If you truly need to generate a webrev across multiple repos, and your hg doesn't play nicely with

Re: Cannot create webrev

2013-03-09 Thread Jonathan Gibbons
hg: unknown command 'foutgoing' Means you either don't have the forest extension installed or it doesn't work on Mercurial 2.5.1 -- Jon On 03/09/2013 10:54 AM, Ben Evans wrote: Hi, I can't seem to create a webrev on Mac 10.7, Mercurial 2.5.1 - the webrev script fails with the below output