[vtk-developers] [ParaView3-Developer] [InfoVis] VTK_USE_QT

[Jeff Baumes]

Let's remove VTK_HAS_QT and maybe enforce that FindQt works, and
immediately bail out if it doesn't.

Jeff

On Fri, Apr 10, 2009 at 7:41 PM, Utkarsh Ayachit
<utkarsh.ayachit at kitware.com> wrote:
> VTK_HAS_QT is just for convenience. It QT dependent files are included
> before Qt is found, cmake tends to spit out too many errors that hide
> the fact that what's missing is merely Qt. So VTK_HAS_QT. I am not too
> keen on it either, and I don't have any hesitation removing it. We can
> simply use VTK_USE_QT and it should work fine.
>
> VTK_USE_QT cannot merely replace VTK_USE_QVTK , since VTK_USE_QVTK
> includes compilation of QVTKWidget which needs Rendering which is
> optional. That's the reason for creation of this new VTK_USE_QT as
> well.
>
> On Fri, Apr 10, 2009 at 4:26 PM, Timothy M. Shead <tshead at sandia.gov> wrote:
>> Utkarsh Ayachit wrote:
>>>
>>> As per the request, I've committed changes to VTK to add a new top
>>> level option VTK_USE_QT. When this option is set and Qt is found,
>>> VTK_HAS_QT is set. The internal cmake files can then use the value of
>>> this cache variable to add files that depend on Qt.
>>>
>>
>> Why the distinction between USE_QT and HAS_QT?  We don't seem to do this for
>> other optional libraries (such as boost) ...
>>
>> Also, it seems as if USE_QT should replace USE_QVTK, and we might look at
>> whether GUISUPPORT still makes sense ...
>>
>> Cheers,
>> Tim
>>
>>
>> --
>> Timothy M. Shead
>> Data Analysis & Visualization (1424)
>> Sandia National Laboratories
>> 505-284-0139
>>
>>
> _______________________________________________
> ParaQ-Developer mailing list
> ParaQ-Developer at paraview.org
> http://public.kitware.com/cgi-bin/mailman/listinfo/paraq-developer
>



-- 
Jeff Baumes, Ph.D.
R&D Engineer, Kitware Inc.
(518) 881-4932
jeff.baumes at kitware.com