Re: [Buildroot] [PATCH v2 1/1] package/composer: needs host-php to extract phar files

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Bernd,

On Fri,  4 Aug 2023 17:50:09 +0200
Bernd Kuhls <bernd@kuhls.net> wrote:

> Although composer has host-php as build dependency, in the case of
> BR2_PER_PACKAGE_DIRECTORIES=y host-php was not built during extract stage
> of the composer package.

It's not just BR2_PER_PACKAGE_DIRECTORIES=y that is broken. If you run
"make host-composer-extract", you'd get the same result, without
per-package directories. I've adjusted a bit the commit message.

>  HOST_COMPOSER_DEPENDENCIES = host-php
> +HOST_COMPOSER_EXTRACT_DEPENDENCIES = host-php

Here I hesitated a bit because those dependencies (build and extract)
are now slightly redundant. However, it's true that we need host-php at
extract time, but also to use host-composer. But for host packages we
don't really have the concept of runtime dependency: the only way to
ensure a runtime dependency will be there at runtime is to make it a
build-time dependency, which is what we do here with
<pkg>_DEPENDENCIES. So I've kept it like you did, but I will certainly
be willing to hear arguments of others as I don't have a very strong
opinion there.

Applied to master with the tweaked commit log. Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot