Matthew Burgess wrote: > The unibyte-bracket-expr test does 'cp $in /t' which, if the tests are > being run as root, will cause '/t' to be created (and obviously, if > the tests aren't being run as root, that command should fail). The > attached patch simply removes that copy command, as that didn't affect > the status of the test on my machine (x86 Linux) and I couldn't see > any other reference to '/t'.
Thanks again. Here's the patch I expect to push in your name. Since you're listed as "Author", yet I wrote the log entry, I'll wait for your ACK before pushing it. >From a7e745b9d40e3c94cc5527a257c37f334fa0c9d0 Mon Sep 17 00:00:00 2001 From: Matthew Burgess <[email protected]> Date: Wed, 31 Aug 2011 21:18:54 +0200 Subject: [PATCH] tests: remove debug code that would cp to /t * tests/unibyte-bracket-expr: Remove debug artifact introduced by 2011-06-02 commit de5f7000, "tests: exercise a uni-byte [...] bug: requires ru_RU.KOI8-R". [bug introduced in grep-2.9] --- tests/unibyte-bracket-expr | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/tests/unibyte-bracket-expr b/tests/unibyte-bracket-expr index a0b51dd..45ad5d9 100755 --- a/tests/unibyte-bracket-expr +++ b/tests/unibyte-bracket-expr @@ -32,7 +32,6 @@ for i in 8 9 a b c d e f; do in=in-$i$j b=$(printf "\\x$i$j") echo "$b" > $in || framework_failure_ - cp $in /t grep "[$b]" $in > out || fail=1 compare out $in || fail=1 done -- 1.7.7.rc0.362.g5a14
