From owner-svn-src-projects@freebsd.org Thu Dec 7 00:15:00 2017 Return-Path: Delivered-To: svn-src-projects@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 8245AE913FB for ; Thu, 7 Dec 2017 00:15:00 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 404ED69ACB; Thu, 7 Dec 2017 00:15:00 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (unknown [127.0.1.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id 2EB5F15BFA; Thu, 7 Dec 2017 00:14:59 +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 685732B65; Thu, 7 Dec 2017 00:14:58 +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 LmpytGmMhyUZ; Thu, 7 Dec 2017 00:14:55 +0000 (UTC) Subject: Re: svn commit: r325894 - projects/bsd_rdma_4_9 DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 0C7252B5F To: Hans Petter Selasky , src-committers@freebsd.org, svn-src-projects@freebsd.org References: <201711161513.vAGFDThx060235@repo.freebsd.org> <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> <3436ea01-e8b9-b04d-ecc6-352da5dff215@selasky.org> From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: Date: Wed, 6 Dec 2017 16:14:59 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <3436ea01-e8b9-b04d-ecc6-352da5dff215@selasky.org> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="vA2JWGWfbQxH5pTmaFIinEX0xS5V9B2sn" X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Dec 2017 00:15:00 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --vA2JWGWfbQxH5pTmaFIinEX0xS5V9B2sn Content-Type: multipart/mixed; boundary="4lqkcn6XH7co2I8VAGV6xWAkIQEUGgDmJ"; protected-headers="v1" From: Bryan Drewery To: Hans Petter Selasky , src-committers@freebsd.org, svn-src-projects@freebsd.org Message-ID: Subject: Re: svn commit: r325894 - projects/bsd_rdma_4_9 References: <201711161513.vAGFDThx060235@repo.freebsd.org> <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> <3436ea01-e8b9-b04d-ecc6-352da5dff215@selasky.org> In-Reply-To: <3436ea01-e8b9-b04d-ecc6-352da5dff215@selasky.org> --4lqkcn6XH7co2I8VAGV6xWAkIQEUGgDmJ Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 11/24/2017 5:23 AM, Hans Petter Selasky wrote: > On 11/17/17 00:47, Bryan Drewery wrote: >> On 11/16/2017 1:12 PM, Hans Petter Selasky wrote: >>> On 11/16/17 17:56, Bryan Drewery wrote: >>>> On 11/16/2017 7:13 AM, Hans Petter Selasky wrote: >>>>> Author: hselasky >>>>> Date: Thu Nov 16 15:13:28 2017 >>>>> New Revision: 325894 >>>>> URL: https://svnweb.freebsd.org/changeset/base/325894 >>>>> >>>>> Log: >>>>> =C2=A0=C2=A0=C2=A0 Update list of hardcoded OFED build order depend= encies for >>>>> =C2=A0=C2=A0=C2=A0 the buildworld target. >>>>> =C2=A0=C2=A0=C2=A0 =C2=A0=C2=A0 Sponsored by:=C2=A0=C2=A0=C2=A0 Mel= lanox Technologies >>>>> >>>>> Modified: >>>>> =C2=A0=C2=A0=C2=A0 projects/bsd_rdma_4_9/Makefile.inc1 >>>>> >>> >>> Hi, >>> >>>>> >>>> >>>> This isn't right, you don't need to include any 'include' >>>> directories in >>>> here as 'make includes' is ran before 'make libraries'. >>> >>> I tried leaving the include directory out and it failed. >> >> 1. >> I don't see any reason for that.=C2=A0 You shouldn't need >> contrib/ofed/include__L anywhere nor contrib/ofed/include in _ofed_lib= >> list. >> I see contrib/ofed/include hooked up fine for 'make _includes' so long= >> as WITH_OFED is defined. >> >> What was the error? >> >> >> 2. >> Unrelated but I don't like the new directory structure of >> contrib/ofed/usr.bin/* -> contrib/ofed/*.=C2=A0 Now we have contribute= d >> sources in with our FreeBSD Makefiles. >> >> It also makes it more complex in the _ofed_lib list since before we >> could just have contrib/ofed/usr.lib but now must list out every libra= ry. >> >> 3. >> contrib/ofed/Makefile is also super complex now.=C2=A0 It should reall= y just >> look like the Makefile.inc1 changes (minus the include and libthr >> directories).=C2=A0 Just add a .WAIT after include in SUBDIR and make = it >> first. >> You have this chained dependency thing but dependencies are already >> inherited just fine. >> SUBDIR_DEPEND_foo=3D bar >> SUBDIR_DEPEND_baz=3D foo >> >> baz will implicitly depend on bar being built, there's no need to >> explicitly add bar as a dependency but it's not wrong to do so if it h= as >> a LIBADD on it. >> >=20 > See: > https://svnweb.freebsd.org/changeset/base/326146 It's totally wrong and ignores my advice. I'd really just like the whole merge reverted because now I have to go rewrite your rewrite back to what was proper. --=20 Regards, Bryan Drewery --4lqkcn6XH7co2I8VAGV6xWAkIQEUGgDmJ-- --vA2JWGWfbQxH5pTmaFIinEX0xS5V9B2sn 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 iQEcBAEBAgAGBQJaKIgDAAoJEDXXcbtuRpfPUxsIAIdlQyaoE6eYZLUc1jSeUMd6 qS4XJa+PswNArYN8yTPGQpxUxZT2UTKLV2qdMV/OFuJytcUX8yEbaPbT60EPSqeQ htKY5/Gxc9kPM3zbfwNIqqiN/0Z6/TDx1aAJNFxukzBYNmB1wAhSf6eBFQM8+jbP qMPcGwg97GAyRhP9Co+t9EjXOSkYXzqTl/1olT6HWHHQNpifjwTW0vPMTojFre9Y G4MPulLm5JDXS85QeRVIdQvgVez42b9fZdmL93BRKi6JKb2G4o7Lg3JOvpD6Nc70 kD2G7UCzpKrICWUgUBpybN+rJW2Dlw6gKf3yOFU8WpCxgHNayCcpzjJITMz5vMc= =YudM -----END PGP SIGNATURE----- --vA2JWGWfbQxH5pTmaFIinEX0xS5V9B2sn--