View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0010243 | ParaView | Feature | public | 2010-02-05 15:11 | 2011-09-01 13:31 | ||||
Reporter | Ken Moreland | ||||||||
Assigned To | Andy Bauer | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | 3.8 | Fixed in Version | 3.12 | ||||||
Summary | 0010243: ParaView and CoProcessing CMake packages should be loaded together | ||||||||
Description | When 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. | ||||||||
Additional Information | I'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. | ||||||||
Tags | No tags attached. | ||||||||
Project | |||||||||
Topic Name | |||||||||
Type | |||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0019611) Andy Bauer (developer) 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 (manager) 2010-03-23 22:04 |
Assuming fixed. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2010-02-05 15:11 | Ken Moreland | New Issue | |
2010-02-05 15:20 | Utkarsh Ayachit | Status | backlog => tabled |
2010-02-05 15:20 | Utkarsh Ayachit | Assigned To | => Andy Bauer |
2010-02-24 16:41 | Andy Bauer | Note Added: 0019611 | |
2010-02-24 16:41 | Andy Bauer | Status | tabled => @80@ |
2010-02-24 16:41 | Andy Bauer | Fixed in Version | => Development |
2010-02-24 16:41 | Andy Bauer | Resolution | open => fixed |
2010-03-23 22:04 | Alan Scott | Note Added: 0019999 | |
2010-03-23 22:04 | Alan Scott | Status | @80@ => closed |
2011-06-16 13:10 | Zack Galbreath | Category | Feature Request => Feature |
2011-09-01 13:31 | Utkarsh Ayachit | Fixed in Version | Development => 3.12 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |