summaryrefslogtreecommitdiffstats
path: root/lib/libalpm/package.c
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-09-28 03:24:04 +0200
committerDan McGee <dan@archlinux.org>2011-09-28 03:24:04 +0200
commitea3c47825ea6a8cfd0e0d63fd7da0120fcacabc8 (patch)
tree00964c595ed8bc97e526af6eab4f965b2e16903a /lib/libalpm/package.c
parent69962184bb3cc313f744de6553ef31b4eb256a01 (diff)
downloadpacman-ea3c47825ea6a8cfd0e0d63fd7da0120fcacabc8.tar.gz
pacman-ea3c47825ea6a8cfd0e0d63fd7da0120fcacabc8.tar.xz
Revert "libalpm: compare pkgname with strcoll"
This commit was made with the intent of displaying "correctly" sorted package lists to users. Here are some reasons I think this is incorrect: * It is done in the wrong place. If a frontend application wants to show a different order of packages dependent on locale, it should do that on its own. * Even if one wants a locale-specific order, almost all package names are all ASCII and language agnostic, so this different comparison makes little sense and may serve only to confuse people. * _alpm_pkg_cmp was unlike any other comparator function. None of the rest had any dependency on anything but the content of the structs being compared (e.g., they only used strcmp() or other basic comparison operators). This reverts commit 3e4d2c3aa65416487939148828afb385de2ee146. Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/package.c')
-rw-r--r--lib/libalpm/package.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c
index 11bf01a8..a5ff238f 100644
--- a/lib/libalpm/package.c
+++ b/lib/libalpm/package.c
@@ -641,7 +641,7 @@ int _alpm_pkg_cmp(const void *p1, const void *p2)
{
const alpm_pkg_t *pkg1 = p1;
const alpm_pkg_t *pkg2 = p2;
- return strcoll(pkg1->name, pkg2->name);
+ return strcmp(pkg1->name, pkg2->name);
}
/* Test for existence of a package in a alpm_list_t*