{"action":"created","issue":{"url":"https://api.github.com/repos/pettarin/glyphIgo/issues/5","labels_url":"https://api.github.com/repos/pettarin/glyphIgo/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/pettarin/glyphIgo/issues/5/comments","events_url":"https://api.github.com/repos/pettarin/glyphIgo/issues/5/events","html_url":"https://github.com/pettarin/glyphIgo/issues/5","id":63197936,"number":5,"title":"Font subsetting excludes the substitution and ligature glyphs","user":{"login":"abhaga","id":168362,"avatar_url":"https://avatars.githubusercontent.com/u/168362?v=3","gravatar_id":"","url":"https://api.github.com/users/abhaga","html_url":"https://github.com/abhaga","followers_url":"https://api.github.com/users/abhaga/followers","following_url":"https://api.github.com/users/abhaga/following{/other_user}","gists_url":"https://api.github.com/users/abhaga/gists{/gist_id}","starred_url":"https://api.github.com/users/abhaga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abhaga/subscriptions","organizations_url":"https://api.github.com/users/abhaga/orgs","repos_url":"https://api.github.com/users/abhaga/repos","events_url":"https://api.github.com/users/abhaga/events{/privacy}","received_events_url":"https://api.github.com/users/abhaga/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-20T09:40:14Z","updated_at":"2015-06-07T09:43:43Z","closed_at":null,"body":"The current font subsetting algorithm works by selecting the glyphs corresponding to given characters and removing everything else. This removes all the substitution and ligature glyphs as well. For Indic scripts (and potentially for other scripts), this leaves the font unusable.\r\n\r\nThe correct way would be to pull in for every character, all the substitution glyphs and all ligature glyphs. Out of many font subsetting python scripts I found online, only the one included in fonttools works correctly: https://github.com/behdad/fonttools/blob/master/Lib/fontTools/subset.py\r\n"},"comment":{"url":"https://api.github.com/repos/pettarin/glyphIgo/issues/comments/109730179","html_url":"https://github.com/pettarin/glyphIgo/issues/5#issuecomment-109730179","issue_url":"https://api.github.com/repos/pettarin/glyphIgo/issues/5","id":109730179,"user":{"login":"pettarin","id":4086163,"avatar_url":"https://avatars.githubusercontent.com/u/4086163?v=3","gravatar_id":"","url":"https://api.github.com/users/pettarin","html_url":"https://github.com/pettarin","followers_url":"https://api.github.com/users/pettarin/followers","following_url":"https://api.github.com/users/pettarin/following{/other_user}","gists_url":"https://api.github.com/users/pettarin/gists{/gist_id}","starred_url":"https://api.github.com/users/pettarin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pettarin/subscriptions","organizations_url":"https://api.github.com/users/pettarin/orgs","repos_url":"https://api.github.com/users/pettarin/repos","events_url":"https://api.github.com/users/pettarin/events{/privacy}","received_events_url":"https://api.github.com/users/pettarin/received_events","type":"User","site_admin":false},"created_at":"2015-06-07T09:43:43Z","updated_at":"2015-06-07T09:43:43Z","body":"@abhaga I am studying your https://github.com/behdad/fonttools , which is extremely nice piece of work, well done!\r\n\r\nOn PyPI I found https://pypi.python.org/pypi/FontTools . From the metadata, I guess it is the old code from SourceForge (which does not contain subset.py). Are you planning to release the code in your current GitHub repo on PyPI? If so, I will switch glyphIgo from fontforge to it, requiring fonttools as a dependency. If not, might I include your subset.py in glyphIgo, with the due ack in the README?"}}