This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours

Bug 873635

Summary: [ml_IN] Meera and Rachana rendering issues with Harfbuzz
Product: [Fedora] Fedora Reporter: Ani Peter <apeter>
Component: smc-fontsAssignee: Rajeesh <rajeeshknambiar>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 19CC: discuss, fonts-bugs, i18n-bugs, nav007, psatpute, rajeeshknambiar, santhosh.thottingal
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: smc-fonts-5.0.1-7.fc19 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1023847 (view as bug list) Environment:
Last Closed: 2013-09-22 20:29:11 EDT Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Bug Depends On:    
Bug Blocks: 1023847    
Attachments:
Description Flags
patch to fix rakar reodereing
none
patch to fix rachana rakar reordereing
none
this fixes ഗ്ദ്ധ്രോ string as well none

Description Ani Peter 2012-11-06 06:24:45 EST
Description of problem:

Harfbuzz rendering with Meera and Rachana fonts fail for the following cases with test case numbers here http://silpa.org.in/pub/tests/hb/ml/hbview_test_results-2012-11-06/hbview_test_results.html:

Meera:
2. അഥൎവ്വം (ൎdot reph is positioned correctly, but വ്വം is incorrect)
15. ക്ത്ര
19. ഖ്ര
55. സ്പ്ലേ

Rachana:
1. അൎത്ഥം (dot reph not formed)
2. അഥൎവ്വം ((dot reph not formed)
15. ക്ത്ര
19. ഖ്ര
53. സ്പ്രി
54. സ്പ്രേ (Meera does it right) 

Thanks
Ani
Comment 1 Ani Peter 2012-11-06 06:26:11 EST
Bug filed upstream: https://savannah.nongnu.org/bugs/index.php?37675

Thanks
Ani
Comment 2 Ani Peter 2012-11-06 07:10:00 EST
Please note ഖ + ് + ര must be formed as ഖ്ര. Similar alphabets - ഠ, ഫ, ങ, ഞ, ണ.

Thanks
Comment 3 Pravin Satpute 2012-11-06 07:11:47 EST
Created attachment 639327 [details]
patch to fix rakar reodereing

added pref.
Comment 4 Pravin Satpute 2012-11-06 07:12:24 EST
Created attachment 639328 [details]
patch to fix rachana rakar reordereing
Comment 5 Santhosh Thottingal 2012-11-06 13:31:35 EST
In the patch r3 xx is ര + ്- this is wrong order for the prebase ra. It should be
് + ര( xx r3)

Tried reversing it in the patch, but that cause other sequences fail(eg: ഗ്ദ്ധ്രോ, ക്ത്രോ ).
Comment 6 Pravin Satpute 2012-11-07 00:07:26 EST
(In reply to comment #5)
> In the patch r3 xx is ര + ്- this is wrong order for the prebase ra. It
> should be
> ് + ര( xx r3)
> 
> Tried reversing it in the patch, but that cause other sequences fail(eg:
> ഗ്ദ്ധ്രോ, ക്ത്രോ ).

We have not changed sequence, just moved sequence it to appropriate feature tag. see the earlier reph in pstf feature, it was using same sequence is  -> "r3 xx"
Comment 7 Pravin Satpute 2012-11-07 01:43:03 EST
Created attachment 639852 [details]
this fixes ഗ്ദ്ധ്രോ string as well

I am getting same rendering of ക്ത്രോ in pango and in harfbuzz-ng, so no need to fix it.
Comment 8 Pravin Satpute 2013-04-01 01:25:58 EDT
Hi Ani,

  I think upstream has done some improvement in this regards, can you just get status update from them? 

  Then might be we can look at open issues.
Comment 9 Rajeesh 2013-04-01 08:22:11 EDT
The current status is as follows:

Meera:
2. അഥൎവ്വം (ൎdot reph is positioned correctly, but വ്വം is incorrect) - not fixed
15. ക്ത്ര - fixed
19. ഖ്ര - not fixed/cannot be fixed at font level for mlym spec
55. സ്പ്ലേ - not fixed, would need glyph modification

Rachana:
1. അൎത്ഥം (dot reph not formed) - fixed, dot reph is added to Rachana, positioning not correct yet
2. അഥൎവ്വം ((dot reph not formed) - fixed, dot reph added, positioning not correct yet
15. ക്ത്ര - fixed
19. ഖ്ര - not fixed/cannot be fixed at font level for mlym spec
53. സ്പ്രി - fixed
54. സ്പ്രേ (Meera does it right) - fixed
Comment 10 Pravin Satpute 2013-04-02 00:45:58 EDT
Thanks Rajeesh for update. 
When do you expect next upstream release?

If its bit far, i think we should build smc-fonts in Fedora with patch.
Comment 11 Fedora End Of Life 2013-04-03 13:09:40 EDT
This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.

(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora19
Comment 12 Pravin Satpute 2013-06-20 02:50:29 EDT
Hi Rajeesh,

  Any further improvement in this?
Comment 13 Rajeesh 2013-06-20 06:29:47 EDT
Hi Praveen,

As most of the issues are fixed except ഖ്ര (which cannot be fixed for mlym script), we could do a new release as is. At the same time, discussion related to  സ്പ്ലേ is going on in harfbuzz mailing list. Will discuss and update.
Comment 14 Rajeesh 2013-06-23 14:40:04 EDT
Latest status in Rachana:

1. അൎത്ഥം - fixed
2. അഥൎവ്വം - fixed
15. ക്ത്ര - fixed
19. ഖ്ര - not fixed/cannot be fixed at font level for mlym spec
53. സ്പ്രി - fixed
54. സ്പ്രേ - fixed
55. സ്പ്ലേ - not fixed, probably need a contextual lookup (not sure how to)
Comment 15 Pravin Satpute 2013-08-01 07:42:37 EDT
Rajeesh, that is really nice.

As harfbuzz is getting integrated at more and more places. (gnome, libreoffice..)  i think soonish we need upstream release of this, so that users can get latest version and smc-meera will work seamless with Harfbuzz.
Comment 16 Rajeesh 2013-08-11 14:06:41 EDT
I'm glad to announce that both Rachana and Meera are ported to new mlm2 spec and all these issues are fixed.

Even in old spec, "55. സ്പ്ലേ" is fixed too.
We're planning a new release soon, stay tuned!
Comment 17 Fedora Update System 2013-09-04 02:38:13 EDT
smc-fonts-5.0.1-7.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/smc-fonts-5.0.1-7.fc19
Comment 18 Fedora Update System 2013-09-04 02:39:44 EDT
smc-fonts-5.0.1-7.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/smc-fonts-5.0.1-7.fc20
Comment 19 Fedora Update System 2013-09-04 12:06:00 EDT
Package smc-fonts-5.0.1-7.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing smc-fonts-5.0.1-7.fc20'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-15736/smc-fonts-5.0.1-7.fc20
then log in and leave karma (feedback).
Comment 20 Fedora Update System 2013-09-22 20:29:11 EDT
smc-fonts-5.0.1-7.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 21 Pravin Satpute 2013-09-22 23:57:12 EDT
Rajeesh, do we have this bug open in upstream?

If not i would like to keep this bug open until next upstream release.
Comment 22 Rajeesh 2013-09-23 04:11:21 EDT
Yes, see #comment1 by Ani. We'll use the upstream bug for further developments on this. Thanks Praveen.
Comment 23 Fedora Update System 2014-03-08 23:44:03 EST
smc-fonts-5.0.1-7.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.