Fix Mac fontconfig build (try 2)

Fix Mac fontconfig build (try 2)

The Mac fontconfig just #defines the cache directory, which works fine if fontconfig never has to look up any fonts (the case until now). If it has to actually find fonts from the disk, the cache directory and config directory need to be properly defined as well as running fc-cache to populate the cache directory.

Populating the cache directory can take some time, but should only happen on a clean build. To remove this extra time, we have to not build poppler on Mac, which can now be accomplished with GYP_DEFINES="skia_mac_poppler=0"

Committed: http://code.google.com/p/skia/source/detail?r=13007

[epoger, 2014-01-09 18:42:42]

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
File third_party/fontconfig/process-template.py (right):

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
third_party/fontconfig/process-template.py:1: #!/usr/bin/python
Do the broken Trybot runs indicate a real problem with this CL?

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
third_party/fontconfig/process-template.py:25: (args, junk) =
parser.parse_args()
I think the preferred convention in Python is to use '_' as the variable name
when you are just throwing away a field.

See http://stackoverflow.com/questions/5893163/underscore-in-python

[vandebo (ex-Chrome), 2014-01-09 19:24:32]

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
File third_party/fontconfig/process-template.py (right):

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
third_party/fontconfig/process-template.py:1: #!/usr/bin/python
On 2014/01/09 18:42:42, epoger wrote:
> Do the broken Trybot runs indicate a real problem with this CL?

I don't think so.  The failures are in random .cc files.

https://codereview.chromium.org/132333002/diff/30001/third_party/fontconfig/p...
third_party/fontconfig/process-template.py:25: (args, junk) =
parser.parse_args()
On 2014/01/09 18:42:42, epoger wrote:
> I think the preferred convention in Python is to use '_' as the variable name
> when you are just throwing away a field.
> 
> See http://stackoverflow.com/questions/5893163/underscore-in-python

Done.

[epoger, 2014-01-09 19:26:19]

lgtm

[commit-bot: I haz the power, 2014-01-09 22:45:52]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/vandebo@chromium.org/132333002/100001

[commit-bot: I haz the power, 2014-01-09 22:54:30]

Change committed as 13007