diff options
| author | Egbert Eich <eich@suse.com> | 2022-11-28 19:19:31 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-28 19:19:31 (GMT) |
| commit | 158dff6b503436e275777d293c0bfc1d9714eaa0 (patch) | |
| tree | 1dc4221b6b40681a39ff62f8e765abe90e1d2ee9 | |
| parent | eac2cd54e209cfa9556174f3fc1a592533aa64ad (diff) | |
| download | hdf5-158dff6b503436e275777d293c0bfc1d9714eaa0.zip hdf5-158dff6b503436e275777d293c0bfc1d9714eaa0.tar.gz hdf5-158dff6b503436e275777d293c0bfc1d9714eaa0.tar.bz2 | |
Make sure testsuite can handle multiple dependent scripts (#2256)
Commit afc54d75a19 to the test suite to eliminate a separate
shell script did not take into account that there may be
multiple dependent test scripts which resulted in the
message:
`/bin/sh: line 7: [: too many arguments`.
when running the test suite.
Beware that this new version still makes use of non-posix GNU
extensions to `test`.
Signed-off-by: Egbert Eich <eich@suse.com>
Signed-off-by: Egbert Eich <eich@suse.com>
| -rw-r--r-- | config/conclude.am | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/config/conclude.am b/config/conclude.am index d430573..227f0d3 100644 --- a/config/conclude.am +++ b/config/conclude.am @@ -209,7 +209,14 @@ $(TEST_SCRIPT_CHKSH) $(TEST_SCRIPT_PARA_CHKSH) dummysh.chkexe_: chkname=`basename $(@:.chkexe_=.chkexe)`;\ log=`basename $(@:.chkexe_=.chklog)`; \ echo "============================"; \ - if [ $${chkname} -nt $$cmd ] && [ $${chkname} -nt $(SCRIPT_DEPEND) ]; then \ + newer=true; \ + for i in $${cmd} $(SCRIPT_DEPEND); do \ + if [ $${chkname} -ot $$i ]; then \ + newer=false; \ + break; \ + fi; \ + done; \ + if $${newer}; then \ echo "No need to test $${tname} again."; \ else \ echo "============================" > $${log}; \ |