Dabek - here's the new webrev incorporating your suggestions:

http://cr.opensolaris.org/~jmr/um_5172_v2_Nov18/

Michal has just tested it and gets the following output:

ERROR
Update All unexpected error:
Could not retrieve filelist from 'http://osol-re.sfbay.sun.com'
URLError reason: (8, 'node name or service name not known')

Which is the same as the CLI.

JR

Danek Duvall wrote:
> On Tue, Nov 18, 2008 at 01:12:41AM +0000, jmr wrote:
>
>   
>> Webrev: http://cr.opensolaris.org/~jmr/um_5172_v1_Nov18/
>>     
>
> That isn't strictly correct -- uex.args could be an object that evaluates
> True, but still isn't subscriptable, like, say, an integer.
>
> Why not have an except clause to handle EnvironmentError and test
> uex.errno?  That's how most of the rest of our code deals with this.
>
> Danek
>   

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

Reply via email to