LGTM
Michal
Padraig O'Briain wrote:
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
