On Thu, Sep 03, 2009 at 07:55:25AM +0100, Padraig O'Briain wrote:
> On 08/27/09 10:22, Padraig O'Briain wrote:
>> The webrev, http://cr.opensolaris.org/~padraig/ips-10539-v1/, fixes
>>
>> 10539 Package Manager hangs after canceling Update All
>
> Could someone have a look at this proposed change to api.py as it causes
> severe problems to the GUI?
Yes. Brock or Shawn should probably also look at this, but here are my
suggestions.
- lines 237 & 238: Since you've moved self.__reset_unlock() and
self.__activity_lock.release() out of conditional that follows this,
it probably makes sense to remove it here too, and just let it get
called from the common location.
- line 240: If you take the above suggestion, this should be an elif
not an if, I think.
-j
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
