webrev.ksh on Ubuntu 12.04

[Dave Pointon]

Hi Anthony ,

On Thu, 2013-11-14 at 22:55 +0400, Anthony Petrov wrote:
> I never commit my changes when generating a webrev, and I've never had a 
> problem with that (for many years, that is). Yes, it issues a warning, 
> but the rest of the script works fine. How exactly does it give up in 
> your case?
> 

'hg outgoing: invalid arguments
hg: parse error: can't negate that
expr: syntax error'

is the indicator - the nett result of which is that an empty report is
generated along with the above errors.

If I don't commit a change to my local clone, then my (modified)
webrev.ksh fails to find anything on which to report - and thus produces
an empty report but no errors.

> PS. Please use Reply-All instead of Reply when replying to mailing list 
> messages.
> 

Reply all duly used :-)

> --
> best regards,
> Anthony
> 
> On 11/14/2013 04:44 AM, Dave Pointon wrote:
> > Awesome, TFT Anthony,
> >
> > I didn't know about the hg defpath comand, I shall investiagte further,
> > but until then, if I don't commit, webrev complains (that I might need
> > to commit) and gives up earlier - maybe webrev needs better help by way
> > of properly explaining what the options actually do.
> >
> > BTW, the default path represents our (the IBM) clone path.
> >
> > Thanx again ,
> >
> > Best rgds ,
> > Dave P
> >
> > On Thu, 2013-11-14 at 01:36 +0400, Anthony Petrov 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
> >>>
<snip>

Best 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