Description of problem: Pango, Harfbuzz and Windows layout engine all appear to support CCMP for Indic. However, ICU still does not. The ICU developers are under the incorrect assumption that the Windows layout engine does not support CCMP for Indic. So, in terms of CCMP support for Indic, it looks like we just need ICU fixed: http://bugs.icu-project.org/trac/ticket/7601 For details and examples refer to the aforementioned ICU bug.
Do we think that Steven White's patch is what we want here ?, i.e. http://bugs.icu-project.org/trac/attachment/ticket/7601/IndicReordering.cpp.diff I can add that in on an experimental basis to rawhide if you want ?
Yes, it looks like. Will be nice if can get rawhide build for test.
(In reply to comment #1) > Do we think that Steven White's patch is what we want here ? Just double check that the bit mask does not conflict with an existing one. I vaguely remember something along those lines with his first patch.
yeah, bit mask is a dup of blwsFeatureMask
plausible tweak of the patch checked in, build blocked at the moment due to kernel/grubby nvr requires/provides problems
done as icu-4.8.1-2.fc17, wonder what side-effects it will have
Hi Caolan, if you could spin me a libicu package for F16, I'm more than happy to test it for you.
http://koji.fedoraproject.org/scratch/caolanm/task_3618123/
ack, that's a 4.8.X, need a 4.6.X for F-16, bah
http://koji.fedoraproject.org/scratch/caolanm/task_3618187/
Created attachment 550804 [details] ICU Indic CCMP testcase
Created attachment 550805 [details] screenshot without fix
Created attachment 550806 [details] screenshot with fix
The first row of the testcase file contains a sequence of Unicode Sinhala that tests FreeSerif's lookup that uses the CCMP feature. Caolan, the RPM you built ensures that FreeSerif's lookup that uses CCMP works correctly. Thanks. Do you plan on pushing an RPM with the fix into F16? F16 already has a -3 revision RPM, so the -2.1 revision RPM you built is older than what's available in the repository.
icu-4.6-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/icu-4.6-4.fc16
icu-4.6-4.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.