View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003099 | ThirdParty | Bug | public | 2018-11-05 11:16 | 2018-12-29 11:17 |
| Reporter | fantaz | Assigned To | chris | ||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | Linux | OS | Ubuntu | OS Version | 18.10 |
| Summary | 0003099: Paraview compilation fails on Ubuntu 18.10 | ||||
| Description | The Paraview compilation fails on Ubuntu 18.10 due to a bug in ParaView''s CMakeLists.txt script in Plugins/StreamLinesRepresentation/ The error is reported when the non existing doc folder from Plugins/StreamLinesRepresentation cannot be copied. It is a known bug (links are in the Additional Information), and the resolution is simple: Delete or comment the line 46 from the $WM_THIRD_PARTY_DIR/ParaView-5.4.0/Plugins/StreamLinesRepresentation/CMakeLists.txt The line to be commented or deleted: DOCUMENTATION_DIR "${CMAKE_CURRENT_SOURCE_DIR}/doc" | ||||
| Steps To Reproduce | Follow the installation instructions from OF wiki (https://openfoamwiki.net/index.php/Installation/Linux/OpenFOAM-6/Ubuntu) for Ubuntu 18.04 | ||||
| Additional Information | * The actual [bug report](https://gitlab.kitware.com/paraview/paraview/issues/18330): * [Merge request](https://gitlab.kitware.com/paraview/paraview/merge_requests/2647) discussion * The actual [commit](https://gitlab.kitware.com/paraview/paraview/commit/f9c7e4b01341f45aec55f452b583c544e2557ca4) in ParaView repository. * https://gitlab.com/snippets/1774169 - rough instructions for a resolution | ||||
| Tags | No tags attached. | ||||
|
|
No expert here, but the offending line could be deleted via sed command: sed -i.bak '/DOCUMENTATION_DIR /d' $ParaView_SOURCE_DIR/Plugins/StreamLinesRepresentation/CMakeLists.txt by adding a function in ParaViewFunctions file, which could be called from the makeParaView script e.g. removeStreamLinesRepresentationCMakeListsLine() { local ver=$(getVersion) if [ "$ver" == "5.4.0" ] then sed -i.bak '/DOCUMENTATION_DIR /d' $ParaView_SOURCE_DIR/Plugins/StreamLinesRepresentation/CMakeLists.txt fi } |
|
|
Thanks for your notes, I am sure people compiling ParaView on Ubuntu 18.04 will find them useful. We don't provide the ParaView sources with the OpenFOAM ThirdParty directory anymore so I don't think there is anything we need to do right now. I hope KitWare will resolve these and other build problems we have experienced in version 5.6. |
|
|
Sorry if I wasn't clear, but the issue is about compiling ParaView 5.4.0 (with Qt-4) for OF-6 on Ubuntu 18.10. The Ubuntu 18.10 uses CMake 3.12, and that triggers the compilation error. The compilation instructions for 18.04 apply to 18.10, except for that minor tweak. |
|
|
It appears the problem is with ParaView rather than OpenFOAM and it is not clear what we can do about it. Have you reported the problem to KitWare? |
|
|
No I didn't report the problem to KitWare. And I agree it's ParaView's problem, not OF. I just hoped that a little tweak in the makeParaView and ParaViewFunctions could allow Ubuntu 18.10 users to build and use OF. Bottom line: I think that I found the solution for 18.10 users and thought that it would be cool to share with you. Nothing more. And it's quite understandable if OF devs don't feel the need to add code for an unsupported OS. I get that. Do you guys think that instead of bug report, wiki would be more appropriate place to add instructions for OF-6? |
|
|
It is not clear what changes to our makeParaView will resolve the issue. Have you tested the ParaView-5.6 release candidates? If these solve the problem then the best approach will be to wait for the release of 5.6 and update OpenFOAM to use that version. |
|
|
I haven't had time to look into this myself, but from the overall description here, it looks like the simplest solution for building ParaView 5.4.0 on Ubuntu 18.10 is to build a custom CMake version that is compatible with this version of ParaView. The "makeCmake" script still exists in "ThirdParty-6" and "ThirdParty-dev", along with the README file in that folder which states from where to download CMake 3.9.0... so it's just a matter of... Here are the commands that I usually write on the step-by-step community wiki pages (openfoamwiki.net): cd $WM_THIRD_PARTY_DIR wget -P download https://www.cmake.org/files/v3.9/cmake-3.9.0.tar.gz ./makeCmake > log.makeCmake 2>&1 wmRefresh And that's it, problem solved, ParaView 5.4.0 should be compilable without additional tinkering ;) @Chris and @Henry: Here at openfoam.org: - In the page https://openfoam.org/download/source/third-party-software/ - is already indicated that: The ThirdParty repository contains scripts for compiling these software packages. The README file contains some information about the compiling the software, but we will provide step-by-step instructions for compiling Scotch/PT-Scotch and ParaView below. - So on the section "Installing ParaView" on that page, it's only missing the indication that CMake may need to be custom built on some unsupported systems by running makeCmake, after downloading+unpacking the respective CMake package that is reported on the README file. |
|
|
I agree. Will try to test on 5.6 RC. Thanks for your time. |
|
|
@wyldckat: Yes, that's the right approach. Thanks for the feedback |
|
|
@fantaz I think it may be possible to install cmake 3.10 on Ubuntu 18.10 with: sudo apt-get install cmake=3.10.2-1ubuntu2 If that version is OK, then this might be the simplest solution. |
|
|
@chris I'm not really fond of making changes to default set up. IMHO, the solution proposed by wyldckat is both elegant and non intrusive. |
|
|
@chris: i leave it up to you to decide on whether it's worth updating the instructions at https://openfoam.org/download/source/third-party-software/ regarding the CMake version, as indicated in your comment above and/or my comment above. |
|
|
For Ubuntu 18.10, compiling ParaView 5.6.0 works fine. Packages for OpenFOAM 6 and ParaView 5.6.0 are anyway available precompiled for Ubuntu 18.04 and 18.10: https://openfoam.org/news/paraview-5-6/ |
|
|
See final note |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2018-11-05 11:16 | fantaz | New Issue | |
| 2018-11-05 11:58 | fantaz | Note Added: 0010132 | |
| 2018-11-05 12:24 | henry | Note Added: 0010133 | |
| 2018-11-06 07:30 | fantaz | Note Added: 0010136 | |
| 2018-11-06 09:00 | henry | Note Added: 0010139 | |
| 2018-11-06 10:17 | fantaz | Note Added: 0010140 | |
| 2018-11-06 10:34 | henry | Note Added: 0010141 | |
| 2018-11-06 10:51 | wyldckat | Note Added: 0010142 | |
| 2018-11-06 10:53 | fantaz | Note Added: 0010143 | |
| 2018-11-06 10:58 | fantaz | Note Added: 0010144 | |
| 2018-11-07 21:33 | chris | Note Added: 0010165 | |
| 2018-11-08 07:00 | fantaz | Note Added: 0010168 | |
| 2018-12-28 20:46 | wyldckat | Assigned To | => chris |
| 2018-12-28 20:46 | wyldckat | Status | new => assigned |
| 2018-12-28 20:46 | wyldckat | Category | Contribution => Bug |
| 2018-12-28 20:48 | wyldckat | Note Added: 0010241 | |
| 2018-12-29 11:17 | chris | Note Added: 0010244 | |
| 2018-12-29 11:17 | chris | Status | assigned => resolved |
| 2018-12-29 11:17 | chris | Resolution | open => fixed |
| 2018-12-29 11:17 | chris | Note Added: 0010245 | |
| 2018-12-29 11:17 | chris | Status | resolved => closed |
