Yikes, stupid search and replace error.  Fixed and updated webrev.

Thanks for reviewing.

-Brad

On Mon, Sep 22, 2008 at 05:31:02PM -0700, Brock Pytlik wrote:
> 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

Reply via email to