bug-coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Failed: phase compiling: coreutils-6.9-3 failed


From: Jim Meyering
Subject: Re: Failed: phase compiling: coreutils-6.9-3 failed
Date: Wed, 02 May 2007 23:36:09 +0200

>> On Mac OS X 10.4.9 with gcc 4.0.1 making ends in:
> (snip)
>>      rm: illegal option -- -
>>      usage: rm [-f | -i] [-dPRrvW] file ...
>>             unlink file
>>      ./fail-2eperm: cannot access required version (6.9) of rm
>>      out exp differ: char 5, line 1
>>      1,2c1
>>      < rm: a/b: Operation not permitted
>>      < rm: a: Directory not empty
>>      ---
>>      > rm: cannot remove `a/b': Operation not permitted
>>      FAIL: fail-2eperm
>
> I can't reproduce your problem, but I the attached patch will fix it.

Thank you.
I've applied that patch.

 ChangeLog            |    7 +++++++
 THANKS               |    2 ++
 tests/rm/fail-2eperm |    6 +++---
 3 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3149e49..a2bb132 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2007-05-02  Jim Meyering  <address@hidden>
+
+       Invoke rm via 'setuidgid ... env PATH="$PATH" ...', as in fail-eperm.
+       * tests/rm/fail-2eperm: Patch from AIDA Shinra.
+       Reported by Peter Dyballa.
+       * THANKS: Add AIDA Shinra and Peter Dyballa.
+
 2007-04-28  Paul Eggert  <address@hidden>

        * src/nohup.c (usage): Describe how standard input and output
diff --git a/THANKS b/THANKS
index 2033c38..533ea94 100644
--- a/THANKS
+++ b/THANKS
@@ -12,6 +12,7 @@ Aaron Hawley                        address@hidden
 Achim Blumensath                    address@hidden
 Adam Klein                          address@hidden
 Adrian Bunk                         address@hidden
+AIDA Shinra                         address@hidden
 Akim Demaille                       address@hidden
 Alain Magloire                      address@hidden
 Alan Iwi                            address@hidden
@@ -399,6 +400,7 @@ Pawel Prokop                        address@hidden
 Per Cederqvist                      address@hidden
 Per Kristian Hove                   address@hidden
 Peter Breitenlohner                 address@hidden
+Peter Dyballa                       address@hidden
 Peter Eriksson                      address@hidden
 Peter Fales                         address@hidden
 Peter Horst                         address@hidden
diff --git a/tests/rm/fail-2eperm b/tests/rm/fail-2eperm
index 0761950..5038175 100755
--- a/tests/rm/fail-2eperm
+++ b/tests/rm/fail-2eperm
@@ -2,7 +2,7 @@
 # Like fail-eperm, but the failure must be for a file encountered
 # while trying to remove the containing directory with the sticky bit set.

-# Copyright (C) 2003, 2006 Free Software Foundation, Inc.
+# Copyright (C) 2003, 2006-2007 Free Software Foundation, Inc.

 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -52,13 +52,13 @@ fail=0

 # Try to ensure that $NON_ROOT_USERNAME can access
 # the required version of rm.
-rm_version=`setuidgid $NON_ROOT_USERNAME rm --version|sed -n '1s/.* //p'`
+rm_version=`setuidgid $NON_ROOT_USERNAME env PATH="$PATH" rm --version|sed -n 
'1s/.* //p'`
 case $rm_version in
   $PACKAGE_VERSION) ;;
   *) echo "$0: cannot access required version ($PACKAGE_VERSION) of rm" 1>&2
      fail=1 ;;
 esac
-setuidgid $NON_ROOT_USERNAME rm -rf a 2> out-t && fail=1
+setuidgid $NON_ROOT_USERNAME env PATH="$PATH" rm -rf a 2> out-t && fail=1

 # On some systems, we get `Not owner'.  Convert it.
 # On other systems (HPUX), we get `Permission denied'.  Convert it, too.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]