svn commit: r296122 - head/share/mk

[Bryan Drewery]

Author: bdrewery
Date: Fri Feb 26 22:13:52 2016
New Revision: 296122
URL: https://svnweb.freebsd.org/changeset/base/296122

Log:
  Move PROGS logic to proper place and remove redundant and unneeded logic.
  
  - bsd.progs.mk is safe to include regardless of PROGS/PROGS_CXX/SCRIPTS
    being set.
  - bsd.progs.mk includes bsd.prog.mk always and will bring in
    bsd.files.mk and bsd.obj.mk.
  - DIRDEPS_BUILD: There's no need for _SKIP_BUILD or _SKIP_STAGING as the
    PROGS were implicitly being built by the staging dependency anyway.  This
    was also preventing other objects from building if they were not part of
    the staging sets.
  - DIRDEPS_BUILD: This fixes PROGS without bsd.test.mk not staging.
  
  MFC after:	2 weeks
  Sponsored by:	EMC / Isilon Storage Division

Modified:
  head/share/mk/bsd.progs.mk
  head/share/mk/bsd.test.mk

Modified: head/share/mk/bsd.progs.mk
==============================================================================
--- head/share/mk/bsd.progs.mk	Fri Feb 26 22:13:48 2016	(r296121)
+++ head/share/mk/bsd.progs.mk	Fri Feb 26 22:13:52 2016	(r296122)
@@ -59,11 +59,19 @@ all: ${PROGS}
 
 # We cannot capture dependencies for meta mode here
 UPDATE_DEPENDFILE = NO
+
+.if ${MK_STAGING} != "no"
+.if !empty(PROGS)
+stage_files.prog: ${PROGS}
+.endif
+.endif	# ${MK_STAGING} != "no"
 .endif
 .endif	# PROGS || PROGS_CXX
 
 # These are handled by the main make process.
 .ifdef _RECURSING_PROGS
+MK_STAGING= no
+
 _PROGS_GLOBAL_VARS= CLEANFILES CLEANDIRS CONFGROUPS FILESGROUPS INCSGROUPS \
 		    SCRIPTS
 .for v in ${_PROGS_GLOBAL_VARS}

Modified: head/share/mk/bsd.test.mk
==============================================================================
--- head/share/mk/bsd.test.mk	Fri Feb 26 22:13:48 2016	(r296121)
+++ head/share/mk/bsd.test.mk	Fri Feb 26 22:13:52 2016	(r296122)
@@ -93,30 +93,4 @@ beforecheck realcheck aftercheck check: 
 .ORDER: beforecheck realcheck aftercheck
 check: beforecheck realcheck aftercheck
 
-.ifdef PROG
-# we came here via bsd.progs.mk below
-# parent will do staging.
-MK_STAGING= no
-.endif
-
-.if !empty(PROGS) || !empty(PROGS_CXX) || !empty(SCRIPTS)
 .include <bsd.progs.mk>
-.endif
-.include <bsd.files.mk>
-
-.if !defined(PROG) && ${MK_STAGING} != "no"
-.if !defined(_SKIP_BUILD)
-# this will handle staging if needed
-_SKIP_STAGING= no
-# but we don't want it to build anything
-_SKIP_BUILD=
-.endif
-.if !empty(PROGS)
-stage_files.prog: ${PROGS}
-.endif
-.include <bsd.prog.mk>
-.endif
-
-.if !target(objwarn)
-.include <bsd.obj.mk>
-.endif