Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 Feb 2016 22:13:52 +0000 (UTC)
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r296122 - head/share/mk
Message-ID:  <201602262213.u1QMDqEj033374@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201602262213.u1QMDqEj033374>