Ah OK, I tested on the failure condition with the as.p5i file you
posted, which showed the correct error dialog as expected. I did not
test on a good p5i file. I just have and both the original webrev and
the new one do not work. All throw up the error dialog.
JR
Michal Pryc wrote:
John,
Sorry for that, the v2 contained origins without "" which is necessary
for checking in __dict__ as well there was one unused variable:
http://cr.opensolaris.org/~migi/8410_v3
Michal
Michal Pryc wrote:
John,
New webrev:
http://cr.opensolaris.org/~migi/8410_v2
best
Michal
Michal Pryc wrote:
I did this for purpose.
Currently we don't have an api error to catch and we could have in
this place other errors then just the one described in 8410. I don't
want to solve other errors as this could cause not catching quite
important possible bugs and I do want to check if the error comed
from not existing URI. I am happy to remove try,except clausule, but
at some point this clauseule will need to appear in this place (once
the api will throw some error). Please let me know which way do you
prefer?
Michal
jmr wrote:
Built and tested it - works fine Michal.
One question on the webrev:
You are not using origins here at all. Either you should be using
the lack of origins to trigger the error or just use the exception
handler, but not both.
+ origins = False
+ repo = pub.selected_repository
+ if repo and origins in repo.__dict__ and
len(repo.origins) > 0:
+ origins = True
+ try:
+
repo_gui.webinstall_new_pub(self.w_webinstall_dialog, pub)
+ except AttributeError:
+ msg = _("Failed to add %s.\n") % pub
+ msg += _("No URI specified")
+ self.__error_occurred(
+ self.w_webinstall_dialog,
+ msg, gtk.MESSAGE_ERROR,
_("Repository Error"))
JR
Michal Pryc wrote:
Hello,
The webrev:
http://cr.opensolaris.org/~migi/8410_v1/
Fixes bug:
http://defect.opensolaris.org/bz/show_bug.cgi?id=8410
I have created sample p5i file which should show the bug.
http://cr.opensolaris.org/~migi/as.p5i
Please download this file and open using command:
pfexec packagemanager /path/to/as.p5i
With this fix proper error dialog will occur. There is no l10n
impact as those strings were already in other places of the PM.
best
Michal Pryc
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
