This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU Inetutils ".
The branch, master has been updated
via 4a46c58ba2f722e5142e5d72d69018eb52254a6a (commit)
from 9652277837d7f3cf66ef33dbea1f069109e2ac6d (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://git.savannah.gnu.org/cgit/inetutils.git/commit/?id=4a46c58ba2f722e5142e5d72d69018eb52254a6a
commit 4a46c58ba2f722e5142e5d72d69018eb52254a6a
Author: Mats Erik Andersson <[email protected]>
Date: Wed Aug 13 23:40:43 2014 +0200
tests/libls.sh: Update testing method.
Avoid a line enumeration using `sed', which seemingly
caused a false negative in the build daemon of NixOS.
diff --git a/ChangeLog b/ChangeLog
index 4b5eb0d..c4dd37b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2014-08-13 Mats Erik Andersson <[email protected]>
+
+ * tests/libls.sh <VERBOSE>: Execute `set -x'.
+ (REPLY_a1, REPLY_A1, diff): Changed. Update the check
+ based on these, thus avoiding `sed -n '$='' which could
+ not be trusted on NixOS' build daemon.
+
2014-08-11 Mats Erik Andersson <[email protected]>
* tests/libls.sh: Improve failure diagnostic when testing
diff --git a/tests/libls.sh b/tests/libls.sh
index 29a4f41..f69a559 100755
--- a/tests/libls.sh
+++ b/tests/libls.sh
@@ -42,11 +42,15 @@ if test -z "${VERBOSE+set}"; then
bucket='>/dev/null'
fi
+if test -n "${VERBOSE:+set}"; then
+ set -x
+fi
+
# Several runs with different switches are compared by
# a simple count of printed lines.
#
-REPLY_a1=`$LS -a1 | $SED -n '$='`
-REPLY_A1=`$LS -A1 | $SED -n '$='`
+REPLY_a1=`$LS -a1`
+REPLY_A1=`$LS -A1`
REPLY_C=`$LS -C`
REPLY_Cf=`$LS -Cf`
@@ -71,10 +75,12 @@ REPLY_Cuts=`$LS -Cuts ..`
#
errno=0
-test $REPLY_a1 -eq `expr $REPLY_A1 + 2` ||
+diff=`{ echo "$REPLY_a1"; echo "$REPLY_A1"; } | sort | uniq -u`
+
+test `echo "$diff" | wc -l` -eq 2 &&
+test `echo "$diff" | grep -c -v -e '^\.\{1,2\}$'` -eq 0 ||
{ errno=1; echo >&2 'Failed to tell switch -a apart from -A.'
# Attempt a diagnosis.
- diff=`{ $LS -a1 && $LS -A1; } | sort | uniq -u`
if test -z "$diff"; then
echo >&2 'Flags -a and -A produce identical lists.'
else
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 7 +++++++
tests/libls.sh | 14 ++++++++++----
2 files changed, 17 insertions(+), 4 deletions(-)
hooks/post-receive
--
GNU Inetutils
_______________________________________________
Commit-inetutils mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/commit-inetutils