Author: kevans
Date: Fri Jul  6 11:23:14 2018
New Revision: 336026
URL: https://svnweb.freebsd.org/changeset/base/336026

Log:
  config(8): Fix broken ABI
  
  r336019 introduced ${SRCTOP}/sys to the include paths in order to pull in a
  new sys/{c,}nv.h. This is wrong, because the build tree's ABI isn't
  guaranteed to match what's running on the host system.
  
  Fix instead by removing -I${SRCTOP}/sys and installing the libnv headers
  with `make -C lib/libnv includes`... this may or may not get re-worked in
  the future so that a userland lib isn't installing includes from sys/.
  
  Reported by:  bdrewery

Modified:
  head/lib/libnv/Makefile
  head/usr.sbin/config/Makefile

Modified: head/lib/libnv/Makefile
==============================================================================
--- head/lib/libnv/Makefile     Fri Jul  6 10:13:42 2018        (r336025)
+++ head/lib/libnv/Makefile     Fri Jul  6 11:23:14 2018        (r336026)
@@ -17,6 +17,9 @@ SRCS+=        msgio.c
 SRCS+= nvlist.c
 SRCS+= nvpair.c
 
+INCSDIR=       ${INCLUDEDIR}/sys
+INCS=  ${SRCTOP}/sys/sys/cnv.h ${SRCTOP}/sys/sys/nv.h
+
 HAS_TESTS=
 SUBDIR.${MK_TESTS}+= tests
 

Modified: head/usr.sbin/config/Makefile
==============================================================================
--- head/usr.sbin/config/Makefile       Fri Jul  6 10:13:42 2018        
(r336025)
+++ head/usr.sbin/config/Makefile       Fri Jul  6 11:23:14 2018        
(r336026)
@@ -14,7 +14,7 @@ kernconf.c: kernconf.tmpl
        ${FILE2C} 'char kernconfstr[] = {' ',0};' < \
            ${SRCDIR}/kernconf.tmpl > kernconf.c
 
-CFLAGS+= -I. -I${SRCDIR} -I${SRCTOP}/sys
+CFLAGS+= -I. -I${SRCDIR}
 
 NO_WMISSING_VARIABLE_DECLARATIONS=
 
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to