[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-10-22 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #6 from commit-h...@freebsd.org ---
A commit references this bug:

Author: ngie
Date: Thu Oct 23 00:51:53 UTC 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
- 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

Changes:
_U  stable/10/
  stable/10/share/mk/bsd.dep.mk

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-10-22 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

Garrett Cooper  changed:

   What|Removed |Added

 Status|Needs MFC   |Issue Resolved
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-09-10 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

Garrett Cooper  changed:

   What|Removed |Added

 Status|Needs Triage|Needs MFC

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-09-10 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #5 from commit-h...@freebsd.org ---
A commit references this bug:

Author: ngie
Date: Wed Sep 10 07:55:52 UTC 2014
New revision: 271365
URL: http://svnweb.freebsd.org/changeset/base/271365

Log:
  Remove many false positives with make checkdpadd

  - Reduce DPADD and LDADD in checkdpadd to -l
  - 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
  MFC after: 2 weeks

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

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-08-19 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #4 from Garrett Cooper  ---
*** Bug 192757 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-08-17 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #3 from yaneurab...@gmail.com ---
I was also playing around with some other patches I had to change LDATF_C to
-latf-c for instance. Again, not sure if this is how things were intended, but
specifying -rpath or -L with LDFLAGS seems like it might enhance things
properly..

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-08-16 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #2 from yaneurab...@gmail.com ---
This is really easy to work around:

diff --git a/share/mk/bsd.dep.mk b/share/mk/bsd.dep.mk
index 14619eb..0299b37 100644
--- a/share/mk/bsd.dep.mk
+++ b/share/mk/bsd.dep.mk
@@ -215,7 +215,7 @@ cleandepend:
 .endif

 .if !target(checkdpadd) && (defined(DPADD) || defined(LDADD))
-_LDADD_FROM_DPADD= ${DPADD:C;^/usr/lib/lib(.*)\.a$;-l\1;}
+_LDADD_FROM_DPADD= ${DPADD:C;^.*/lib(.*)\.(a|so)$;-l\1;}
 _LDADD_CANONICALIZED=  ${LDADD:S/$//}
 checkdpadd:
 .if ${_LDADD_FROM_DPADD} != ${_LDADD_CANONICALIZED}

I'm not sure if this is the right solution though...

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-08-16 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

--- Comment #1 from yaneurab...@gmail.com ---
This is a bit worse with atf.*.mk and other private libraries as they do
similar things (this is with the code from my isilon-atf branch on my github
fork --
https://github.com/yaneurabeya/freebsd/commit/1804982cd418df56a7f6713802677a982f2ecb41#diff-d41d8cd98f00b204e9800998ecf8427e
):

% (cd lib/libc/tests/gen; make t_vis.checkdpadd)
(cd /usr/src/lib/libc/tests/gen && make -f /usr/src/lib/libc/tests/gen/Makefile
_RECURSING_PROGS=  SUBDIR= PROG=t_vis  checkdpadd)
/usr/src/lib/libc/tests/gen
DPADD -> /usr/obj/usr/src/lib/atf/libatf-c/libatf-c.a
LDADD -> /usr/obj/usr/src/lib/atf/libatf-c/libatf-c.so

Could -rpath be used with LDFLAGS instead with applications that need private
libraries to avoid infecting other binaries with this?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


[Bug 192730] [build] make checkdpadd failures with LD* variables being added to LDADD

2014-08-16 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=192730

yaneurab...@gmail.com changed:

   What|Removed |Added

Summary|[build] make checkdpadd |[build] make checkdpadd
   |failures (discrepancy   |failures with LD* variables
   |between .a and .so  |being added to LDADD
   |filenames) after r269648|

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"