Padraig O'Briain wrote:
I have not tested this yet but I have a query about the change at line
1519, i.e. the replacement of the call to
__process_after_search_failure().
There are other places in do_remote_search where
__process_after_search_failure. Do these also need to be changed?
I think when you get 0 results it is not correct to be using the
__process_after_search_failure(), the search succeeded it just didn't
find anything. The other instances are genuine failures and should be
reported as such, in those cases leaving the previous results seems OK
to me as the user will see an error dialog and realize something is wrong.
JR
Padraig
On 04/22/09 11:56, jmr wrote:
Hi here is a webrev to change the default PM Search All Repos to
match the CLI default search. This gives us a further x2 increase in
speed and makes the searching consistent as users are doing an exact
match search across all package actions in both cases. Users can
always add wild card '*' to the search term if they want and restrict
it to description if they need to as well, though this will slow down
searching.
There are some minor PM text changes and changes to Help required
because of this change, but l10n and Doc are happy to do this for B4.
Some suggested text for Help is in the 8335 bug.
8335 Search All Repositories should match CLI default search behavior
8336 PM Search All Repositories is not clearing old search result
when no matches found
webrev (not including OLH changes):
http://cr.opensolaris.org/~jmr/pm_8335_8336_search_all_22April_1140am/
JR
_______________________________________________
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
