Date: Mon, 9 Oct 2017 19:33:39 -0600 From: Warner Losh <imp@bsdimp.com> To: Steven Hartland <steven.hartland@multiplay.co.uk> Cc: Warner Losh <imp@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r324449 - in head/sys/boot: arm/uboot efi/boot1 sparc64/loader Message-ID: <CANCZdfo4OiEmFjxzqxAuMZTY_CUYQgFBraWVL5MW=dEt9Ta3ng@mail.gmail.com> In-Reply-To: <d06e414b-8b14-a2e8-3583-71dece358697@multiplay.co.uk> References: <201710092211.v99MBwCq010906@repo.freebsd.org> <c4834afc-df53-35ea-ac55-55524ef73521@multiplay.co.uk> <CANCZdfpQnJU51%2Brs=U0vC0B8K3d3jf8MH8OjbHPRYaB%2B7wRZLw@mail.gmail.com> <CANCZdfq4zE4pZQBwjNjQvZAEL7-gYxgraWH5SnM20M5kgU8HNg@mail.gmail.com> <d06e414b-8b14-a2e8-3583-71dece358697@multiplay.co.uk>
next in thread | previous in thread | raw e-mail | index | archive | help
Yea, that's my bad. I didn't do a completely clean build with stand.h gone, so I didn't catch it before the commit. I've just pushed in a fixup so that we always include the path to stand.h in the include path, since almost everybody needs it, and it is harmless to the others to include. Sorry for the bumps... Warner On Mon, Oct 9, 2017 at 6:38 PM, Steven Hartland < steven.hartland@multiplay.co.uk> wrote: > Yer even no -j fails :( > > > On 10/10/2017 01:01, Warner Losh wrote: > > Oh, killed /usr/include/stand.h and found it. I'll post a fix when I get > back. > > On Mon, Oct 9, 2017 at 6:00 PM, Warner Losh <imp@bsdimp.com> wrote: > >> Can you find out? A clean build works for me. Chances are good that >> sys/boot/efi/boot1/Makefile needs a line like >> CFLAGS+=-I${SASRC} or similar. I have to go out for 2 hours, but will >> look into when I get back if you can't make progress. I don't see one there >> and I had to add it a couple of other places. >> >> Warner >> >> On Mon, Oct 9, 2017 at 5:56 PM, Steven Hartland < >> steven.hartland@multiplay.co.uk> wrote: >> >>> Not sure which of these sets of changes caused the issue but a clean >>> build from scratch is currently failing here with: >>> >>> In file included from /usr/home/smh/freebsd/base/hea >>> d/sys/boot/efi/boot1/ufs_module.c:41: >>> In file included from /usr/home/smh/freebsd/base/hea >>> d/sys/boot/efi/boot1/boot_module.h:35: >>> /usr/home/smh/freebsd/base/head/sys/boot/efi/boot1/../include/efilib.h:33:10: >>> fatal error: 'stand.h' file not found >>> #include <stand.h> >>> ^~~~~~~~~ >>> >>> Build was with -j24 in case it matters, going to try without -j but that >>> will take many hours >>> >>> >>> On 09/10/2017 23:11, Warner Losh wrote: >>> >>> Author: imp >>> Date: Mon Oct 9 22:11:57 2017 >>> New Revision: 324449 >>> URL: https://svnweb.freebsd.org/changeset/base/324449 >>> >>> Log: >>> Prefer ${LIBSTAND} to -lstand >>> >>> Sponsored by: Netflix >>> >>> Modified: >>> head/sys/boot/arm/uboot/Makefile >>> head/sys/boot/efi/boot1/Makefile >>> head/sys/boot/sparc64/loader/Makefile >>> >>> Modified: head/sys/boot/arm/uboot/Makefile >>> ============================================================================== >>> --- head/sys/boot/arm/uboot/Makefile Mon Oct 9 21:06:16 2017 (r324448) >>> +++ head/sys/boot/arm/uboot/Makefile Mon Oct 9 22:11:57 2017 (r324449) >>> @@ -121,7 +121,7 @@ CFLAGS+= -fPIC >>> NO_WERROR.clang= >>> >>> DPADD= ${LIBFICL} ${LIBUBOOT} ${LIBFDT} ${LIBUBOOT_FDT} ${LIBSTAND} >>> -LDADD= ${LIBFICL} ${LIBUBOOT} ${LIBFDT} ${LIBUBOOT_FDT} -lstand >>> +LDADD= ${LIBFICL} ${LIBUBOOT} ${LIBFDT} ${LIBUBOOT_FDT} ${LIBSTAND} >>> >>> OBJS+= ${SRCS:N*.h:R:S/$/.o/g} >>> >>> >>> Modified: head/sys/boot/efi/boot1/Makefile >>> ============================================================================== >>> --- head/sys/boot/efi/boot1/Makefile Mon Oct 9 21:06:16 2017 (r324448) >>> +++ head/sys/boot/efi/boot1/Makefile Mon Oct 9 22:11:57 2017 (r324449) >>> @@ -91,7 +91,7 @@ LIBEFI= ${.OBJDIR}/../libefi/libefi.a >>> # as well as required string and memory functions for all platforms. >>> # >>> DPADD+= ${LIBEFI} ${LIBSTAND} >>> -LDADD+= ${LIBEFI} -lstand >>> +LDADD+= ${LIBEFI} ${LIBSTAND} >>> >>> DPADD+= ${LDSCRIPT} >>> >>> >>> Modified: head/sys/boot/sparc64/loader/Makefile >>> ============================================================================== >>> --- head/sys/boot/sparc64/loader/Makefile Mon Oct 9 21:06:16 2017 (r324448) >>> +++ head/sys/boot/sparc64/loader/Makefile Mon Oct 9 22:11:57 2017 (r324449) >>> @@ -86,7 +86,7 @@ CFLAGS+= -I${.CURDIR}/../../../../lib/libstand/ >>> CFLAGS+= -I${SRCTOP}/sys >>> >>> DPADD= ${LIBFICL} ${LIBZFSBOOT} ${LIBOFW} ${LIBSTAND} >>> -LDADD= ${LIBFICL} ${LIBZFSBOOT} ${LIBOFW} -lstand >>> +LDADD= ${LIBFICL} ${LIBZFSBOOT} ${LIBOFW} ${LIBSTAND} >>> >>> loader.help: help.common help.sparc64 >>> cat ${.ALLSRC} | \ >>> >>> >>> >>> >> > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfo4OiEmFjxzqxAuMZTY_CUYQgFBraWVL5MW=dEt9Ta3ng>