Skip to content

Commit

Permalink
FIX: remove cwd from mac font path search
Browse files Browse the repository at this point in the history
While including the cwd makes sense at first glance, it does not
because

a) the result is cached so the next time your cwd will be
   different but we will not find those files
b) the time it takes to search all the files is causing prolems
c) the other 2 platforms do not do this

The comma was introduced to fix what looked like a bug (implicit
string concatenation instead of adding the empty string to the list)
in #11963.

Original code come in via 4799341 in
2011.

closes #12176
  • Loading branch information
tacaswell committed Sep 20, 2018
1 parent 2ac048b commit 07e2d54
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lib/matplotlib/font_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,6 @@
"/System/Library/Fonts/",
# fonts installed via MacPorts
"/opt/local/share/fonts",
"",
]

if not USE_FONTCONFIG and sys.platform != 'win32':
Expand Down

0 comments on commit 07e2d54

Please sign in to comment.