← 输入框|github.com|刷新
Skip to content

Fix 3187521 - crash due to unescaped fontpaths - #59

Closed
ivanov wants to merge 1 commit into
matplotlib:masterfrom
ivanov:fix-3187521
Closed

ivanov wants to merge 1 commit into
matplotlib:masterfrom
ivanov:fix-3187521

Conversation

@ivanov

@ivanov ivanov commented Mar 26, 2011

Copy link
Copy Markdown
Member

Some paths were improperly being interpreted as a regular expression, leading to errors.

@jkseppan

Copy link
Copy Markdown
Member

I don't see how calling repr fixes this - it'll just put some quotation marks around the directory name, right? The glob function used here doesn't call any Unix functions, it calls fnmatch which compiles the pattern to a regexp. What we need to do is call os.listdir on each directory and filter the results with fnmatch.filter("*.ttf") etc.

Or am I getting this wrong?

@jkseppan

Copy link
Copy Markdown
Member

See pull request #60

@mdboom

mdboom commented Mar 28, 2011

Copy link
Copy Markdown
Member

Chose alternative in pull request #60 -- therefore closing this.

@mdboom mdboom closed this Mar 28, 2011
@richbwood richbwood mentioned this pull request Dec 19, 2012
magnunor pushed a commit to magnunor/matplotlib that referenced this pull request Dec 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants