file.py: line 17: think there's a typo with "replace_requiredd" Other than that, LGTM.
Brock Brad Hall wrote: > On Tue, Sep 16, 2008 at 04:43:02PM -0700, [EMAIL PROTECTED] wrote: > >> Brad, >> >> >>> The new webrev is posted at: http://cr.opensolaris.org/~bhall/bug-388-1/ >>> >> This webrev looks a lot better. Thanks for making these changes. A >> dumb question: why is there no repair method for hardlink.py? >> > > Oops, good catch. Updated webrev in place. > > >> Other than that, this looks good to me. >> > > Great, thanks again for reviewing. > > -Brad > _______________________________________________ > pkg-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/pkg-discuss > _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
