svn commit: r273500 - stable/10/share/mk

Garrett Cooper ngie at FreeBSD.org
Thu Oct 23 00:51:53 UTC 2014


Author: ngie
Date: Thu Oct 23 00:51:52 2014
New Revision: 273500
URL: https://svnweb.freebsd.org/changeset/base/273500

Log:
  MFC r271365:
  
    Remove many false positives with make checkdpadd
  
    - Reduce DPADD and LDADD in checkdpadd to -l<foo>
    - Skip over -Wl,[es]*-group because -Wl,--end-group and
      -Wl,--start-group might be required to properly link objects (see
      usr.bin/clang/lldb as an example)
  
    This caveat has been present for a while with some components of
    the build. However, these false positives were made more more apparent
    after r269648.
  
    Phabric: D635
    Reviewed by: jmmv (an earlier version)
    PR: 192730

Modified:
  stable/10/share/mk/bsd.dep.mk
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/mk/bsd.dep.mk
==============================================================================
--- stable/10/share/mk/bsd.dep.mk	Thu Oct 23 00:43:08 2014	(r273499)
+++ stable/10/share/mk/bsd.dep.mk	Thu Oct 23 00:51:52 2014	(r273500)
@@ -215,8 +215,10 @@ cleandepend:
 .endif
 
 .if !target(checkdpadd) && (defined(DPADD) || defined(LDADD))
-_LDADD_FROM_DPADD=	${DPADD:C;^/usr/lib/lib(.*)\.a$;-l\1;}
-_LDADD_CANONICALIZED=	${LDADD:S/$//}
+_LDADD_FROM_DPADD=	${DPADD:R:T:C;^lib(.*)$;-l\1;g}
+# Ignore -Wl,--start-group/-Wl,--end-group as it might be required in the
+# LDADD list due to unresolved symbols
+_LDADD_CANONICALIZED=	${LDADD:N:R:T:C;^lib(.*)$;-l\1;g:N-Wl,--[es]*-group}
 checkdpadd:
 .if ${_LDADD_FROM_DPADD} != ${_LDADD_CANONICALIZED}
 	@echo ${.CURDIR}


More information about the svn-src-stable mailing list