Your message dated Sat, 06 Nov 2010 13:40:39 +0100
with message-id <[email protected]>
and subject line Re: Bug#602605: unblock: apt-xapian-index/0.41
has caused the Debian Bug report #602605,
regarding unblock: apt-xapian-index/0.41
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
602605: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=602605
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: [email protected]
Usertags: freeze-exception

Please unblock package apt-xapian-index. It fixes all severe bugs found
after my talk at the MiniDebconf:

apt-xapian-index (0.41) unstable; urgency=low

  * Fixed typo in dbus config
  * Fixed DeprecationWarning in set_sort_by_value. Closes: #601880.
  * Reset cached sort value if --sort is not provided for search.
    Closes: #601881.

 -- Enrico Zini <[email protected]>  Sat, 06 Nov 2010 11:25:07 +0000

A mini feature crept in as part of the fix to #601880, I hope you don't
mind. The deprecation warning required the 2-parameters form of
set_sort_by_value to be used; however, since we were missing reverse
sort (without which sorting by some parameters like package size of age
would be useless), I added the missing parameter with its proper
purpose.

The relevant commit is this:

  commit 6d3d2d4ae74d802b3ac1b859c33924feddc492f8
  Author: Enrico Zini <[email protected]>
  Date:   Sat Oct 30 15:29:55 2010 +0200
  
      Allow to reverse sort order
  
  diff --git a/axi-cache b/axi-cache
  index 69eee8a..531e3a9 100755
  --- a/axi-cache
  +++ b/axi-cache
  @@ -249,8 +249,11 @@ if not helponly:
                   # TODO: didn't this use to work?
                   #self.enquire.set_cutoff(int(self.get_cache_last("cutoff", 
60)))
   
  +                reverse = sort[0] == '-' or sort[-1] == '-'
  +                sort = sort.strip('-')
  +
                   # Sort by the requested value
  -                self.enquire.set_sort_by_value(values[sort])
  +                self.enquire.set_sort_by_value(values[sort], reverse)
   
           def get_matches(self, first=None, count=20):
               if first is None:
  @@ -329,7 +332,7 @@ class Cmdline(object):
           self.parser = Parser(self, usage="usage: %prog [options] command 
[args]",
                           version="%prog "+ VERSION,
                           description="Query the Apt Xapian index.ENDDESC")
  -        self.parser.add_option("-s", "--sort", help="sort by the given 
value, as listed in %s" % axi.XAPIANDBVALUES)
  +        self.parser.add_option("-s", "--sort", help="sort by the given 
value, as listed in %s. Add a '-' to reverse sort order" % axi.XAPIANDBVALUES)
           self.parser.add_option("--tags", action="store_true", help="show 
matching tags, rather than packages")
           self.parser.add_option("--tabcomplete", action="store", 
metavar="TYPE", help="suggest words for tab completion of the current command 
line (type is 'plain' or 'partial')")
           self.parser.add_option("--last", action="store_true", help="use 
'show --last' to limit tab completion to only the packages from the last search 
results")

unblock apt-xapian-index/0.41

-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash



--- End Message ---
--- Begin Message ---
On 11/06/2010 01:25 PM, Enrico Zini wrote:
> Package: release.debian.org
> Severity: normal
> User: [email protected]
> Usertags: freeze-exception
> 
> Please unblock package apt-xapian-index. It fixes all severe bugs found
> after my talk at the MiniDebconf:
> 
> apt-xapian-index (0.41) unstable; urgency=low
> 
>   * Fixed typo in dbus config
>   * Fixed DeprecationWarning in set_sort_by_value. Closes: #601880.
>   * Reset cached sort value if --sort is not provided for search.
>     Closes: #601881.
> 
>  -- Enrico Zini <[email protected]>  Sat, 06 Nov 2010 11:25:07 +0000
> 

Unblocked.

Thanks for your work,

-- 
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/


--- End Message ---

Reply via email to