diff options
| author | Dan McGee <dan@archlinux.org> | 2011-09-27 20:24:04 -0500 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-09-27 20:24:04 -0500 |
| commit | ea3c47825ea6a8cfd0e0d63fd7da0120fcacabc8 (patch) | |
| tree | 00964c595ed8bc97e526af6eab4f965b2e16903a /lib/libalpm | |
| parent | 69962184bb3cc313f744de6553ef31b4eb256a01 (diff) | |
| download | 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')
| -rw-r--r-- | lib/libalpm/package.c | 2 |
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* |