View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0002034||ThirdParty||[All Projects] Bug||public||2016-03-27 20:22||2016-03-30 19:30|
|Fixed in Version|
|Summary||0002034: Proposed environment override for isolating ParaView's ".git" lookup|
|Description||In issue #1986 was somewhat reported and a partial fix was done for a problem where the script "CMake/ParaViewDetermineVersion.cmake" in at least ParaView 4.4 and 5.0 will pick-up by mistake the version number "3.0", which lead to installing ParaView's builds at the folders "ParaView-*/*/paraview-3.0".|
The reason why this happens is because in the "ThirdParty-dev" repository is the tag "version-3.0.0": https://github.com/OpenFOAM/ThirdParty-dev/tree/version-3.0.0
This occurs because the command "git describe" is used to collect from this tag for the version information, which will in turn report something like this:
After some researching into how Git provides the ability to isolate paths, my search ended at the manual for "git config": https://git-scm.com/docs/git-config.html - where "core.worktree" mentions a few overriding environment variables and "GIT_DIR" in specific can be used to block this issue from ever happening again... as long as we use the "makeParaView*" scripts.
Attached are the following packages:
- ParaView_Function_updates.tar.gz - Meant to be unpacked in the directory "ThirdParty-dev" and provides the simple fix for the files "etc/tools/ParaView*Functions", namely to run the following command at the end of the function "setDirs()":
- paraview_shell_updates.tar.gz - Meant for the directory "OpenFOAM-dev", which updates the files "etc/config.*/paraview" to no longer use the replacement folder "paraview-3.0" if found.
The first package can be deployed straight away in ThirdParty-dev; but I'm not certain when the second package should be deployed in OpenFOAM-dev, even though it's the development branch, which is in constant evolution.
|Tags||No tags attached.|
ParaView_Function_updates.tar.gz (4,638 bytes)
paraview_shell_updates.tar.gz (2,353 bytes)
Thanks for sorting this out Bruno.
Resolved in OpenFOAM-dev by commit c7179bd0d08d183f4f2a9d0b9af1df3e84d12ca2
Resolved in ThirdParty-dev by commit 8c9c235db686de2612c62112b37e132bfa773b16
|2016-03-27 20:22||wyldckat||New Issue|
|2016-03-27 20:22||wyldckat||Status||new => assigned|
|2016-03-27 20:22||wyldckat||Assigned To||=> henry|
|2016-03-27 20:22||wyldckat||File Added: ParaView_Function_updates.tar.gz|
|2016-03-27 20:23||wyldckat||File Added: paraview_shell_updates.tar.gz|
|2016-03-27 20:23||wyldckat||Relationship added||related to 0001986|
|2016-03-30 19:30||henry||Note Added: 0006066|
|2016-03-30 19:30||henry||Status||assigned => resolved|
|2016-03-30 19:30||henry||Resolution||open => fixed|