Huie-Ying Lee wrote:
   ... cut ...
> 
> The webrev is updated at http://cr.opensolaris.org/~hylee/pyopenssl.

This looks okay to me.

Just one minor-cosmetic comment ....

in usr/src/lib/pyopenssl/Makefile.sfw you could move the install bits 
(lines 64 to 86) out into a separate 'install-sfw' just so these don't 
clutter up the Makefile.sfw file

And does this pkg need a man page created for it?

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

Reply via email to