Just saw your comment Shawn, thanks. I have just submitted a followup to strip the keys(), having checked that it works without it.

JR

On 16/05/2011 16:22, Shawn Walker wrote:
On 05/16/11 08:15 AM, Brock Pytlik wrote:
On 05/16/11 03:41, John Rice wrote:
HI - here is a followup webrev to:
18269 CLI should provide a way to view a publishers signature policy,
to fix the following issue:

18343 pkg publisher pub is broken when pub is a system publisher
http://cr.opensolaris.org/~jmr/cli_18343_pkg_publisher_pub_broken_may16_1140am


As pub.properties is an _InternalProxy dictionary wrapper object which
no has_key() method I am checking the keys() list. Might be better to
add the has_keys() to the _InternalProxy, but not in this simple
followup webrev.

JR


LGTM. Seems to me there should really be something in the test suite
testing this behavior.
Brock

You should be able to say:

if "signature-required-names" in pub.properties:

...the .keys() shouldn't be needed. If it is, there's something with the properties dictionary.

Apologies for missing this the first time,
-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

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

Reply via email to