tests: fix syntax error in setuidgid_has_perm_
authorBernhard Voelker <mail@bernhard-voelker.de>
Thu, 1 Nov 2012 15:50:51 +0000 (16:50 +0100)
committerPádraig Brady <P@draigBrady.com>
Thu, 1 Nov 2012 17:37:58 +0000 (17:37 +0000)
An invalid sed expression lead to the following error:
  + sed -n 'ls/.* //p'
  sed: -e expression #1, char 2: extra characters after command
Use "one" instead of "el".
Bug introduced in v8.19-111-g51a4b04.

* init.cfg (setuidgid_has_perm_): s/ls/1s/.
* NEWS (Build-related): Mention the fix.

NEWS
init.cfg

diff --git a/NEWS b/NEWS
index 3b40375..c824e41 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -21,6 +21,11 @@ GNU coreutils NEWS                                    -*- outline -*-
   build failure when building from modified sources, as is common practice
   for a patched distribution package.
 
+  The check in the root-only tests to test whether our dummy user,
+  $NON_ROOT_USERNAME, is able to run binaries from the build directory
+  failed.  As a result, these tests have been skipped unnecessarily.
+  [bug introduced in coreutils-8.20]
+
 
 * Noteworthy changes in release 8.20 (2012-10-23) [stable]
 
index 304b918..e853b0a 100644 (file)
--- a/init.cfg
+++ b/init.cfg
@@ -352,7 +352,7 @@ setuidgid_has_perm_()
 {
   local rm_version=$(
     setuidgid $NON_ROOT_USERNAME env PATH="$PATH" rm --version |
-    sed -n 'ls/.* //p'
+    sed -n '1s/.* //p'
   )
   case ":$rm_version:" in
       :$PACKAGE_VERSION:) ;;