Red Hat Bugzilla – Bug 860550
do not recognize GASP table version 1
Last modified: 2012-12-20 10:22:25 EST
Description of problem:
In LiberatoinSans-Regular.sfd, i did changes and made GASP table version 1 and saved.
But when i again reopen it, it should me GASP table 1
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. open LiberationSans-Regular.sfd
2. Element-> Font Information -> Grid Fitting
3. Select Version 1
4. Save and reopen
5. Again it shows Version 0
It does not recognize GASP table version 1
Once save, it should show Version 1 GASP table values
Liberation fonts sfd http://git.fedorahosted.org/cgit/liberation-fonts.git/tree/src/LiberationSans-Regular.sfd
upstream bug id
Thanks for filing this upstream. ;)
Created attachment 617672 [details]
Commit that fixed this
The version of the GASP table didn't used to be saved to the SFD at all. This was fixed upstream on 2011-09-18 with commit a0fd436141fa402056ec9f247b3f50cf2b3ecb31, by appending the GASP version to the end of the GASPTable: line of the SFD, and reading it back correctly.
(Sorry, can't patch this today, but feel free if you have time, Kevin.)
Thanks Paul for pointing towards patch.
I have done scratch build with it http://koji.fedoraproject.org/koji/taskinfo?taskID=4532402
Still this does not solve problem fully.
Fontforge is not able to store 4th and 5th column of GASP table
that is Symmetric Smoothing and Grid Fitting W/ Sym-Smoothing.
Does any development happened on this as well?
I think we need to move F17 to FontForge version 20120731-b. This is already in Rawhide (and, I think, F18). There have been plenty of common crashers fixed since the last release, and this is probably our last chance to update that doesn't bring with it a heap of user interface changes.
As an aside: Pravin, if you wish to file any more bugs upstream (and it's great that you're willing to do so), please use the tracker on GitHub instead of SourceForge, as that is the new upstream. The current list is here:
Might be we should update, upstream URL in spec file as well.
I'm ok with a f17 bugfix only update... ;)
Can you push it out Paul?
fontforge-20120731b-1.fc17 has been submitted as an update for Fedora 17.
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing fontforge-20120731b-1.fc17'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).
fontforge-20120731b-1.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.