Date: Thu, 8 Oct 2015 10:00:30 -0600 From: Warner Losh <imp@bsdimp.com> To: Bryan Drewery <bdrewery@freebsd.org> 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: r288911 - head/share/mk Message-ID: <CANCZdfrTppL=umBHYxj-gGrjb8omR3BcocWJtEXkSPAktVOGzg@mail.gmail.com> In-Reply-To: <5615C935.2060204@FreeBSD.org> References: <201510060418.t964Innu071170@repo.freebsd.org> <56140CAD.8080200@FreeBSD.org> <8D5D1CE5-BC57-464D-9C47-63EFF4C1CF2F@bsdimp.com> <5615C935.2060204@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Oct 7, 2015 at 7:39 PM, Bryan Drewery <bdrewery@freebsd.org> wrote: > On 10/6/2015 1:24 PM, Warner Losh wrote: > >> With the META_MODE changes, sjg introduced this /etc/src-env.conf file > >> that is included from sys.mk early, that can be used for overriding > >> things like MAKEOBJDIRPREFIX, enabling META_MODE (it needs to be set > >> extremely early for AUTO_OBJ support, among other things). > >> > >> As far as I can tell, the sys.mk change to include src.conf early was > >> done out of convenience. Meaning, we could remove that and just add > >> back a .include <src.init.mk> or similar at the top of all src > Makefiles. > > All src makefiles? Yea, I=E2=80=99d rather hoped to avoid that, though = it is > easily > > scripted. I=E2=80=99d thought of this solution at the time I did the MA= KESYSPATH > > hack, and rejected it as being too unwieldy. And having that at the top > > of all the files would still require MAKESYSPATH need to be =E2=80=A6/s= hare/mk > > to work out. I was rather hoping we could find some good way around > > doing that. > > r289000 confuses me. Clearly src.opts.mk is still needed in Makefiles. > So why are we including src.conf in sys.mk and not src.opts.mk? > We're including it there because Simon wanted it included there so meta-mode settings were visible in sys.mk. I thought it lame at the time, but had no better solution. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrTppL=umBHYxj-gGrjb8omR3BcocWJtEXkSPAktVOGzg>