From owner-svn-src-all@freebsd.org Thu Dec 7 00:50:26 2017 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 1F88FE928A0; Thu, 7 Dec 2017 00:50:26 +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 DC0116B5CE; Thu, 7 Dec 2017 00:50:25 +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 C568616986; Thu, 7 Dec 2017 00:50: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 149F82C2D; Thu, 7 Dec 2017 00:50: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 jObnSUbt1RkB; Thu, 7 Dec 2017 00:50:16 +0000 (UTC) Subject: Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys/... DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 8B1D62C27 From: Bryan Drewery To: Hans Petter Selasky , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201711241450.vAOEoSFW056744@repo.freebsd.org> <0fec20d5-cb8f-4937-5dd0-54b8b7461772@selasky.org> <8e2fd368-ad3e-27bb-0834-4b6f2f99e905@FreeBSD.org> Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: Date: Wed, 6 Dec 2017 16:50:19 -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: <8e2fd368-ad3e-27bb-0834-4b6f2f99e905@FreeBSD.org> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="N7qFonVXa2aX9q0lSbr4edMau1wcRpcxQ" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.25 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: Thu, 07 Dec 2017 00:50:26 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --N7qFonVXa2aX9q0lSbr4edMau1wcRpcxQ Content-Type: multipart/mixed; boundary="ItijETbQp6HRpNjgeh6DvPB0f26c8E2md"; protected-headers="v1" From: Bryan Drewery To: Hans Petter Selasky , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: Subject: Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys/... References: <201711241450.vAOEoSFW056744@repo.freebsd.org> <0fec20d5-cb8f-4937-5dd0-54b8b7461772@selasky.org> <8e2fd368-ad3e-27bb-0834-4b6f2f99e905@FreeBSD.org> In-Reply-To: <8e2fd368-ad3e-27bb-0834-4b6f2f99e905@FreeBSD.org> --ItijETbQp6HRpNjgeh6DvPB0f26c8E2md Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 12/6/2017 4:48 PM, Bryan Drewery wrote: > On 12/6/2017 4:38 PM, Bryan Drewery wrote: >> On 12/6/2017 4:28 PM, Hans Petter Selasky wrote: >>> On 12/07/17 01:10, Bryan Drewery wrote: >>>> I'm a bit speechless.=C2=A0 Why??=C2=A0 The old version was 90% fine= and I sent >>>> you advice on how to make it work with the new layout. >>> >>> Hi, >>> >>> I tried to listen to your advice, splitting up the Makefiles, not >>> listing every dependency. Further how the OFED libraries are linked h= as >>> changed. Two iterations for building the OFED libraries is not enough= , >>> like before. We really need four iterations. >>> >>> I don't see how the current approach is "wrong". Can you explain a bi= t >>> more? >> >> 1. SUBDIR=3D../../somewhere else is inherently wrong. >> 2. Building a library from a directory named '2' is wrong. >> 3. Not just using proper SUBDIR_DEPEND and making all these subdirs is= >> not needed and complicates things MORE by splitting them all up. >> 4. Building a library from a directory where its real bsd.lib.mk is no= t >> at is wrong. >> 5. Depending on an include directory in the Makefile.inc1 library >> dependencies is simply not needed - I told you that. >> 6. Using these /0 /1 /2 /3 directories actually hurts the 'make >> libraries' parallelism/dependency graph. It ends up building /1 rather= >> than building libibcm and libibumad directly in its graph. It creates= >> an indirection which adds more unneeded submake recursion. >> >> The whole thing needlessly obfuscates these libraries. >> >> It's fine, I'm going to rewrite it all now. >> >=20 > 7. We don't put FreeBSD Makefiles in the same directory as contributed > files. It's against the pattern. So moving contrib/ofed/usr.lib/* to > contrib/ofed/*/ is wrong. It complicates future merges of contrib code.= > Really none of these Makefiles should be in contrib/ofed - the proper > thing would be to move them out, not to inline them with the contribute= d > code. >=20 8. And you blew away my Makefile.depend files, which I do normally say I'll deal with, but a far simpler directory structure movement would have easily kept them and I could have updated them. But now there's not even something to start from since they were deleted for no reason. --=20 Regards, Bryan Drewery --ItijETbQp6HRpNjgeh6DvPB0f26c8E2md-- --N7qFonVXa2aX9q0lSbr4edMau1wcRpcxQ 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 iQEcBAEBAgAGBQJaKJBLAAoJEDXXcbtuRpfPhrMH+gPxofqjUZb+I+W9LTuB2ewZ 6Sf0G2/0n7yIk2LXHMnCRIL1S7I9RR+perSrO2xhCvreC8xpRhjkdxlErYfYS0tp rz65+ZSk2p7nOnVn3sBllLkTQAhyVA/AocaeBBAPRYzHeBTYytBWsrSwegPabbVn LjeR6Nht7jGdZ1F1CfxAtScxX4bF17wXgxt2i4tqzS78zi9VR9oZXYcloCdLOlP7 O5IjFpkVeOn9yKWWd88VaxsUEqVmLigoKVMLDPcVrKrnd2BtfUOHsTLi+ZoUA+16 Ac2NK8Hw39B0AIhIY+h236ro7mO/GiyuhDYIz7f2TcdZ+S+2BN9CyDZTZmbA1MU= =i5Op -----END PGP SIGNATURE----- --N7qFonVXa2aX9q0lSbr4edMau1wcRpcxQ--