On Fri, 2013-11-15 at 20:55 +0400, Anthony Petrov wrote: > Hi Dave, > > Please post the exact and complete output of the script on the console. > Also, please post the content of your .hg/hgrc file in the repo you're > trying to generate a webrev for. >
$ ( ksh make/scripts/webrev.ksh 2>&1 | tee webrev.ksg.out ; echo ; cat .hg/hgrc ) 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 [paths] default=ssh://jtc-hg.hursley.ibm.com//home/hg/jtchg-ibm-adm/composite/80/jcl/head > If you clone a new fresh repo from any of the OpenJDK forests, does it > also fail to generate a webrev for uncommitted changes? > Yes, I'm afraid so ... $ ( hg status ; ksh make/scripts/webrev.ksh 2>&1 | tee webrev.ksg.out ; echo ; cat .hg/hgrc ) M configure SCM detected: mercurial No outgoing, perhaps you haven't commited. Workspace: /home/dpointo8/work/repos/Mercurial/upstream Compare against: http://hg.openjdk.java.net/jdk8/jdk8 Output to: /home/dpointo8/work/repos/Mercurial/upstream/webrev Output Files: configure patch cdiffs udiffs wdiffs[fail] sdiffs frames old new index.html: Done. Output to: /home/dpointo8/work/repos/Mercurial/upstream/webrev [paths] default = http://hg.openjdk.java.net/jdk8/jdk8 <snip> HTH , 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