Page MenuHomeFreeBSD

Include src.opts.mk after SHLIBDIR has been defined so libnv is installed to /lib , not /usr/lib
ClosedPublic

Authored by ngie on Sep 7 2014, 8:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 10:34 AM
Unknown Object (File)
Dec 27 2023, 12:43 AM
Unknown Object (File)
Aug 27 2023, 4:04 PM
Unknown Object (File)
Aug 27 2023, 11:21 AM
Unknown Object (File)
Aug 8 2023, 5:42 PM
Unknown Object (File)
Aug 3 2023, 2:17 AM
Unknown Object (File)
Jul 7 2023, 6:18 AM
Unknown Object (File)
May 11 2023, 4:27 PM
Subscribers
None

Details

Reviewers
rpaulo
jmmv
Summary

Include src.opts.mk after SHLIBDIR has been defined so libnv is installed to /lib , not /usr/lib

MFC after: 3 days

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ngie retitled this revision from to Include src.opts.mk after SHLIBDIR has been defined so libnv is installed to /lib , not /usr/lib.
ngie updated this object.
ngie added reviewers: jmmv, rpaulo.
rpaulo edited edge metadata.

This looks fine, but I wonder if it would be better to set SHLIBDIR=/lib without the '?' modifier.

This revision is now accepted and ready to land.Sep 7 2014, 9:13 PM
In D739#3, @rpaulo wrote:

This looks fine, but I wonder if it would be better to set SHLIBDIR=/lib without the '?' modifier.

SHLIBDIR?= is the predominant pattern in lib/ :/...

$ grep -r LIBDIR= lib
lib/atf/Makefile.inc:CFLAGS+=   -DATF_LIBDIR='"${LIBDIR}"'
lib/libiconv_modules/Makefile.inc:SHLIBDIR= /usr/lib/i18n
lib/libiconv_modules/Makefile.inc:SHLIBDIR= /usr/lib32/i18n
lib/libiconv_modules/Makefile.inc:LIBDIR=        ${SHLIBDIR}
$ grep -r 'LIBDIR?=' lib                                                                                                                                           
lib/libedit/Makefile:SHLIBDIR?= /lib
lib/ncurses/ncurses/Makefile:SHLIBDIR?= /lib
lib/libufs/Makefile:SHLIBDIR?= /lib
lib/libthr/Makefile:SHLIBDIR?= /lib
lib/libkvm/Makefile:SHLIBDIR?= /lib
lib/libmd/Makefile:SHLIBDIR?= /lib
lib/libdevstat/Makefile:SHLIBDIR?= /lib
lib/libutil/Makefile:SHLIBDIR?= /lib
lib/libjail/Makefile:SHLIBDIR?= /lib
lib/libkiconv/Makefile:SHLIBDIR?=       /lib
lib/libcrypt/Makefile:SHLIBDIR?=        /lib
lib/libcasper/Makefile:SHLIBDIR?=       /lib
lib/libexpat/Makefile:SHLIBDIR?=        /lib
lib/libbegemot/Makefile:SHLIBDIR?= /lib
lib/libnv/Makefile:SHLIBDIR?= /lib
lib/libgeom/Makefile:SHLIBDIR?= /lib
lib/msun/Makefile:SHLIBDIR?= /lib
lib/libsbuf/Makefile:SHLIBDIR?=         /lib
lib/libcam/Makefile:SHLIBDIR?=  /lib
lib/libcapsicum/Makefile:SHLIBDIR?=     /lib
lib/libulog/Makefile:SHLIBDIR?=/lib
lib/libc/Makefile:SHLIBDIR?= /lib
lib/libpcap/Makefile:SHLIBDIR?= /lib
lib/libalias/libalias/Makefile:SHLIBDIR?= /lib
lib/libalias/modules/Makefile.inc:SHLIBDIR?= /lib
lib/libipsec/Makefile:SHLIBDIR?= /lib
lib/libcxxrt/Makefile:SHLIBDIR?=        /lib
lib/libz/Makefile:SHLIBDIR?=    /lib
lib/libpjdlog/Makefile:SHLIBDIR?=       /lib

I agree that explicitly setting LIBDIR makes more sense, but I need to do some digging first to determine whether or not that breaks anything.

Committed as r271241 -- thanks!