{"action":"created","issue":{"url":"https://api.github.com/repos/googlei18n/fontmake/issues/86","repository_url":"https://api.github.com/repos/googlei18n/fontmake","labels_url":"https://api.github.com/repos/googlei18n/fontmake/issues/86/labels{/name}","comments_url":"https://api.github.com/repos/googlei18n/fontmake/issues/86/comments","events_url":"https://api.github.com/repos/googlei18n/fontmake/issues/86/events","html_url":"https://github.com/googlei18n/fontmake/issues/86","id":158427458,"number":86,"title":"Space are not allowed in glyph names: standard says they should be","user":{"login":"marekjez86","id":13322623,"avatar_url":"https://avatars.githubusercontent.com/u/13322623?v=3","gravatar_id":"","url":"https://api.github.com/users/marekjez86","html_url":"https://github.com/marekjez86","followers_url":"https://api.github.com/users/marekjez86/followers","following_url":"https://api.github.com/users/marekjez86/following{/other_user}","gists_url":"https://api.github.com/users/marekjez86/gists{/gist_id}","starred_url":"https://api.github.com/users/marekjez86/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekjez86/subscriptions","organizations_url":"https://api.github.com/users/marekjez86/orgs","repos_url":"https://api.github.com/users/marekjez86/repos","events_url":"https://api.github.com/users/marekjez86/events{/privacy}","received_events_url":"https://api.github.com/users/marekjez86/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-03T18:33:09Z","updated_at":"2016-06-08T22:19:23Z","closed_at":"2016-06-08T22:17:46Z","body":"the following quoted paragraph was copied from http://monotype.github.io/OpenType_Table_Source/otl_source.html#glyph-references\r\n\r\n\"Glyph References\r\nGlyphs are usually represented by their name from the ‘post’ table or ‘CFF ‘ table. If there is no format 2 post table or no names stored in the ‘CFF ‘ table then it is possible to refer to glyphs by Unicode or by glyph index.\r\n\r\nTo refer to a glyph by Unicode use U or u followed by a space, followed by the hexadecimal Unicode. To refer to a glyph by index use a # followed by a space followed by the index in decimal. For example: in a normal Latin font in standard order the lowercase a can be referred to in the following ways:\r\n\r\na\r\nU 0061\r\nu 0061\r\n\\# 68\r\n\"\r\n\r\n\r\nBased on the above the following four lines refer to the same item:\r\na\r\nU 0061\r\nu 0061\r\n\\# 68\r\n\r\nBut here is the from a pipeline execution which \r\nTraceback (most recent call last):\r\n  File \"/usr/local/bin/fontmake\", line 9, in <module>\r\n    load_entry_point('fontmake==1.0', 'console_scripts', 'fontmake')()\r\n  File \"build/bdist.linux-x86_64/egg/fontmake/__main__.py\", line 43, in main\r\n  File \"build/bdist.linux-x86_64/egg/fontmake/font_project.py\", line 237, in run_from_glyphs\r\n  File \"build/bdist.linux-x86_64/egg/fontmake/font_project.py\", line 282, in run_from_ufos\r\n  File \"build/bdist.linux-x86_64/egg/fontmake/font_project.py\", line 144, in build_otfs\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/misc/loggingTools.py\", line 369, in wrapper\r\n    return func(*args, **kwds)\r\n  File \"build/bdist.linux-x86_64/egg/fontmake/font_project.py\", line 183, in save_otfs\r\n  File \"/usr/local/lib/python2.7/dist-packages/ufo2ft/__init__.py\", line 21, in compileOTF\r\n    featureCompiler.compile()\r\n  File \"/usr/local/lib/python2.7/dist-packages/ufo2ft/makeotfParts.py\", line 38, in compile\r\n    self.setupFile_featureTables()\r\n  File \"/usr/local/lib/python2.7/dist-packages/ufo2ft/makeotfParts.py\", line 169, in setupFile_featureTables\r\n    table = mtiLib.build(feafile, self.outline)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 1089, in build\r\n    return parseTable(lines, font, tableTag=tableTag)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 988, in parseTable\r\n    }[tableTag](lines, font)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 893, in parseGSUB\r\n    return parseGSUBGPOS(lines, font, 'GSUB')\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 874, in parseGSUBGPOS\r\n    lookup = parseLookup(lines, tableTag, font, lookupMap)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 830, in parseLookup\r\n    parseLookupSubTable(subtable, lines, font, lookupMap)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 756, in parseContextSubst\r\n    return parseContext(self, lines, font, \"ContextSubst\", lookupMap=lookupMap)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 713, in parseContext\r\n    classDefs[idx] = parseClassDef(lines, klass=klass)\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 634, in parseClassDef\r\n    glyph = makeGlyph(line[0])\r\n  File \"/usr/local/lib/python2.7/site-packages/fonttools-3.0-py2.7.egg/fontTools/mtiLib/__init__.py\", line 35, in makeGlyph\r\n    assert s.find(' ') < 0, \"Space found in glyph name: %s\" % s\r\nAssertionError: Space found in glyph name: u 200D"},"comment":{"url":"https://api.github.com/repos/googlei18n/fontmake/issues/comments/224747590","html_url":"https://github.com/googlei18n/fontmake/issues/86#issuecomment-224747590","issue_url":"https://api.github.com/repos/googlei18n/fontmake/issues/86","id":224747590,"user":{"login":"behdad","id":142891,"avatar_url":"https://avatars.githubusercontent.com/u/142891?v=3","gravatar_id":"","url":"https://api.github.com/users/behdad","html_url":"https://github.com/behdad","followers_url":"https://api.github.com/users/behdad/followers","following_url":"https://api.github.com/users/behdad/following{/other_user}","gists_url":"https://api.github.com/users/behdad/gists{/gist_id}","starred_url":"https://api.github.com/users/behdad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/behdad/subscriptions","organizations_url":"https://api.github.com/users/behdad/orgs","repos_url":"https://api.github.com/users/behdad/repos","events_url":"https://api.github.com/users/behdad/events{/privacy}","received_events_url":"https://api.github.com/users/behdad/received_events","type":"User","site_admin":false},"created_at":"2016-06-08T22:19:23Z","updated_at":"2016-06-08T22:19:23Z","body":"Thanks.  Please try again and let me know if it worked."}}
{"actor":{"display_login":"behdad"}}