On 03/08/12 02:19, Saurabh Vyas wrote:
On 02/20/12 23:38, Danek Duvall wrote:
...
This seems to be a difference between looking at an image without any
packages installed and one with, as if I install a package into my test
image first, I get the r.origins stack trace.
I think maybe get_highest_ranked_publisher() should always return None if
there are no configured publishers, regardless of whether there are any
installed packages, as there's no sense of publisher ranking if there is no
publisher configuration.
Tested my fix on following scenarios :
- No publisher configured for the current image
root@onnv175b-nd:~# pkg publisher -P
PUBLISHER TYPE STATUS URI
- Image with No package
root@onnv175b-nd:~# pkg -R /var/tmp/img publisher -P
PUBLISHER TYPE STATUS URI
works as expected
New webrev :
https://cr.opensolaris.org/action/browse/pkg/saurabhv/CR-7141684-rev3/webrev/
NOTE : I remove one test inside : test_publisher_apis (api/t_api.py)
as now get_highest_ranked_publisher() returns 'None' when there are no
configured publishers.
Please don't make this change. I specifically had
get_highest_ranked_publisher() return the first publisher because of an
issue with packagemanager. That's also why I had the unit test for it.
I believe this will break search functionality in the packagemanager.
-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
