Does "hg outgoing"/"hg incoming" work? This doesn't seem to be webrev.ksh problem.
On Wed, Nov 13, 2013 at 10:36 PM, Anthony Petrov <anthony.pet...@oracle.com> wrote: > 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://anth...@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 >> >
