FS#27221 - Collation in Pacman and “makepkg” are different
Attached to Project:
Pacman
Opened by Marty (vadmium) - Tuesday, 22 November 2011, 03:40 GMT
Last edited by Allan McRae (Allan) - Monday, 05 December 2011, 00:50 GMT
Opened by Marty (vadmium) - Tuesday, 22 November 2011, 03:40 GMT
Last edited by Allan McRae (Allan) - Monday, 05 December 2011, 00:50 GMT
|
Details
Summary and Info:
Git commit ea3c478 reverted commit 3e4d2c3 “libalpm: compare pkgname with strcoll”. Now Pacman compares package names with “strcmp” instead of “strcoll”. However the reverted commit seems closely tied to the commit immediately following it: d7dccd5 “makepkg: remove unnecessary use of sort”. So I suggest reverting the second commit to restore the “sort” call. An alternative might be to set LC_COLLATE=C for the two “comm” calls. Steps to Reproduce: * Build a package with the “makepkg --rmdeps” option, using LC_COLLATE=en_US.UTF-8 or similar * Presumably if you have enough hyphenated packages installed it will trigger the fault. For example I have the following packages installed, but “avrdude” should come before “avr-gdb” according to the “sort” and “comm” commands. avahi avr-gdb avr-libc avrdude babl Example messages: ==> Leaving fakeroot environment. ==> Finished making: edumagnet 0.11-1 (Tue Nov 22 03:19:15 UTC 2011) comm: file 1 is not in sorted order comm: file 2 is not in sorted order comm: file 1 is not in sorted order comm: file 2 is not in sorted order ==> ERROR: An unknown error has occurred. Exiting... ==> Removing installed dependencies... checking dependencies... Targets (1): python-m2crypto-0.21.1-1 Workaround: LC_COLLATE=C makepkg |
This task depends upon
Closed by Allan McRae (Allan)
Monday, 05 December 2011, 00:50 GMT
Reason for closing: Fixed
Additional comments about closing: http://projects.archlinux.org/pacman.git /commit/?h=maint&id=a521cea9
Monday, 05 December 2011, 00:50 GMT
Reason for closing: Fixed
Additional comments about closing: http://projects.archlinux.org/pacman.git /commit/?h=maint&id=a521cea9
Comment by Allan McRae (Allan) -
Tuesday, 22 November 2011, 06:20 GMT
Patch