Paul Cunningham wrote:
>> Here's my webrev:
>> http://cr.opensolaris.org/~jwalker/meld/6841654/
> 
> 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 ?

The CDDL headers had double spaces which I removed.
Since this package is used as an example, it needs
to be the best shape possible :)

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

The sfw cteam has decided that man pages should use the
same license as the package being ported. Thus, no
CDDL headers anymore.

Cheers,
Jim

Reply via email to