Hi,

> as far as I can tell it looks ok. You basically updated CGI.pm files
> with latest version (or the version I see in 5.16).

It might be nice to put this information to the top of the patch.


> >http://jurassic.us.oracle.com/net/mogo/builds3/cmohrman/userland_perl-sec-7030196/webrev/

I like how you separated the patchlevel.h changes into separate patch.
Clever move!

LGTM (and I guess some backporting work to S11, S10, S9 and maybe S8 for
me :) )
-- 
        Vlad
_______________________________________________
userland-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/userland-discuss

Reply via email to