I have respun the webrev again to correct installupdate.py:
http://cr.opensolaris.org/~padraig/ips-9556-v3/

Padraig

On 07/01/09 09:15, Michal Pryc wrote:
Padraig,

The proper link:

http://cr.opensolaris.org/~padraig/ips-9566-v2/

I think that this import should be also in the installupdate.py:
http://cr.opensolaris.org/~padraig/ips-9566-v2/src/gui/modules/installupdate.py.wdiff.html

Michal

Padraig O'Briain wrote:
I have respun the webrev http://cr.opensolaris.org/~padraig/ips-9556-v2/ to add "import os"
to beadmin..py.

Padraig

On 06/29/09 14:16, Michal Pryc wrote:
Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-9556-v1/, fixes
9556 Failure in UpdateManager when there is no space

The change to beadmin.py is to get the glade file the same way as installupdate.py does. The change to installupdate.py is to use os..path.join when building up the glade file name.

LGTM

Michal

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

Reply via email to