Date: Thu, 16 Nov 2017 15:47:19 -0800 From: Bryan Drewery <bdrewery@FreeBSD.org> To: Hans Petter Selasky <hps@selasky.org>, src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: Re: svn commit: r325894 - projects/bsd_rdma_4_9 Message-ID: <b95ef558-751e-af4f-3b28-ea71202bf784@FreeBSD.org> In-Reply-To: <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> References: <201711161513.vAGFDThx060235@repo.freebsd.org> <a46a9d84-eb10-0fcf-9312-983dad229a4c@FreeBSD.org> <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --bde0g249v2wHScUXbMpLLPDd8nThlmnOC Content-Type: multipart/mixed; boundary="0bnLOvSIiUw0gsuupQkoeXgdMAoTWpKUN"; protected-headers="v1" From: Bryan Drewery <bdrewery@FreeBSD.org> To: Hans Petter Selasky <hps@selasky.org>, src-committers@freebsd.org, svn-src-projects@freebsd.org Message-ID: <b95ef558-751e-af4f-3b28-ea71202bf784@FreeBSD.org> Subject: Re: svn commit: r325894 - projects/bsd_rdma_4_9 References: <201711161513.vAGFDThx060235@repo.freebsd.org> <a46a9d84-eb10-0fcf-9312-983dad229a4c@FreeBSD.org> <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> In-Reply-To: <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> --0bnLOvSIiUw0gsuupQkoeXgdMAoTWpKUN Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable 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 Update list of hardcoded OFED build order dependencies f= or >>> =C2=A0=C2=A0 the buildworld target. >>> =C2=A0=C2=A0 =C2=A0=C2=A0 Sponsored by:=C2=A0=C2=A0=C2=A0 Mellanox Te= chnologies >>> >>> Modified: >>> =C2=A0=C2=A0 projects/bsd_rdma_4_9/Makefile.inc1 >>> >=20 > Hi, >=20 >>> >> >> This isn't right, you don't need to include any 'include' directories = in >> here as 'make includes' is ran before 'make libraries'. >=20 > I tried leaving the include directory out and it failed. 1. I don't see any reason for that. You shouldn't need contrib/ofed/include__L anywhere nor contrib/ofed/include in _ofed_lib li= st. 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/*. Now we have contributed 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 library.= 3. contrib/ofed/Makefile is also super complex now. It should really just look like the Makefile.inc1 changes (minus the include and libthr directories). Just add a .WAIT after include in SUBDIR and make it first= =2E 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 has a LIBADD on it. >=20 >> >> You don't need libthr either as it is in prebuild_libs which is built >> before all of this. >=20 > OK. I'll update it tomorrow. >=20 >> >> I suspect the actual change needed is far smaller than this. >> >=20 > Thank you for your review. >=20 > --HPS --=20 Regards, Bryan Drewery --0bnLOvSIiUw0gsuupQkoeXgdMAoTWpKUN-- --bde0g249v2wHScUXbMpLLPDd8nThlmnOC 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 iQEcBAEBAgAGBQJaDiOHAAoJEDXXcbtuRpfPiA0H/3Kv0Wt4DdEEmtNkkQoAPAW6 Evdi5nuSCwviclhoHx+/YUX9z8I4U2cseqZYMImRmhAC+8w1OYMnVbFndXGFZLBy zm/MPJRJPJ1p1+8P3kX73RGvDoUCTCJYBtzFDpwGB6vVSNBH0x4kb+cQvVOETwUm Zsimg+R8oQjnvaf+tLDxUbrrR88+fUGvaOx+Wl3JwxXNmZAgkQDvvxsb3sahzH8K PUg4NIZj8IJ+dOgqqKJR472TCK9ldhJ6gUsHcGd0lsolNKv3jSnV2wJbQE5/634M lFdgeA6dWezSyW1fFxgOdoLjkPbj+p8DeL+OL62xWNHE9ofsdScTCX+vSi/k2kw= =LN6c -----END PGP SIGNATURE----- --bde0g249v2wHScUXbMpLLPDd8nThlmnOC--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?b95ef558-751e-af4f-3b28-ea71202bf784>