How to create a webrev from the top level directory

[Kelly O'Hair]

On Dec 20, 2011, at 8:46 PM, Mario Torre wrote:

> Maybe webrev should also be updated to not need the forest extension.

Yup. That's my feeling, however the last time I looked at this script I ran running from the room
with a severe case of intestinal discomfort. :^(

> 
> Anyway, other than for build scripts, or other similar things that
> naturally spawn across the whole forest, it's usually better to only
> generate webrev for the specific tree you are targeting.

usually, that is true.

-kto

> 
> Mario
> 
> Il giorno 21-dic-2011 02.12, "Pete Brunet" <peter.brunet at oracle.com> ha
> scritto:
> 
>> Thanks Kelly, I guess either this page needs to be updated to give
>> instructions on installing Forest or on how to do a webrev without Forest.
>> 
>> http://wikis.sun.com/display/OpenJDK/Mac+OS+X+Port
>> 
>> Pete
>> 
>> On 12/20/11 6:10 PM, Kelly O'Hair wrote:
>>> You need the Mercurial forest extension working, and I usually use this:
>>> 
>>>    webrev -f -N -o somedir
>>> 
>>> The -N says I only want a webrev of the edited working set files.
>>> 
>>> Yeah I know, the forest extension is a pain. Sorry.
>>> 
>>> -kto
>>> 
>>> On Dec 20, 2011, at 7:30 AM, Pete Brunet wrote:
>>> 
>>>> I've been asked to create a webrev.  According to
>>>> http://openjdk.java.net/guide/webrevHelp.html
>>>> from the toplevel directory (macosx-port) I should execute
>>>> ksh ./make/scripts/webrev.ksh -f
>>>> but that produced a lot of error messages so I did
>>>> cd jdk
>>>> ksh ../make/scripts/webrev.ksh
>>>> and that created my webrev
>>>> 
>>>> What is the proper command to get it to work from the parent directory?
>>>> 
>>>> Pete
>>