This patch adds -lldap after the libactive_directory.a library, so that it's dependencies are filled when using --as-needed. I'm not sure if this is the best way to do this, so might be worth asking flameeyes to check it over to see if it looks ok. Otherwise should be fine, don't forget the ebuild need eautomake to be run after applying the patch. Patch to follow.
Created attachment 83827 [details, diff] adtool-1.3-as-needed.patch
flameeyes, what do you think about this patch?
Created attachment 83874 [details, diff] adtool-1.3-asneeded.patch Hmm I think adtool is one of the strangests autotool user I ever seen... The problem in this case is linking ordering, as the libraries are passed in LDFLAGS instead of LDADD... also, it's usually quicker to pass them per-target instead of global, so I've done the attached patch. It will probably require a full eautoreconf tho, as the automake seems old to me.
Christian, don't tell me you disabled mailing for attachments.. :P
(In reply to comment #4) > Christian, don't tell me you disabled mailing for attachments.. :P Oops! Fixed, thanks. Anyway, I could swear it was my cat... :-D
in portage: thanks to both of you
