MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010243ParaViewFeaturepublic2010-02-05 15:112011-09-01 13:31
ReporterKen Moreland 
Assigned ToAndy Bauer 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.8Fixed in Version3.12 
Project
Topic Name
Type
Summary0010243: ParaView and CoProcessing CMake packages should be loaded together
DescriptionWhen loading linking to the CoProcessing library, you have to independently find the ParaView and CoProessing packages and hope the user picked the builds that go together. This is pretty stupid, especially since the coprocessing library is now embedded in the ParaView source. Instead, you should only have to load in one or the other. At least one of the following two should be implemented.

1. In ParaViewConfig.cmake, there should be an entry for PARAVIEW_ENABLE_COPROCESSING and the ParaView use file should automatically find the CoProcessingConfig.cmake file and run the CoProcessing use file.

2. In CoProcessingConfig.cmake, there should be an entry for ParaView_DIR and the CoProcessing use file should automatically find the ParaViewConfig.cmake file and run the ParaView use file.

Of these two options, I like the 1st one better. It is easier to find the ParaViewConfig.cmake than the CoProcessingConfig.cmake file (which is hidden in a subdirectory). I also feel that the CoProcessing library should be considered "part" of ParaView and should be loaded with it.
Steps To Reproduce
Additional InformationI'm happy to implement this if you just assign the bug back to me. I would have done it already, but I did not know if it was OK for me to mess around with the implementation of the CoProcessing library.
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-02-05 15:11Ken MorelandNew Issue
2010-02-05 15:20Utkarsh AyachitStatusbacklog => tabled
2010-02-05 15:20Utkarsh AyachitAssigned To => Andy Bauer
2010-02-24 16:41Andy BauerNote Added: 0019611
2010-02-24 16:41Andy BauerStatustabled => @80@
2010-02-24 16:41Andy BauerFixed in Version => Development
2010-02-24 16:41Andy BauerResolutionopen => fixed
2010-03-23 22:04Alan ScottNote Added: 0019999
2010-03-23 22:04Alan ScottStatus@80@ => closed
2011-06-16 13:10Zack GalbreathCategoryFeature Request => Feature
2011-09-01 13:31Utkarsh AyachitFixed in VersionDevelopment => 3.12

Notes
(0019611)
Andy Bauer   
2010-02-24 16:41   
This is fixed by doing suggestion 1. Files changed are CMakeLists.txt, ParaViewConfig.cmake.in, ParaVuewUse.cmake, and CoProcessing/CMakeLists.txt.
(0019999)
Alan Scott   
2010-03-23 22:04   
Assuming fixed.