Package: mutt-patched
Version: 1.5.20-9
Severity: normal
Hi,
The sidebar-sorted patch doesn't follow the locale settings when
sidebar_sort=yes. This is inconsistent with the rest of mutt which does
follow locale when sorting folders.
For example with 3 folders "A", "b", and "C", using LANG=en_GB.UTF-8:
Sorted folders with the mutt browser: A, b, C
Sorted folders with the mutt sidebar: A, C, b
Here is a patch that changes strcmp() to strcoll() to fix this sorting
problem.
Thanks,
Arnaud
-- System Information:
Debian Release: squeeze/sid
APT prefers testing
APT policy: (900, 'testing'), (800, 'unstable')
Architecture: i386 (i686)
Kernel: Linux 2.6.32-3-686-bigmem (SMP w/1 CPU core)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Versions of packages mutt-patched depends on:
ii libc6 2.11.2-2 Embedded GNU C Library: Shared lib
ii libcomerr2 1.41.12-1 common error description library
ii libgnutls26 2.8.6-1 the GNU TLS library - runtime libr
ii libgpg-error0 1.6-1 library for common error values an
ii libgpgme11 1.2.0-1.2 GPGME - GnuPG Made Easy
ii libgssapi-krb5-2 1.8.1+dfsg-5 MIT Kerberos runtime libraries - k
ii libidn11 1.15-2 GNU Libidn library, implementation
ii libk5crypto3 1.8.1+dfsg-5 MIT Kerberos runtime libraries - C
ii libkrb5-3 1.8.1+dfsg-5 MIT Kerberos runtime libraries
ii libncursesw5 5.7+20100313-2 shared libraries for terminal hand
ii libsasl2-2 2.1.23.dfsg1-5 Cyrus SASL - authentication abstra
ii libtokyocabinet8 1.4.37-6 Tokyo Cabinet Database Libraries [
ii mutt 1.5.20-9 text-based mailreader supporting M
mutt-patched recommends no packages.
mutt-patched suggests no packages.
-- no debconf information
Use strcoll() to sort the sidebar using the system's locale settings.
--- a/sidebar.c
+++ b/sidebar.c
@@ -63,7 +63,7 @@
needsort=0;
tmp = Incoming;
for ( ; tmp ; tmp=tmp->next ) {
- if (tmp->next != NULL && strcmp(tmp->path,
tmp->next->path) > 0) {
+ if (tmp->next != NULL && strcoll(tmp->path,
tmp->next->path) > 0) {
needsort=1;
prev = tmp->prev;
next = tmp->next;
