Date: Wed, 1 May 2013 17:07:50 -0500 From: Brooks Davis <brooks@freebsd.org> To: Garrett Cooper <yaneurabeya@gmail.com> Cc: sjg@freebsd.org, toolchain@freebsd.org, joel@freebsd.org Subject: Re: Fwd: docs/178286: [PATCH] document the LOCAL_* vars in build(7) Message-ID: <20130501220750.GD45806@lor.one-eyed-alien.net> In-Reply-To: <CAGHfRMBtumFoaEDXavpo16z18bwHJC83O75CbcZpEvYcK7GCnA@mail.gmail.com> References: <201305012123.r41LNcEL048006@red.freebsd.org> <201305012130.r41LU031023258@freefall.freebsd.org> <CAGHfRMBtumFoaEDXavpo16z18bwHJC83O75CbcZpEvYcK7GCnA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 01, 2013 at 02:35:17PM -0700, Garrett Cooper wrote: > Hi Brooks and Joel, > I'd really appreciate if you guys could help out with this. I'm > having to fix the Isilon build system due to severe lack of > documentation in build(7). So many mistakes have been made because > people don't understand how to write FreeBSD Makefiles... One minor concern I have with this patch is documenting LOCAL_LIB_DIRS. I've not posted to the lists yet, but now that I understand buildworld better, I think I implemented it wrong. It should have required that the directories be also be listed in LOCAL_DIRS. The reason is that you could then have LOCAL_DIRS=foo and LOCAL_LIB_DIRS=foo/lib so you only added one directory to the FreeBSD tree but could still have libs. I feared that someone might have started using this feature which would mean we can't easily change that. Is Isilon using it? -- Brooks
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130501220750.GD45806>