On Wed 08 Apr 2009 at 05:52PM, Shawn Walker wrote:
> Danek Duvall wrote:
> >An updated webrev is available at
> >
> >    http://cr.opensolaris.org/~dduvall/pkg-memoryerror2/
> >
> >which takes into account most of the comments made so far.  The remaining
> >one is Dan's suggestion to code the memory inspection in C, which I'm
> >reluctant to do this late in the release cycle.  If it's preferable, I can
> >rip those bits out entirely for now, and we can investigate a more stable
> >solution for the future.
> 
> I think this is fine, and I don't believe it is necessary to do the 
> memory inspection in C at this point.  If the memory introspection is 
> really a point of contention, I'd just use the generic message and not 
> bother providing the information at all since it's only a "nice-to-have."

I'll live.  It's better to have it than not.

        -dp

-- 
Daniel Price, Solaris Kernel Engineering    http://blogs.sun.com/dp
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to