[MacPorts] #29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2 ------------------------------------------+--------------------------------- In revision r64476 to address ticket #23959, sorting was changed for all platforms although the bug only occurs when building & using glib2 on Snow Leopard (x86_64). Could the patch added in r64476 be made conditional, so that at least on Leopard, sorting (of fonts, filenames, etc.) uses again the same sort order like before (alphabetical, not case-sensitive), like e.g. in the Finder and native file dialogs? Upstream bug report for glib is [https://bugzilla.gnome.org/show_bug.cgi?id=612019 Bug 612019 - Alphabetical sorting not working in OSX Snow Leopard 10.6.x] Attached patch was used to quickly test sorting order without changes to 'glib/gunicollate.c' (it patchfiles-deletes the earlier added patchfile if platform is 'darwin i386') - possibly the test could be done in the patch instead? -- Ticket URL: <https://trac.macports.org/ticket/29051> MacPorts <http://www.macports.org/> Ports system for Mac OS
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2, glib2-devel ------------------------------------------+--------------------------------- Changes (by ryandesign@…): * port: glib2 => glib2, glib2-devel Comment: Perhaps you don't realize that "platform i386" applies on ''all'' Intel architectures -- i386 and x86_64 (just as "platform powerpc" applies on all PowerPC architectures -- ppc and ppc64). -- Ticket URL: <https://trac.macports.org/ticket/29051#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2, glib2-devel ------------------------------------------+--------------------------------- Comment(by suv-sf@…): Main point of the ticket was to discuss if the patch from r64476 could be applied selectively (only on Snow Leopard, or those system where file sorting was broken - AFAICT the bug never affected Leopard) - whatever Portfile magic is best, I don't know, I'm still learning ;) . I tested building glib2 on Mac OS X 10.5.8 i386 without having the patch to 'glib/gunicollate.c' applied (i.e. having <CoreServices/CoreServices.h> included), and the sorting e.g. in GTK+ font selectors and file browsers is what users expect on Mac OS X (not case-sensitive) and how it used to be on Leopard, before the problems on Snow Leopard had been noticed and addressed by not including the CoreServices header file. -- Ticket URL: <https://trac.macports.org/ticket/29051#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2, glib2-devel ------------------------------------------+--------------------------------- Comment(by jmr@…): Are you sure it works on 10.5/x86_64 ( or indeed ppc64) and doesn't work on 10.6/i386? It doesn't look like that would be the case going by the discussion in the upstream bug. -- Ticket URL: <https://trac.macports.org/ticket/29051#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2, glib2-devel ------------------------------------------+--------------------------------- Comment(by suv-sf@…): Replying to [comment:3 jmr@…]:
Are you sure it works on 10.5/x86_64 (or indeed ppc64) and doesn't work on 10.6/i386? It doesn't look like that would be the case going by the discussion in the upstream bug.
No, I'm not sure about 10.5.8/x86_64 (or ppc64) nor 10.6/i386 - I only have access to 10.5.8/i386. The patchfile added in r64476 has this comment: {{{/* Carbon is not available on 64-bit */}}}, but nothing is done to test for 64-bit, and {{{HAVE_CARBON}}} is undefined anyway (originally this was decided by the maintainer for the packaged versions of GIMP on OS X, to have GIMP behave consistently on all supported platforms [*]). Could the patch file include a test for 64bit, like e.g. in this [http://trac.macports.org/browser/trunk/dports/graphics/gimp2/files/patch- plug-ins-twain-tw_mac.c.diff?rev=61996 GIMP patch file], only reversed (i.e. if {{{__LP64__}}} is defined, undefine {{{HAVE_CARBON}}})? [*] from the [http://sourceforge.net/tracker/index.php?func=detail&aid=2898204&group_id=21... related bug report] in the bug tracker of GIMP on OS X:
Particially disabling the HAVE_CARBON code in glib/gunicollate.c will use the standard GLIB collation function, which in turn will give a lexicographical sort order. This is nearly the same as in Finder. Althouhg this fix doesn't use OS X functions to create the collation keys, it will give a useable sort order back to Snow Leoapard builds. So I decided to use this code not only on SL but in future builds on Leopard (and Tiger) too to be consistent between the various OS X builds.
-- Ticket URL: <https://trac.macports.org/ticket/29051#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#29051: glib2: revert to OS X sorting (alphabetical, not case-sensitive) on Mac OS X 10.5.8 Leopard ------------------------------------------+--------------------------------- Reporter: suv-sf@… | Owner: ryandesign@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.9.2 Keywords: | Port: glib2, glib2-devel ------------------------------------------+--------------------------------- Changes (by jmr@…): * owner: macports-tickets@… => ryandesign@… * cc: ryandesign@… (removed) -- Ticket URL: <https://trac.macports.org/ticket/29051#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts