[Buildroot] [PATCH] Don't disable dependency tracking if using <pkg>_OVERRIDE_SRCDIR.

Romain Naour romain.naour at gmail.com
Tue Feb 7 22:14:30 UTC 2017


Hi Ignacy,

Le 06/02/2017 à 20:37, Ignacy Gawędzki a écrit :
> On Mon, Feb 06, 2017 at 07:39:00PM +0100, thus spake Romain Naour:
>> Hi Ignacy,
>>
>> Le 01/02/2017 à 14:53, Ignacy Gawędzki a écrit :
>>> Include the --disable-dependency-tracking option in
>>> <pkg>_CONFIGURE_CMDS only on the condition that <pkg>_OVERRIDE_SRCDIR
>>> is empty.  Dependency tracking is very welcome while developing in
>>> order to properly rebuild when calling make <pkg>-rebuild for
>>> instance.
>>>
>>> Signed-off-by: Ignacy Gawędzki <ignacy.gawedzki at green-communications.fr>
>>> ---
>>>  package/pkg-autotools.mk | 6 ++++--
>>>  1 file changed, 4 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/package/pkg-autotools.mk b/package/pkg-autotools.mk
>>> index f7b8488..e8579f4 100644
>>> --- a/package/pkg-autotools.mk
>>> +++ b/package/pkg-autotools.mk
>>> @@ -199,7 +199,8 @@ define $(2)_CONFIGURE_CMDS
>>>  		--disable-documentation \
>>>  		--with-xmlto=no \
>>>  		--with-fop=no \
>>> -		--disable-dependency-tracking \
>>> +		$$(if $$($$(PKG)_OVERRIDE_SRCDIR),, \
>>> +		   --disable-dependency-tracking) \
>>>  		--enable-ipv6 \
>>>  		$$(DISABLE_NLS) \
>>>  		$$(SHARED_STATIC_LIBS_OPTS) \
>>> @@ -232,7 +233,8 @@ define $(2)_CONFIGURE_CMDS
>>>  		--disable-debug \
>>>  		--with-xmlto=no \
>>>  		--with-fop=no \
>>> -		--disable-dependency-tracking \
>>> +		$$(if $$($$(PKG)_OVERRIDE_SRCDIR),, \
>>> +		   --disable-dependency-tracking) \
>>
>> You're likely working on the target variant, so probably you can let the
>> dependency tracking enabled for the host?
> 
> Dependency tracking was initially *disabled* for both target and host.
> I suppose that <pkg>_OVERRIDE_SRCDIR works for both target and host,
> so I don't see why we should leave it disabled for one and not the
> other.

Indeed <pkg>_OVERRIDE_SRCDIR works for both target and host packages and the
"dependency tracking" was disabled (see [1]) to reduce the build time.

My initial feeling is to allow to enable the "dependency tracking" only when
doing active development on a *target* package using <pkg>_OVERRIDE_SRCDIR.
Buildroot is unlikely used for active development on a *host* package.

But ok, if you're using <pkg>_OVERRIDE_SRCDIR for a host package, you probably
don't really care of the build time...

Lets see other opinion.

Best regards,
Romain

[1]
https://git.busybox.net/buildroot/commit/?id=3e37b0fc6cbbe2245e39fedb247a381fb4e0d992

> 
> Regards,
> 
> Ignacy
> 




More information about the buildroot mailing list