On 04/ 5/10 06:14 PM, Shawn Walker wrote:
On 04/ 5/10 08:10 PM, Brock Pytlik wrote:
On 04/ 2/10 07:51 PM, Brock Pytlik wrote:
Webrev:
http://cr.opensolaris.org/~bpytlik/ips-15482-v1/

Bug:
15482 Fix for bug 6177 breaks bui search

This fixes BUI search so that it works again. I think I covered all
the bases during hand testing (actions/packages, latest versions/all
versions), but I'd appreciate suggestions for other testing I should
do on it.

Thanks,
Brock

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
Update webrev with tests now to make sure basic BUI searching works:
http://cr.opensolaris.org/~bpytlik/ips-15482-v2/

So I just realised that this means that the return value of pkg.server.api.CatalogInterface.search() changed incompatibly. That means that doc/server_api_versions.txt needs to be updated, the pkg.server.api.CURRENT_API_VERSION needs to be bumped in all the templates, etc.

Once that's done, I'm happy.

Thanks for reminding me the server api was versioned as well. I believe I've made the changes needed,
but just to double check, here's another webrev:
http://cr.opensolaris.org/~bpytlik/ips-15482-v3/

Thanks,
Brock

-Shawn

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

Reply via email to