Some comments ...

--- Maxim Kartashev <Maxim.Kartashev at Sun.COM> wrote:

> And here's a copy visible for everyone:
> http://cr.opensolaris.org/~perezzz/cvs.webrev/

1. MINOR pkgdefs/SUNWcvs/depend
   Shouldn't the 'copyright' statement be after the
   'CDDL Header'

2. pkgdefs/SUNWcvs/prototype_com
   Shouldn't the 'man' page bits be in a seperate
   package

3. cmd/cvs/install-cvs
   Does 'MANSCRIPT' get used ?

Otherwise it looks okay to me

Paul


Reply via email to