[Buildroot] [PATCH] core: postpone evaluation on decompressor dependencies

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sun Jul 2 10:36:21 UTC 2017


Hello,

On Sun,  2 Jul 2017 12:30:30 +0200, Yann E. MORIN wrote:
> Currently, we build any extractor dependency as soone as the host

soone -> soon

> utility is missing, even when such utility is not needed.
> 
> That is expecially the case for host-lzip, which is lacking except in

expecially -> especially

> the most recent distributions, and thus is always built, which is a bit

I'm not sure to understand the "except in the most recent
distributions". Indeed, even in the most recent distributions, lzip may
not be installed.

> of a shame since only very few packages need it.
> 
> To fix that, we change the host dependecies check to set a variable
> named after the extension they support, to the name of the host package
> that provides the necessary decompressor, if not installed on the host.
> 
> Then, we introduce a new variable, UNCOMP_TOOLS_DEPENDENCIES, to which

I think EXTRACT_TOOLS_DEPENDENCIES would be easier to understand. Yes,
it's not technically extraction but uncompression, but they are needed
as part of Buildroot "extract" step for each package, so I believe
EXTRACT_TOOLS_DEPENDENCIES would be better.

> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> index f474704980..5745a9cfcf 100644
> --- a/package/pkg-generic.mk
> +++ b/package/pkg-generic.mk
> @@ -942,6 +942,7 @@ DL_TOOLS_DEPENDENCIES += cvs
>  endif # SITE_METHOD
>  
>  DL_TOOLS_DEPENDENCIES += $$(call extractor-dependency,$$($(2)_SOURCE))
> +UNCOMP_TOOLS_DEPENDENCIES += $$(UNCOMP_TOOL_DEPENDENCY$$(suffix $$($(2)_SOURCE)))

I know we looked at it yesterday, but I again don't remember what
extractor-dependency is for. When looking at this code, it seems a bit
weird that extractor dependencies are added to DL_TOOLS_DEPENDENCIES
and "uncompressor dependencies" are added to UNCOMP_TOOLS_DEPENDENCIES.

Can we clarify this ? Should the output of extractor-dependency be added
to UNCOMP_TOOLS_DEPENDENCIES ? Do we need both extractor-dependency and
UNCOMP_TOOL_DEPENDENCY ?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


More information about the buildroot mailing list