cvs commit: src/bin/rm rm.1 rm.c

[Doug Barton]

Doug Barton wrote:
> dougb       2005-09-29 20:40:29 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     bin/rm               rm.1 rm.c 
>   Log:
>   Handle the case where the -P flag is specified for a read-only file
>   earlier, and more gracefully. Previously, this combination would be
>   ignored early in the code where permissions are tested and fail later
>   with a very unhelpful "permission denied" error.
>   
>   Instead, test for this flag in the same block that generates the
>   "override?" messages for read-only files, but instead of trying
>   to guess what the user has in mind, generate an error and exit.
>   
>   Update the man page to reflect this new behavior.
>   
>   Not objected to by:     freebsd-hackers@
>   
>   Revision  Changes    Path
>   1.37      +4 -0      src/bin/rm/rm.1
>   1.53      +5 -4      src/bin/rm/rm.c
> 
> http://www.FreeBSD.org/cgi/cvsweb.cgi/src/bin/rm/rm.1.diff?&r1=1.36&r2=1.37&f=h
> http://www.FreeBSD.org/cgi/cvsweb.cgi/src/bin/rm/rm.c.diff?&r1=1.52&r2=1.53&f=h

Assuming there is no overwhelming bikeshed over this change, I'll request an 
MFC prior to 6.0-RELEASE.

I am of course still willing to listen to rational arguments about why this 
change might be a bad idea, or could be handled in a better way.

Doug

-- 

     This .signature sanitized for your protection