shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[subset/svg] use broader ImportError instead of ModuleNotFoundError

Browse Source 
Fixes https://github.com/fonttools/fonttools/issues/2696
This commit is contained in:
Cosimo Lupo 2022-07-18 11:32:40 +02:00
parent 4139d8914a
commit dbe60eae80
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Lib/fontTools/subset/svg.py
View File

@ -7,7 +7,7 @@ from typing import Dict, Iterable, Iterator, List, Optional, Set, Tuple
try: try:
from lxml import etree from lxml import etree
except ModuleNotFoundError: except ImportError:
# lxml is required for subsetting SVG, but we prefer to delay the import error # lxml is required for subsetting SVG, but we prefer to delay the import error
# until subset_glyphs() is called (i.e. if font to subset has an 'SVG ' table) # until subset_glyphs() is called (i.e. if font to subset has an 'SVG ' table)
etree = None etree = None
@ -193,7 +193,7 @@ def ranges(ints: Iterable[int]) -> Iterator[Tuple[int, int]]:
@_add_method(ttLib.getTableClass("SVG ")) @_add_method(ttLib.getTableClass("SVG "))
def subset_glyphs(self, s) -> bool: def subset_glyphs(self, s) -> bool:
if etree is None: if etree is None:
raise ModuleNotFoundError("No module named 'lxml', required to subset SVG") raise ImportError("No module named 'lxml', required to subset SVG")
# glyph names (before subsetting) # glyph names (before subsetting)
glyph_order: List[str] = s.orig_glyph_order glyph_order: List[str] = s.orig_glyph_order