View Issue Details

IDProjectCategoryView StatusLast Update
0003099ThirdPartyContributionpublic2018-11-08 07:00
ReporterfantazAssigned To 
PrioritynormalSeveritymajorReproducibilityalways
Status newResolutionopen 
PlatformLinuxOSUbuntuOS Version18.10
Product Version6 
Fixed in Version 
Summary0003099: Paraview compilation fails on Ubuntu 18.10
DescriptionThe 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 ReproduceFollow 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
TagsNo tags attached.

Activities

fantaz

2018-11-05 11:58

reporter   ~0010132

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
}

henry

2018-11-05 12:24

manager   ~0010133

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.

fantaz

2018-11-06 07:30

reporter   ~0010136

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.

henry

2018-11-06 09:00

manager   ~0010139

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?

fantaz

2018-11-06 10:17

reporter   ~0010140

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?

henry

2018-11-06 10:34

manager   ~0010141

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.

wyldckat

2018-11-06 10:51

updater   ~0010142

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.

fantaz

2018-11-06 10:53

reporter   ~0010143

I agree.
Will try to test on 5.6 RC. Thanks for your time.

fantaz

2018-11-06 10:58

reporter   ~0010144

@wyldckat:
Yes, that's the right approach. Thanks for the feedback

chris

2018-11-07 21:33

manager   ~0010165

@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.

fantaz

2018-11-08 07:00

reporter   ~0010168

@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.

Issue History

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