summaryrefslogtreecommitdiffstats
path: root/tools/h5diff
diff options
context:
space:
mode:
authorQuincey Koziol <koziol@hdfgroup.org>2013-03-27 18:11:55 (GMT)
committerQuincey Koziol <koziol@hdfgroup.org>2013-03-27 18:11:55 (GMT)
commit86f1fce5f7a699e34ff576905fff265fdb08be4e (patch)
tree46750e3612c2d0004a872a384bcb701b3080721f /tools/h5diff
parent5a14bf5fac8e5228bd3ff96693b789b070874565 (diff)
downloadhdf5-86f1fce5f7a699e34ff576905fff265fdb08be4e.zip
hdf5-86f1fce5f7a699e34ff576905fff265fdb08be4e.tar.gz
hdf5-86f1fce5f7a699e34ff576905fff265fdb08be4e.tar.bz2
[svn-r23469] Description:
Change h5diff_exit() routine to always return a zero exit code when running in parallel, since returning a non-zero exit code can cause MPI implementations to print output and that can throw off our "expected output" comparisons. Note that this change only changes the exit code for situations where an incorrect command-line parameter was given - ph5diff was already returning a zero exit code for all "normal" executions, including those where a serial h5diff would return a non-zero exit code. Tested on: Mac OSX/64 10.8.3 (amazon) w/parallel (Too minor to require h5committest)
Diffstat (limited to 'tools/h5diff')
-rw-r--r--tools/h5diff/ph5diff_main.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/h5diff/ph5diff_main.c b/tools/h5diff/ph5diff_main.c
index 1fc563b..b9bd404 100644
--- a/tools/h5diff/ph5diff_main.c
+++ b/tools/h5diff/ph5diff_main.c
@@ -275,6 +275,9 @@ void h5diff_exit(int status)
if(g_Parallel)
MPI_Finalize();
- exit(status);
+ /* Always exit(0), since MPI implementations do weird stuff when they
+ * receive a non-zero exit value. - QAK
+ */
+ exit(0);
}