On Wed, Jul 08, 2009 at 05:13:06PM -0700, Danek Duvall wrote:
> On Wed, Jul 08, 2009 at 05:05:06PM -0700, [email protected] wrote:
> 
> >     http://cr.opensolaris.org/~johansen/webrev-9929/
> 
> I don't understand why you split captive_portal_test() out into a wrapper
> and private backend.

I can re-coalesce.

> I wonder what "pubentication" means, too.

Hah.  This is leftover from the :s/auth/pub/ when authorities got
changed to publishers.  I caught a "puborties" in a comment recently
too.  I guess I missed this one.  Thanks.

> Does get_datastream() need a check, too?

Not sure that it does.  The get_datatstream is a generic call and it
isn't used by very many transport operations right now.  I can add one
here, but since the caller has to catch all of the other exceptions
coughed up by the transport, I expected that this is only going to be
used in special cases, and when it is used, the caller will know whether
he/she needs to run this test (or not).

-j
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to