Looks good to me.  I added a follow up commit that adds an error message when VTK_USE_FONTCONFIG is enabled but FontConfig is not found:<br><br><a href="http://review.source.kitware.com/#/t/982/">http://review.source.kitware.com/#/t/982/</a><br>

<br>Pat<br><br><div class="gmail_quote">On Mon, Jul 30, 2012 at 9:13 AM, David Lonie <span dir="ltr"><<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Pat,<br>
<div><div class="h5"><br>
On Thu, Jul 26, 2012 at 10:49 PM, Pat Marion <<a href="mailto:pat.marion@kitware.com">pat.marion@kitware.com</a>> wrote:<br>
> I noticed that a fontconfig support was recently added to Rendering/FreeType<br>
> module.  This looks like a really useful feature!  For users who wish to<br>
> avoid the dependency, I think there should be a VTK_USE_FONTCONFIG option,<br>
> allowing fontconfig to be disabled.  Currently,<br>
> Rendering/FreeType/CMakeLists.txt uses fontconfig when FONTCONFIG_FOUND is<br>
> true.<br>
<br>
</div></div>Sorry for the late reply -- will this work for you?<br>
<br>
<a href="http://review.source.kitware.com/#/c/6637/" target="_blank">http://review.source.kitware.com/#/c/6637/</a><br>
<br>
Dave<br>
</blockquote></div><br>