{"action":"closed","issue":{"url":"https://api.github.com/repos/fonttools/fonttools/issues/1057","repository_url":"https://api.github.com/repos/fonttools/fonttools","labels_url":"https://api.github.com/repos/fonttools/fonttools/issues/1057/labels{/name}","comments_url":"https://api.github.com/repos/fonttools/fonttools/issues/1057/comments","events_url":"https://api.github.com/repos/fonttools/fonttools/issues/1057/events","html_url":"https://github.com/fonttools/fonttools/issues/1057","id":258577712,"number":1057,"title":"Merge failure for Noto","user":{"login":"brawer","id":1527880,"avatar_url":"https://avatars2.githubusercontent.com/u/1527880?v=4","gravatar_id":"","url":"https://api.github.com/users/brawer","html_url":"https://github.com/brawer","followers_url":"https://api.github.com/users/brawer/followers","following_url":"https://api.github.com/users/brawer/following{/other_user}","gists_url":"https://api.github.com/users/brawer/gists{/gist_id}","starred_url":"https://api.github.com/users/brawer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brawer/subscriptions","organizations_url":"https://api.github.com/users/brawer/orgs","repos_url":"https://api.github.com/users/brawer/repos","events_url":"https://api.github.com/users/brawer/events{/privacy}","received_events_url":"https://api.github.com/users/brawer/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-09-18T18:32:30Z","updated_at":"2018-02-18T22:44:20Z","closed_at":"2018-02-18T22:44:20Z","author_association":"COLLABORATOR","body":"This merge command:\r\n\r\n```sh\r\nfonttools merge \\\r\n    ../noto-source/master_ttf_interpolatable/NotoSans-Regular.ttf \\\r\n    ../noto-source/master_ttf_interpolatable/NotoSansCypriot-Regular.ttf\r\n```\r\n\r\nfails with the following error:\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"./fonttools\", line 12, in <module>\r\n    sys.exit(main())\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/__main__.py\", line 29, in main\r\n    runpy.run_module(mod, run_name='__main__')\r\n  File \"/usr/lib/python2.7/runpy.py\", line 192, in run_module\r\n    fname, loader, pkg_name)\r\n  File \"/usr/lib/python2.7/runpy.py\", line 72, in _run_code\r\n    exec code in run_globals\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/merge.py\", line 1079, in <module>\r\n    sys.exit(main())\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/misc/loggingTools.py\", line 372, in wrapper\r\n    return func(*args, **kwds)\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/merge.py\", line 1072, in main\r\n    font = merger.merge(args)\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/merge.py\", line 913, in merge\r\n    table = clazz(tag).merge(self, tables)\r\n  File \"/home/sascha/src/fonttools/Lib/fontTools/merge.py\", line 575, in merge\r\n    assert (table is not None and table is not NotImplemented), \"Have duplicates to resolve for font %d but no GSUB\" % (i + 1)\r\nAssertionError: Have duplicates to resolve for font 2 but no GSUB\r\n```\r\n\r\nThe same error happens for quite a few other fonts in Noto. Examples are NotoSansCarian-Regular.ttf or NotoSansDeseret-Regular.ttf, but there's more. See the attached [merge-failures.zip](https://github.com/fonttools/fonttools/files/1311746/merge-failures.zip) for some fonts I'm trying to merge.\r\n\r\nThis might also be a problem with the Noto sources, but I'm not sure what exactly would be wrong, or how exactly the vendor should be told to change the sources so that the shards can be merged into a single font."}}
{"actor":{"display_login":"behdad"}}