Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Jun 2019 22:00:38 +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: r349179 - head/share/mk
Message-ID:  <201906182200.x5IM0cqY030942@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bdrewery
Date: Tue Jun 18 22:00:38 2019
New Revision: 349179
URL: https://svnweb.freebsd.org/changeset/base/349179

Log:
  Rework r349061: Don't apply guessed dependencies if there is a custom target.
  
  This is still targeting bin/sh cyclic dependency issues.  Only apply
  guessed dependencies that are explicitly set for an object (which
  gnu/lib/cc/cc_tools needs) and if no custom target exists with its
  own dependencies.
  
  This was manifesting as a missing yacc.h in usr.bin/mkesdb_static when
  built without -j (or -B). No actual yacc.h dependency ordering was
  defined but with -j it got lucky and built fine.
  
  Before r349061 the behavior was different for META_MODE but that logic
  difference isn't needed.
  
  X-MFC-With:	r349061
  Sponsored by:	DellEMC

Modified:
  head/share/mk/bsd.dep.mk

Modified: head/share/mk/bsd.dep.mk
==============================================================================
--- head/share/mk/bsd.dep.mk	Tue Jun 18 21:05:10 2019	(r349178)
+++ head/share/mk/bsd.dep.mk	Tue Jun 18 22:00:38 2019	(r349179)
@@ -264,7 +264,9 @@ _depfile=	${.OBJDIR}/${_dep_obj}
 # - For meta mode we still need to know which file to depend on to avoid
 #   ambiguous suffix transformation rules from .PATH.  Meta mode does not
 #   use .depend files when filemon is in use.
-${__obj}: ${OBJS_DEPEND_GUESS:N*.h}
+.if !target(${__obj})
+${__obj}: ${OBJS_DEPEND_GUESS}
+.endif
 ${__obj}: ${OBJS_DEPEND_GUESS.${__obj}}
 .endif	# !exists(${_depfile}) || defined(_meta_filemon)
 .endfor



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