webrev failure?

[Martijn Verburg]

Hi Dmitry,

Thanks for the hints! After following the forest extension install
instructions at
http://hg.openjdk.java.net/jdk7u/jdk7u4/raw-file/tip/README-builds.html#hg
and running "ksh ./make/scripts/webrev.ksh -N" I get:

*** failed to import extension forest from forest_extension/forest.py:
No module named repo
   SCM detected: mercurial
*** failed to import extension forest from forest_extension/forest.py:
No module named repo
*** failed to import extension forest from forest_extension/forest.py:
No module named repo
*** failed to import extension forest from forest_extension/forest.py:
No module named repo

 No outgoing, perhaps you haven't commited.
*** failed to import extension forest from forest_extension/forest.py:
No module named repo
*** failed to import extension forest from forest_extension/forest.py:
No module named repo
*** failed to import extension forest from forest_extension/forest.py:
No module named repo
      Workspace: /Users/karianna/Documents/workspace/jdk8_tl
      Output to: /Users/karianna/Documents/workspace/jdk8_tl/webrev
   Output Files:
	common/autoconf/configure
		 patch cdiffs udiffs sdiffs frames old new
	get_source.sh
		 patch cdiffs udiffs sdiffs frames old new
     index.html: Done.
Output to: /Users/karianna/Documents/workspace/jdk8_tl/webrev

So I'm a bit unsure about the "*** failed" messages but the "No
outgoing, perhaps you haven't commited." makes complete sense.

Do I need to alter forest.py in order to get the list of modules correct?

Cheers,
Martijn


On 11 August 2012 19:52, Dmitry Samersoff <Dmitry.Samersoff at oracle.com> wrote:
> Martin,
>
> 1. Make sure you have a forest extension
> 2. Try webrev -N to create webrev against your current workspace rather
>    than against remote repository
>
> -Dmitry
>
> On 2012-08-11 20:04, Martijn Verburg wrote:
>> Hi all,
>>
>> Apologies if these are the wrong mailing lists, but I'm assuming that
>> the webrev tool falls under this domain somewhat.
>>
>> In order to get patches into the OpenJDK as cleanly as possible, we're
>> looking to utilise webrev (since it's the std and all).
>>
>> Running "ksh ./make/scripts/webrev.ksh -f" over the latest jdk8/tl
>> source (no patches, but a full build was completed using build-infra)
>> and I got a host of errors:
>>
>> --------
>>
>>    SCM detected: mercurial
>> hg: unknown command 'foutgoing'
>> abort: cannot follow file not in parent revision: "Mercurial"
>> abort: cannot follow file not in parent revision: "basic"
>> abort: cannot follow file not in parent revision: "add"
>> abort: cannot follow file not in parent revision: "annotate"
>> ..
>> ..
>>
>> --------
>>
>> A separate, yet related set of errors are at
>> http://pastebin.com/q0tF1A4m for sake of brevity in this mail.
>>
>> I assume I'm running this tool incorrectly, I expected a result of
>> something like "You've changed nothing, nothing to se here move along
>> please" :-)
>>
>> Cheers,
>> Martijn
>>
>
>
> --
> Dmitry Samersoff
> Java Hotspot development team, SPB04
> * There will come soft rains ...
>
>