Fix flipped paths in non-writable config dir warning. by QuLogic · Pull Request #18014 · matplotlib/matplotlib · GitHub
Skip to content

Fix flipped paths in non-writable config dir warning.#18014

Merged
jklymak merged 1 commit into
matplotlib:masterfrom
QuLogic:fix-non-writable-config
Jul 22, 2020
Merged

Fix flipped paths in non-writable config dir warning.#18014
jklymak merged 1 commit into
matplotlib:masterfrom
QuLogic:fix-non-writable-config

Conversation

@QuLogic

@QuLogic QuLogic commented Jul 22, 2020

Copy link
Copy Markdown
Member

PR Summary

Fixes #18011.

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/next_api_changes/* if API changed in a backward-incompatible way

@QuLogic QuLogic added this to the v3.3.1 milestone Jul 22, 2020
@jklymak jklymak merged commit 9fcf958 into matplotlib:master Jul 22, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jul 22, 2020
@QuLogic QuLogic deleted the fix-non-writable-config branch July 22, 2020 19:54
QuLogic added a commit that referenced this pull request Jul 22, 2020
…014-on-v3.3.x

Backport PR #18014 on branch v3.3.x (Fix flipped paths in non-writable config dir warning.)
@john-zenden

john-zenden commented Aug 12, 2022

Copy link
Copy Markdown

@tacaswell

Copy link
Copy Markdown
Member

One of the things that goes into the cache directory is a json file of the fonts we found. Pre-generating that file and uncluding it in your image will speed up the initial import time by avoid a (possibly expensive) file walking process.

@john-zenden

Copy link
Copy Markdown

@tacaswell Is the JSON filename always the same?

@tacaswell

Copy link
Copy Markdown
Member

Yes, but if we change the spec of the file we will change the filename. We have not changed it recently, but I suspect we will change it soon. If you need to support more than one version of mpl include both files.

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.

Confusing error message when home config directory not writable

4 participants