Srirama,

Srirama Sharma wrote:
   ... cut ...
> 
> Have updated the webrev at 
> http://cr.opensolaris.org/~srirama/simplewbem/ with all the above changes.
> 
> Please review.

This mainly looks good to me now, except ...

 >
 >> 2. usr/src/cmd/simplewbem/*.1
 >>     Did you write these man pages, if so you might need
 >>     to add the Sun copyright and CDDL stuff.
 >>
 >  and have removed the  sunman-stability section.

don't you need that, either directly in the *.1 files or using the 
sunman-stability modification method with '_install M ...'in 
simplewbem/install-sfw ?

Paul
-- 
----------------------------------------------------------------------
Paul Cunningham
Software Engineer
Tadpole Business Unit

Reply via email to