Utkarsh- yes, I think you are right.  But this change doesn't introduce the behavior.  This change just adds a variable for a user to disable fontconfig, previously, the decision to enable fontconfig was based on whether or not it is installed (FONTCONFIG_FOUND)<br>

<br>Pat<br><br><div class="gmail_quote">On Mon, Jul 30, 2012 at 12:56 PM, Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com" target="_blank">berk.geveci@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Is the main issue the fact that it doesn't compile everywhere?<div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote">On Mon, Jul 30, 2012 at 12:36 PM, Utkarsh Ayachit <span dir="ltr"><<a href="mailto:utkarsh.ayachit@kitware.com" target="_blank">utkarsh.ayachit@kitware.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is that an acceptable change? Looks like this will change the way the<br>
module is built based on some flag. I thought that was against the<br>
policy, isnt' it?<br>
<span><font color="#888888"><br>
Utkarsh<br>
</font></span><div><div><br>
On Mon, Jul 30, 2012 at 8:16 AM, Pat Marion <<a href="mailto:pat.marion@kitware.com" target="_blank">pat.marion@kitware.com</a>> wrote:<br>
> Looks good to me.  I added a follow up commit that adds an error message<br>
> when VTK_USE_FONTCONFIG is enabled but FontConfig is not found:<br>
><br>
> <a href="http://review.source.kitware.com/#/t/982/" target="_blank">http://review.source.kitware.com/#/t/982/</a><br>
><br>
> Pat<br>
><br>
><br>
> On Mon, Jul 30, 2012 at 9:13 AM, David Lonie <<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@kitware.com</a>><br>
> wrote:<br>
>><br>
>> Hi Pat,<br>
>><br>
>> On Thu, Jul 26, 2012 at 10:49 PM, Pat Marion <<a href="mailto:pat.marion@kitware.com" target="_blank">pat.marion@kitware.com</a>><br>
>> wrote:<br>
>> > I noticed that a fontconfig support was recently added to<br>
>> > 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<br>
>> > option,<br>
>> > allowing fontconfig to be disabled.  Currently,<br>
>> > Rendering/FreeType/CMakeLists.txt uses fontconfig when FONTCONFIG_FOUND<br>
>> > is<br>
>> > true.<br>
>><br>
>> 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>
><br>
><br>
><br>
</div></div><div><div>> _______________________________________________<br>
> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
><br>
><br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br>
</div></div></blockquote></div><br>