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