I have checked the GUI related changes and they seem to be fine.
However, you forgot to patch updatemanager.py.
Here is a suggested patch for it.
Is this OK or should it always use sys.argv[0] for actual_cmd?
Padraig
On 09/ 7/10 02:18 AM, Brock Pytlik wrote:
webrev:
http://cr.opensolaris.org/~bpytlik/ips-16852-v1/
Bugs:
16852 pkg should look at its image for certs if active image lacks
certs
16919 pkg should look to image relative directories for certs
Thanks,
Brock
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
diff -r e59121919214 src/updatemanager.py
--- a/src/updatemanager.py Mon Sep 06 16:21:16 2010 +0100
+++ b/src/updatemanager.py Tue Sep 07 09:18:30 2010 +0100
@@ -68,6 +68,7 @@
class Updatemanager:
def __init__(self, image_directory, application_path, nice, check_all,
check_cache):
+ self.__actual_cmd = application_path
global_settings.client_name = gui_misc.get_um_name()
self.api_lock = nrlock.NRLock()
self.image_dir_arg = image_directory
@@ -123,7 +124,8 @@
def __do_image_update(self):
installupdate.InstallUpdate([], self,
- self.image_dir_arg, action = enumerations.IMAGE_UPDATE,
+ self.image_dir_arg, self.__actual_cmd,
+ action = enumerations.IMAGE_UPDATE,
parent_name = _("Update Manager"),
pkg_list = [gui_misc.package_name["SUNWipkg"],
gui_misc.package_name["SUNWipkg-gui"],
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
