Skip to content
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.

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

Catch IO errors when building font cache #7222

Merged
merged 1 commit into from Oct 5, 2016

Conversation

greg-roper
Copy link
Contributor

Problem seen on Windows machines where permission was denied when trying to load specific fonts (DINpro Light).

Problem seen on Windows machines where permission was denied when trying to load specific fonts (DINpro Light).
@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone Oct 5, 2016
@tacaswell tacaswell changed the title Catch IO errors when building font cache [MRG+1] Catch IO errors when building font cache Oct 5, 2016
@tacaswell
Copy link
Member

My only concern would be the exception handling could be unified a bit, but I think that as long as we support python2 we are going to have to keep all of these paths.

@NelleV
Copy link
Member

NelleV commented Oct 5, 2016

I wonder what is wrong if appveyor…
Anyhow, it is unrelated to this PR, so I'll go ahead and merge.

Thanks for the patch!

@NelleV NelleV merged commit b34c55d into matplotlib:master Oct 5, 2016
@QuLogic QuLogic changed the title [MRG+1] Catch IO errors when building font cache Catch IO errors when building font cache Oct 15, 2016
QuLogic pushed a commit to QuLogic/matplotlib that referenced this pull request Oct 16, 2016
Catch IO errors when building font cache
@QuLogic
Copy link
Member

QuLogic commented Oct 16, 2016

Backported to v2.x via 9b2314f.

@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
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.

None yet

5 participants