webrev.ksh on Ubuntu 12.04
Volker Simonis
volker.simonis at gmail.com
Thu Nov 14 01:12:12 PST 2013
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.petrov at 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://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