Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 24 Nov 2017 14:23:33 +0100
From:      Hans Petter Selasky <hps@selasky.org>
To:        Bryan Drewery <bdrewery@FreeBSD.org>, src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   Re: svn commit: r325894 - projects/bsd_rdma_4_9
Message-ID:  <3436ea01-e8b9-b04d-ecc6-352da5dff215@selasky.org>
In-Reply-To: <b95ef558-751e-af4f-3b28-ea71202bf784@FreeBSD.org>
References:  <201711161513.vAGFDThx060235@repo.freebsd.org> <a46a9d84-eb10-0fcf-9312-983dad229a4c@FreeBSD.org> <97fb917d-8495-6ccf-9772-6b05da10e5f1@selasky.org> <b95ef558-751e-af4f-3b28-ea71202bf784@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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:
>>>>     Update list of hardcoded OFED build order dependencies for
>>>>     the buildworld target.
>>>>        Sponsored by:    Mellanox Technologies
>>>>
>>>> Modified:
>>>>     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.  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/*.  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.
> You have this chained dependency thing but dependencies are already
> inherited just fine.
> SUBDIR_DEPEND_foo= bar
> SUBDIR_DEPEND_baz= 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.
> 

See:
https://svnweb.freebsd.org/changeset/base/326146

--HPS



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3436ea01-e8b9-b04d-ecc6-352da5dff215>