cvs commit: src Makefile Makefile.inc1

Alexander Leidinger Alexander at Leidinger.net
Wed May 16 10:40:23 UTC 2007


Quoting Dag-Erling Smorgrav <des at FreeBSD.org> (from Wed, 16 May 2007  
08:46:36 +0000 (UTC)):

> des         2007-05-16 08:46:36 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     .                    Makefile Makefile.inc1
>   Log:
>   Expose all of {check,delete}-old{,-dirs,-files,-libs}.

The delete-old target only removes static libs. Yes, they are libs,  
but they are not used in a life system, they are only used while  
developing. That's the reason why I didn't add "libraries" to the  
description of the delete-old target. The delete-old-libs target  
exists to remove shared libs, which you should not do unconditionally  
on a system (make sure no ports uses them anymore).

For the check-old target there's no reason to split up the targets, as  
you can list them unconditionally. For developing convenience I did  
chose to have the check-old-XXX targets in Makefile.inc1 ("make -f  
Makefile.inc1 TARGET" works just fine). I see no need to expose all  
targets.

So please revert this or do a forced commit to provide a good reason  
why they need to be exposed.

As your recent commits to the delete-old/check-old part created a lot  
of mails and follow up commits because of them, I suggest to discuss  
further changes to the targets before committing the changes.

Bye,
Alexander.

-- 
Any clod can have the facts, but having opinions is an art.
		-- Charles McCabe

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-src mailing list