]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
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 3b40375c587d8eba5d4ac4810dc40815d005a1e2..c824e41a1aa2dac14076b3645d3e717fe27f3256 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 304b9184696b2fe25716a39f2cf66bfdfac2f5c7..e853b0a2f9a55dd31c0d0b244a103ee2575c7a0e 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:) ;;