View Issue Details

IDProjectCategoryView StatusLast Update
0001986ThirdParty[All Projects] Bugpublic2016-03-27 20:23
ReporterzordiackAssigned Tohenry 
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
PlatformGNU/LinuxOSUbuntuOS Version14.04
Product Version 
Fixed in Version 
Summary0001986: ParaView 4.4.0 (and 5.0.0) fails to detect source code version
DescriptionWhen compiling ParaView 4.4.0 or 5.0.0 source code, the configuration script detects wrong version of source code (paraview-3.0) and installs into wrong location. Because ParaView installation is not where OpenFOAM is looking for it (paraview-4.4), the compilation of OpenFOAM readers for ParaView will fail.
Steps To ReproduceCompile ParaView 4.4.0 or ParaView 5.0.0 sources with ./makeParaView4 script.
Additional InformationProposed fix:

Edit file ParaView-5.0.0/CMake/ParaViewDetermineVersion.cmake and comment out lines 41-50 (otherwise source version detection fails, this is also a bug in 4.4.0):

  if (NOT PARAVIEW_GIT_DESCRIBE)
# if (EXISTS ${git_command})
# execute_process(
# COMMAND ${git_command} describe
# WORKING_DIRECTORY ${source_dir}
# RESULT_VARIABLE result
# OUTPUT_VARIABLE output
# ERROR_QUIET
# OUTPUT_STRIP_TRAILING_WHITESPACE
# ERROR_STRIP_TRAILING_WHITESPACE)
# endif()
  else()

Now the installation script will detect the source code version from directory name, and the installation location is correct.
TagsNo tags attached.

Relationships

related to 0002034 resolvedhenry Proposed environment override for isolating ParaView's ".git" lookup 

Activities

henry

2016-02-03 13:02

manager   ~0005894

It is not clear what OpenFOAM version you are building but I have already implemented a work-around for this bug in ParaView in OpenFOAM-dev and 3.0.x.

henry

2016-02-03 18:54

manager   ~0005902

Resolved in OpenFOAM-dev by

commit bea095cc022ec8d9a9e1c438f32a0278ea397fc3
Author: Henry Weller <http://cfd.direct>
Date: Mon Jan 18 18:15:29 2016 +0000

    etc/config/paraview.*: Update include/lib directory name seaching to paraview-3.0 for ParaView-4.?

Resolved in OpenFOAM-3.0.x by

commit c3046cd6e30e49293f7c636150c43047a3e8b3da
Author: Henry Weller <http://cfd.direct>
Date: Mon Jan 18 18:13:00 2016 +0000

    etc/config/paraview.*: Update include/lib directory name seaching to paraview-3.0 for ParaView-4.?

Issue History

Date Modified Username Field Change
2016-02-03 11:56 zordiack New Issue
2016-02-03 13:02 henry Note Added: 0005894
2016-02-03 18:54 henry Note Added: 0005902
2016-02-03 18:54 henry Status new => closed
2016-02-03 18:54 henry Assigned To => henry
2016-02-03 18:54 henry Resolution open => fixed
2016-03-27 20:23 wyldckat Relationship added related to 0002034