8000 [ttFont] xmlWriter import error with saveXML split tables = True requests · Issue #1188 · fonttools/fonttools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[ttFont] xmlWriter import error with saveXML split tables = True requests #1188

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
chrissimpkins opened this issue Feb 21, 2018 · 1 comment
Closed

[ttFont] xmlWriter import error with saveXML split tables = True requests #1188

chrissimpkins opened this issue Feb 21, 2018 · 1 comment

Comments

@chrissimpkins
Copy link
Member

Attempt to dump split OT tables with ttx, executed in Python 3 venv with PyPI release of fontTools:

$ ttx -s -d regular-ttx Hack-Regular.ttf
Dumping "Hack-Regular.ttf" to "regular-ttx/Hack-Regular.ttx"...
ERROR: Unhandled exception has occurred
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/fontTools/ttx.py", line 389, in main
    process(jobs, options)
  File "/usr/local/lib/python3.6/site-packages/fontTools/ttx.py", line 363, in process
    action(input, output, options)
  File "/usr/local/lib/python3.6/site-packages/fontTools/misc/loggingTools.py", line 372, in wrapper
    return func(*args, **kwds)
  File "/usr/local/lib/python3.6/site-packages/fontTools/ttx.py", line 260, in ttDump
    newlinestr=options.newlinestr)
  File "/usr/local/lib/python3.6/site-packages/fontTools/ttLib/ttFont.py", line 221, in saveXML
    self._saveXML(writer, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/fontTools/ttLib/ttFont.py", line 264, in _saveXML
    tableWriter = xmlWriter.XMLWriter(tablePath,
NameError: name 'xmlWriter' is not defined

PR for fix incoming...
@chrissimpkins chrissimpkins mentioned this issue Feb 21, 2018
Merged
@chrissimpkins
Copy link
Member Author

PR #1189

miguelsousa added a commit that referenced this issue Feb 22, 2018
8207
@miguelsousa
[ttx_test] Test for split tables (-s) option
782f28b 
#1188
@miguelsousa miguelsousa mentioned this issue Feb 22, 2018
Merged
miguelsousa added a commit that referenced this issue Feb 22, 2018
@miguelsousa
[ttx_test] Test for split tables (-s) option
ccb1d24 
#1188
miguelsousa added a commit that referenced this issue Feb 22, 2018
@miguelsousa
[ttx_test] Test for split tables (-s) option
bf6852a 
#1188
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chrissimpkins
0