On Fri 27 Jul 2012 at 04:26PM, [email protected] wrote:
> On Fri, Jul 27, 2012 at 03:57:16PM -0700, Dan Price wrote:
> >
> > I introduced a bug in our terminal handling when TERM isn't set or
> > is set to a bogus value. Here's a fix:
> >
> > https://cr.opensolaris.org/action/browse/pkg/dp/setupterm/
>
> This looks good. It took me a moment to realize that the exception had
> to be propogated up to the caller that's trying to instantiate the
> FancyUNIXProgressTracker, but now that I see how this works, it makes
> sense. The Fancy tracker is the only one that uses ttymode, right?
Yes-- correct. Thanks for the review.
-dp
--
Daniel Price, Solaris Kernel Engineering
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
