[svn-r11866] Purpose:

bug fix 504

Description:
testpar/t_mpi would hang if  $HDF5_NOCLEANUP is set.  E.g.,
% env HDF5_NOCLEANUP=yes mpirun -np 3 ./t_mpi

This happened because the environment variables are not exported to all
mpi processes by the mpirun command.  So, some attempted to do cleanup
while others don't and some hang waiting for others to act.

Solution:
Instead individual program checking getenv, they all just called h5_cleanup
no matter.  h5_cleanup now uses getenv_all to check the $HDF5_NOCLEANUP if
it is in parallel mode.

Platforms tested:
h5committested, tested pp in heping too.

1 files changed, 2 insertions, 6 deletions

diff --git a/testpar/testphdf5.c b/testpar/testphdf5.c
index a07ac4a..f607d8f 100644
--- a/testpar/testphdf5.c
+++ b/testpar/testphdf5.c
@@ -516,12 +516,8 @@ int main(int argc, char **argv)
     if (MAINPROCESS && GetTestSummary())
         TestSummary();
 
-    /* Clean up test files, if allowed */
-    if (GetTestCleanup() && !getenv("HDF5_NOCLEANUP"))
-	h5_cleanup(FILENAME, fapl);
-    else
-	/* h5_cleanup would have closed fapl.  Now must do it explicitedly */
-	H5Pclose(fapl);
+    /* Clean up test files */
+    h5_cleanup(FILENAME, fapl);
 
     nerrors += GetTestNumErrs();