[Buildroot] [PATCH v3 1/1] pkg-infra/show-info: dump install_{staging, target} info
Arnout Vandecappelle
arnout at mind.be
Fri Aug 2 20:42:26 UTC 2019
Hi Vadim,
Sorry to come with yet another comment in v3 of this extremely simple patch :-)
On 31/07/2019 17:32, Vadim Kochan wrote:
> provide info if the package will be installed to staging/target
> destinations. Might be useful for analyzing the packages which
> installed only for target/staging.
>
> Signed-off-by: Vadim Kochan <vadim4j at gmail.com>
> ---
> v2:
> 1) use oneliner 'if' to dump install_{staging,target} properties
>
> v3:
> 1) fix evaluation of true/false value via 'subst' (suggested by Yann E. Morin)
>
> package/pkg-utils.mk | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/package/pkg-utils.mk b/package/pkg-utils.mk
> index b7280e930f..ffe198e242 100644
> --- a/package/pkg-utils.mk
> +++ b/package/pkg-utils.mk
> @@ -94,6 +94,8 @@ endef
> define _json-info-pkg-details
> "version": "$($(1)_DL_VERSION)",
> "licenses": "$($(1)_LICENSE)",
> + "install-target": $(if $(subst NO,,$($(1)_INSTALL_TARGET)),true,false),
> + "install-staging": $(if $(subst NO,,$($(1)_INSTALL_STAGING)),true,false),
I still find this ugly.
Could you instead add a macro that does something like
$(subst NO,false,$(subst YES,true,$(1)))
and use that?
It may also be handy in other situations.
I'm not sure what to call it, maybe bool-to-json or yesno-to-bool
Regards,
Arnout
> "downloads": [
> $(foreach dl,$(sort $($(1)_ALL_DOWNLOADS)),
> {
>
More information about the buildroot
mailing list