CMake Option for Compiling Commandline Utilities #722
Conversation
…ENABLE_UTILS`` option. Other cmake updates include automatically searching for opengl on system.
Co-authored-by: Mickael PHILIT <mickey.phy@gmail.com>
| @@ -338,8 +350,8 @@ if (CGNS_ENABLE_HDF5) | |||
| endif () | |||
|
|
|||
| # Check if HDF5 version is 1.8 or greater | |||
| if (HDF5_VERSION VERSION_LESS "1.8.0") | |||
There was a problem hiding this comment.
I haven't found any cmake documentation about HDF5_VERSION_STRING ... while HD5_VERSION should be properly defined.
There was a problem hiding this comment.
Correct, It should be HDF5_VERSION, which is set by find_package HDF5
There was a problem hiding this comment.
After updating to HDF5 version 1.10.9 the error I got doesn't occur, so yes the correct variable should be HDF5_VERSION. I'll push up the change.
There was a problem hiding this comment.
What happens for the case CGNS_ENABLE_CGNSTOOLS=ON, CGNS_ENABLE_UTILS=OFF. Does it still build the command line tools?
The additional CMake commands should not impact the CGNS_ENABLE_CGNSTOOLS option. Yes CGNSTOOLS should be able to compile things like (cgns_to_xxx, xxx_to_cgns, etc). UTILS just allows for compiling the command line tools without the Tk/Tcl GUI.
|
What happens for the case CGNS_ENABLE_CGNSTOOLS=ON, CGNS_ENABLE_UTILS=OFF. Does it still build the command line tools? |
|
Can you add to the Mac github action the option -DCGNS_ENABLE_UTILS=ON? |
Added a new CMake option which enables the build of command line utilities (e.g. cgns_to_plot3d, cgns_to_aflr3, etc.) without having to build the entire cgnstools suite. This avoids Tk/Tcl issues, especially for folks on Mac OS X and solely want to use the command line utilities without compiling the old-school fortran conversion scripts.
Example build case would look like:
cd CGNS/mkdir buildcmake .. -DCGNS_ENABLE_UTILS=ON...
make -j4