Date: Tue, 21 Sep 2004 23:58:36 +0300 From: Ruslan Ermilov <ru@freebsd.org> To: Dag-Erling Smorgrav <des@freebsd.org> Cc: cvs-all@freebsd.org Subject: Re: cvs commit: src Makefile.inc1 src/contrib/bind9 FREEBSD-Upgrade config.h src/etc Makefile src/etc/mtree BSD.include.dist BSD.root.dist src/lib Makefile src/lib/bind Makefile config.h config.mk src/lib/bind/bind Makefile config.h port_after.h ... Message-ID: <20040921205835.GA32943@ip.net.ua> In-Reply-To: <200409211901.i8LJ1m1a076732@repoman.freebsd.org> References: <200409211901.i8LJ1m1a076732@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--rwEMma7ioTxnRzrJ
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
Hi DES,
There's a slight problem with your commit to Makefile.inc1:
%%%
Index: Makefile.inc1
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
RCS file: /home/ncvs/src/Makefile.inc1,v
retrieving revision 1.446
diff -u -r1.446 Makefile.inc1
--- Makefile.inc1 21 Sep 2004 19:01:41 -0000 1.446
+++ Makefile.inc1 21 Sep 2004 20:45:18 -0000
@@ -902,9 +902,6 @@
=20
.if !defined(NO_BIND)
_prebuild_libs+=3D lib/libpthread
-_generic_libs+=3D lib/bind
-
-lib/bind__L: lib/libpthread__L
.endif
=20
_generic_libs+=3D lib
%%%
Since lib/bind is not in the list of "prebuild" libs (no other
libraries depend on it), you don't need to add lib/bind to
_generic_libs (it will be built by "lib"), and similarly the
"__L" interdependencies are redundant. In general, the list
of prebuild libraries and their inter-dependencies is generated
by src/tools/make-libdeps.sh. By applying the diff above you
will again make it consistent with the script's output.
Probably more important that your commit broke the sparc64 and
alpha ports builds because on sparc64 and alpha libpthread is
still installed as "libkse", here's a relevant bit from
src/lib/libpthread/Makefile:
: .if ${MACHINE_ARCH} =3D=3D "alpha" || ${MACHINE_ARCH} =3D=3D "sparc64"
: LIB=3Dkse
: .else
: LIB=3Dpthread
: .endif
On these architectures, libc_r is symlinked to libpthread, so
you need to add lib/libc_r instead of lib/libpthread on alpha
and amd64 to _prebuild_libs.
Now NOLIBPTHREAD and NOLIBC_R builds (arch dependent) will also
require one to specify NO_BIND. This dependency should either
be enforced or at least documented.
Cheers,
--=20
Ruslan Ermilov
ru@FreeBSD.org
FreeBSD committer
--rwEMma7ioTxnRzrJ
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (FreeBSD)
iD8DBQFBUJX7qRfpzJluFF4RAlyQAJ40MO+6uc36RYHOEq0GxQ6uAJeXSACcCDao
NNc0EnziV9/onzp2IPkvN28=
=Aiw/
-----END PGP SIGNATURE-----
--rwEMma7ioTxnRzrJ--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040921205835.GA32943>