On Sat, Sep 05, 2009 at 07:48:44AM +0100, Padraig O'Briain wrote: > [email protected] wrote: >> The second of my two comments doesn't appear to be addressed by this >> webrev. The conditional on line 237 (in the newest webrev) should be an >> elif, so you don't log two different errors in this code path. > > Sorry; missed that comment. I have spun a new webrev: > http://cr.opensolaris.org/~padraig/ips-10539-v3/
This looks good. Thanks for fixing this. -j _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
