Date: Thu, 22 Oct 2015 17:18:39 -0700 From: Bryan Drewery <bdrewery@FreeBSD.org> To: Ian Lepore <ian@freebsd.org> Cc: "src-committers@freebsd.org" <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: r289778 - head/share/mk Message-ID: <232D9171-D360-4110-8682-6B371D9F9CAC@FreeBSD.org> In-Reply-To: <1445558386.14963.38.camel@freebsd.org> References: <201510222341.t9MNfuNK007465@repo.freebsd.org> <1445558386.14963.38.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Oct 22, 2015, at 16:59, Ian Lepore <ian@freebsd.org> wrote: >=20 >> On Thu, 2015-10-22 at 23:41 +0000, Bryan Drewery wrote: >> Author: bdrewery >> Date: Thu Oct 22 23:41:56 2015 >> New Revision: 289778 >> URL: https://svnweb.freebsd.org/changeset/base/289778 >>=20 >> Log: >> For SUBDIR_PARALLEL, when doing 'make clean*' or 'make obj' there >> is no need to >> respect SUBDIR_DEPEND_* or .WAIT. >>=20 >> MFC after: 2 weeks >> Sponsored by: EMC / Isilon Storage Division >=20 > This doesn't feel so safe. People override these targets and do who > -knows-what with them. The clean* seems especially risky since it will > match targets in end-user makefiles that you don't even know about. >=20 > -- Ian >=20 >=20 I can limit it to src tree builds if you want. I cannot imagine any situation where "clean<something>" depends on another d= irectory going first.=20 Similarly it makes sense to always build subdirs in parallel with 'obj' and I= would argue 'clean*' too, regardless of SUBDIR_PARALLEL. I only just realiz= ed that. Yes there are mkdir -p and rm -f races, but we have not added any d= ependencies because of these as is.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?232D9171-D360-4110-8682-6B371D9F9CAC>