webrev.ksh on Ubuntu 12.04

Anthony Petrov anthony.petrov at oracle.com
Wed Nov 13 13:36:43 PST 2013


Hi Dave,

The .hg/hgrc in your repo seems to have an incorrect default-push path. 
Here's mine for example:

[paths]
default = http://hg.openjdk.java.net/jdk8/awt/jdk
default-push = ssh://anthony@hg.openjdk.java.net/jdk8/awt-gate/jdk

Your hgrc should look somewhat similar (just replace the user name). I 
think you forgot to run the hg defpath task after you've cloned your 
repo. Alternatively, don't commit your changes and let webrev generate 
the webrev based on uncommitted modifications.

--
best regards,
Anthony

On 11/14/2013 12:23 AM, Dave Pointon wrote:
> Hi all ,
>
> Can anyone tell me if they've experienced any problems with webrev.ksh
> running on Ubuntu 12.04 with Mercurial V2.7.2.
>
> Although I _have_ a committed change, I see this :
>
> $ ksh ./make/scripts/webrev.ksh
>     SCM detected: mercurial
> hg outgoing: invalid arguments
> hg: parse error: can't negate that
> expr: syntax error
>
>   No outgoing, perhaps you haven't commited.
>        Workspace: /home/dpointo8/work/repos/Mercurial/composite.OJDK-889
> Compare against: default =
> ssh://jtc-hg.hursley.ibm.com//home/hg/jtchg-ibm-adm/composite/80/jcl/head
>        Output
> to: /home/dpointo8/work/repos/Mercurial/composite.OJDK-889/webrev
>     Output Files:
>       index.html: Done.
> Output to: /home/dpointo8/work/repos/Mercurial/composite.OJDK-889/webrev
>
> and get an empty report as a result.
>
> Running ksh -x reveals the problem to be a command of the form 'hg
> outgoing default = /home/path/to/repo' which, self-evidently, mercurial
> can't and doesn't, make any sense of.
>
> My local fix at the moment is ...
>
> @@ -1957,7 +1957,12 @@
>               fi
>           fi
>       fi
> -    #
> +
> +
> +    # Ensure that the path is the only thing in the workspace path i.e.
> remove
> +    # any extraneous prefix such as 'default ='
> +    OUTPWS="`echo $OUTPWS | $AWK '{print $NF}'`"
> +
>       # OUTPWS may contain username:password, let's make sure we remove
> the
>       # sensitive information before we print out anything in the HTML
>       #
>
>
> Any thoughts greatly appreciated.
>
> TIA & rgds ,
>
> --
> Dave Pointon FIAP MBCS
>
> Now I saw, tho' too late, the folly of beginning a work before we count
> the cost and before we we judge rightly of our strength to go thro' with
> it - Robinson Crusoe
>


More information about the jdk8-dev mailing list