webrev.ksh on Ubuntu 12.04

[Dmitry Samersoff]

Dave,

Try

 webrev -N

- compare against local repository.

-Dmitry


On 2013-11-14 00:23, 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
> 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.