From 9e08a0d8c5fd8da07cf80cd979f0d0eb1469aa97 Mon Sep 17 00:00:00 2001 From: Albert Cheng Date: Sun, 22 Jul 2012 01:42:30 -0500 Subject: [svn-r22594] Bug fix: HDFFV-8105 Description: The test command in COPY_TESTFILES_TO_TESTDIR used the wrong operator -a which means and. It is not sufficient to test mere existence as $tstfile could have been a directory or other non-file things. If the file is not readable, the $CP command would fail but its return code was not checked. Solution: Get rid of the test command and go ahead to $CP it and then check its returned code and report errors if detected. Tested; committested. (note that ostrick still failed on errors committed by other staff). --- release_docs/RELEASE.txt | 3 +++ tools/h5diff/testh5diff.sh | 6 ++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/release_docs/RELEASE.txt b/release_docs/RELEASE.txt index fe9e703..338cc61 100644 --- a/release_docs/RELEASE.txt +++ b/release_docs/RELEASE.txt @@ -702,6 +702,9 @@ Bug Fixes since HDF5-1.8.0 release Tools ----- + - h5diff: Fixed the Function COPY_TESTFILES_TO_TESTDIR() of + testh5diff.sh to better report when there is an error in the file + copying. HDFFV-8105 (AKC -2012/07/22) - h5diff: Fixed not to check and display dangling link status without --follow-symlinks option. This also improved performance when comparing lots of external links without the --follow-symlinks diff --git a/tools/h5diff/testh5diff.sh b/tools/h5diff/testh5diff.sh index cb91a19..a208acf 100755 --- a/tools/h5diff/testh5diff.sh +++ b/tools/h5diff/testh5diff.sh @@ -318,11 +318,9 @@ COPY_TESTFILES_TO_TESTDIR() echo $tstfile | tr -d ' ' | grep '^#' > /dev/null RET=$? if [ $RET -eq 1 ]; then - if [ -a $tstfile ]; then - $CP -f $tstfile $TESTDIR - else + $CP -f $tstfile $TESTDIR + if [ $? -ne 0 ]; then echo "Error: FAILED to copy $tstfile ." - echo " $tstfile doesn't exist!" # Comment out this to CREATE expected file exit $EXIT_FAILURE -- cgit v0.12