Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 17 Apr 2003 00:23:35 +1000 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        Bruce Evans <bde@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/usr.sbin/named Makefile
Message-ID:  <20030417000737.J6033@gamplex.bde.org>
In-Reply-To: <200304161350.h3GDorNU058617@repoman.freebsd.org>
References:  <200304161350.h3GDorNU058617@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 16 Apr 2003, Bruce Evans wrote:

> bde         2003/04/16 06:50:53 PDT
>
>   FreeBSD src repository
>
>   Modified files:
>     usr.sbin/named       Makefile
>   Log:
>   "Fixed" LDADD by backing out part of the previous commit.  -lfoo in
>   LDADD is not wrong, but the simple substitution in `make checkdpadd'
>   doesn't work if foo.a is not an installed library, so we use the full
>   path to foo.a in both DPADD and LDADD for non-installed libraries.
>
>   Revision  Changes    Path
>   1.35      +1 -1      src/usr.sbin/named/Makefile

Actually -lfoo was wrong here, since there is no -L to tell the linker
where the non-installed library is.  It worked accidentally because
libisc is actually installed and changing to using the dynamic libisc
didn't cause problems.  Using the installed library is probably right,
but it is not what the bind utilities expect -- all their (FreeBSD)
makefiles that use libisc (only named and named-xfer) have 7 lines of
code to use the non-installed one.  The broken part of the previous
commit reduced these lines to just breaking DPADD and this commit just
makes them a verbose way of doing things again.

The a.out linker had support for determining the libraries that will
be used less hackishly, but using the full path to the (static) library
in both DPADD and LDADD for non-installed libraries was still necessary
to handle the case where the libraries don't exist at `make depend' time.

Bruce



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