From owner-svn-src-all@freebsd.org Tue Oct 6 18:02:24 2015 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id D4E8C9B6EB8; Tue, 6 Oct 2015 18:02:24 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id B7648C25; Tue, 6 Oct 2015 18:02:24 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [IPv6:::1]) by freefall.freebsd.org (Postfix) with ESMTP id B12DF1413; Tue, 6 Oct 2015 18:02:24 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 616051053E; Tue, 6 Oct 2015 18:02:24 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id CN1-wGSx_QMX; Tue, 6 Oct 2015 18:02:21 +0000 (UTC) Subject: Re: svn commit: r288911 - head/share/mk DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 932C910538 To: Warner Losh , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201510060418.t964Innu071170@repo.freebsd.org> From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc X-Enigmail-Draft-Status: N1110 Organization: FreeBSD Message-ID: <56140CAD.8080200@FreeBSD.org> Date: Tue, 6 Oct 2015 11:02:21 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <201510060418.t964Innu071170@repo.freebsd.org> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="XVfixN51tK1juSNcbC00taoMtN8KnmrhB" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Oct 2015 18:02:25 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --XVfixN51tK1juSNcbC00taoMtN8KnmrhB Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 10/5/2015 9:18 PM, Warner Losh wrote: > Author: imp > Date: Tue Oct 6 04:18:48 2015 > New Revision: 288911 > URL: https://svnweb.freebsd.org/changeset/base/288911 >=20 > Log: > Previous versions of bsd.own.mk included bsd.compiler.mk > only when _WITHOUT_SRCCONF wasn't defined. Restore this > behavior because bsd.ports.mk depends on this in subtle > ways. The compat include of bsd.compiler.mk should > be removed in 12 anyway. > =20 > PR: 203540 >=20 Perhaps the wrong place to discuss this, but I will anyhow as I don't think it will change. The sys.mk change to include src.conf breaks building ports in a sub-dir of src. Meaning, /usr/src/ports/. The MAKESYSPATH with '.../share/mk' finds /usr/src/share/mk and runs off with all of the src.*.mk stuff long before the port Makefile includes bsd.port.mk, from /usr/src/share/mk/bsd.port.mk, which has a _WITHOUT_SRCCONF=3D guard set on it to avoid bsd.own.mk from including src.conf. But because sys.mk is already included long before this, src.conf is already included and anything handled in sys.mk has no real way to respect _WITHOUT_SRCCONF unless it is in the environment. [Note that the actual inclusion of src.conf no longer has a _WITHOUT_SRCCONF=3D check, but that is trivial to fix] Why would anyone build ports in a sub-dir of src? It's convenient for a vendor building their own product that needs their own ports tree. Some decisions can't easily be changed; if the root of the source code checkout is already src/, there is no simple way to avoid the problem. 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 or similar at the top of all src Makefiles.= I would really like to find a solution to this as it is a looming problem for my work's build approaching in a few months. I figured out a hack we can use locally, to set _WITHOUT_SRCCONF=3D, when the current directory has "ports/" in it. That works for us, but not for other vendors who don't realize this is coming. Perhaps the scope of people doing this is not large. --=20 Regards, Bryan Drewery --XVfixN51tK1juSNcbC00taoMtN8KnmrhB Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJWFAytAAoJEDXXcbtuRpfPWbEH/ie52EN2C59fjw5L8vtCOfz2 4mPWbGEdz6UkgpOXWVd6K3O+cQa7dvYXMGCj/XQ6kD8/P/udR/8oxnzc5/RU1nBP uYRFMTlZs0HtzjzMyBmWoi2iKyMa09nB8Av8D94T6UiBnVw3O6K8PuvFHRE9ioEl CmLGdRGr6LSWYJtiSj79itiUVwmyvi4DRIkVohpfIyF0G2BD/tje70l1AX4W0TRF F/qpKi2BwOcI5CZVDJoP929cpmDBkezciawmu7sLyiyWkcpJU3eibikJqhsE7LZO 8QQKW4AMBZxGA7ET56u4V4DzR9XubdEBz7iOtc4NAR+A5qmIk0tnm3td3tp78co= =QPmT -----END PGP SIGNATURE----- --XVfixN51tK1juSNcbC00taoMtN8KnmrhB--