diff options
author | Simon McVittie <smcv@debian.org> | 2016-03-31 14:32:55 +0100 |
---|---|---|
committer | Colin Walters (automation) <walters+githubbot@verbum.org> | 2016-03-31 14:22:06 +0000 |
commit | 9dafc820241ed6e663ad4093660fc255f549aa26 (patch) | |
tree | 655421fca817d4d5c38707fdfb14411607bb0036 | |
parent | 8cda8b6866f86ffdda8622c9108e949d0ed6c27f (diff) | |
download | ostree-9dafc820241ed6e663ad4093660fc255f549aa26.tar.gz |
Probe for GNU parallel more accurately
moreutils parallel isn't compatible with the command-line syntax
used here. Because it doesn't implement GNU-style --help and exits 1
when that option is given, this test was correctly skipped when
using moreutils parallel, but only by mistake.
moreutils parallel might conceivably gain --help and --version in
future, but hopefully nothing incompatible with GNU parallel is going
to gain a --gnu option.
Also use the --gnu option to force the new command-line semantics;
some versions optionally supported an incompatible command-line syntax
taken from moreutils parallel.
Signed-off-by: Simon McVittie <smcv@debian.org>
Closes: #232
Approved by: cgwalters
-rwxr-xr-x | tests/test-admin-locking.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/test-admin-locking.sh b/tests/test-admin-locking.sh index 564295d2..01814f9f 100755 --- a/tests/test-admin-locking.sh +++ b/tests/test-admin-locking.sh @@ -25,8 +25,8 @@ set -euo pipefail setup_os_repository "archive-z2" "syslinux" # If parallel is not installed, skip the test -if ! parallel --help >/dev/null 2>&1; then - echo "1..0 # SKIP no /usr/bin/parallel" +if ! parallel --gnu /bin/true </dev/null >/dev/null 2>&1; then + echo "1..0 # SKIP GNU parallel not available" exit 0 fi @@ -42,7 +42,7 @@ echo "rev=${rev}" ${CMD_PREFIX} ostree admin deploy --karg=root=LABEL=MOO --karg=quiet --os=testos testos:testos/buildmaster/x86_64-runtime assert_has_dir sysroot/boot/ostree/testos-${bootcsum} -parallel_cmd=parallel +parallel_cmd="parallel --gnu" if parallel --help | grep -q -e --no-notice; then parallel_cmd="${parallel_cmd} --no-notice" fi |