Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add code to find CGNS tools #187

Merged
merged 1 commit into from
Jul 20, 2018
Merged

Conversation

gsjaardema
Copy link
Contributor

Add code that will find CGNS tools -- cgnscheck, cgnsdiff, cgnslist, cgnscompress, cgnsconvert, cgnsnames, cgnsupdate.

For each of these that is found, the symbol CGNS_{command_uppercase}_BINARY will be defined. These are useful for tests of applications/routines that access CGNS files -- the testing code can check whether a specific command exists and act accordingly.

The NetCDF and HDF5 Find routines contain similar code and naming conventions.

Add code that will find CGNS tools -- cgnscheck, cgnsdiff, cgnslist, cgnscompress, cgnsconvert, cgnsnames, cgnsupdate.

For each of these that is found, the symbol CGNS_{command_uppercase}_BINARY will be defined.  These are useful for tests of applications/routines that access CGNS files -- the testing code can check whether a specific command exists and act accordingly.

The NetCDF and HDF5 Find routines contain similar code and naming conventions.
@bartlettroscoe
Copy link
Member

Okay, I will merge this along with #179, #177, #128.

@gsjaardema
Copy link
Contributor Author

Any idea when this will be merged?

@bartlettroscoe
Copy link
Member

Sorry, just saw this (I don't seem to be getting many of these GitHub email notifications).

I will move this back up higher on my personal todo list.

@gsjaardema
Copy link
Contributor Author

Is this still on your backlog for merging?

@bartlettroscoe bartlettroscoe merged commit ae331bf into TriBITSPub:master Jul 20, 2018
@bartlettroscoe
Copy link
Member

Sorry, I should have merged this a long time ago. I just don't have any way to test this.

@gsjaardema gsjaardema deleted the patch-4 branch November 17, 2018 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants