Jim,

This mainly looks good to me, see minor comments below ..

Paul

Jim Walker wrote:
> I updating meld to the current version.
> 
> Here's my webrev:
> http://cr.opensolaris.org/~jwalker/meld/6841654/
> 
> Note. The language translations continue to
> be a dynamic area for this app.

1. usr/src/cmd/meld/Makefile.sfw
     & usr/src/cmd/meld/install-sfw
     & usr/src/cmd/meld/meld.1
    Cosmetic: did you need to change these, the only thing
    that seem to have changed is the copyright year ?

2. usr/src/cmd/meld/meld.1
    Is there a reason why you have taken out the CDDL header stuff

END


Reply via email to