cvs commit: src Makefile.inc1

Alexander Leidinger Alexander at Leidinger.net
Sun May 20 10:29:21 UTC 2007


Quoting Dag-Erling Smorgrav <des at FreeBSD.org> (Sat, 19 May 2007 20:34:29 +0000 (UTC)):

> des         2007-05-19 20:34:29 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     .                    Makefile.inc1 
>   Log:
>   Revision 1.576 removed too much.  I didn't notice because my /usr is on
>   ZFS, which doesn't support flags...

My suggestion was to remove the chflags and add a echo in the failure
case of rm, because in the current way the flags are removed even if
the user tells to not remove a file at the interactive rm prompt and
the user is not told about it. In my original version there was an echo
that the flags are removed unconditionally.

Dag-Erling doesn't like the echo of removing flags when there are no
flags, and I don't like that there's no echo when removing flags. To
try to come to a consensus I suggested the above.

It seems we need a 3rd opinion (provide your bikeshed color wisely
please, think 10 times before you reply).

Bye,
Alexander.

-- 
It is wise to keep in mind that neither success nor failure is ever final.
		-- Roger Babson
http://www.Leidinger.net  Alexander @ Leidinger.net: PGP ID = B0063FE7
http://www.FreeBSD.org     netchild @ FreeBSD.org  : PGP ID = 72077137


More information about the cvs-all mailing list