[Buildroot] [PATCH 1/2] package/ninja: invoke python3 explicitly to configure the package

[Asaf Kahlon]

Hello,

On Mon, Nov 11, 2019 at 10:04 PM Avi Shukron <avraham.shukron at gmail.com> wrote:
>
> ninja depends on python3 specifically, but the configure.py file
> simply uses "env python". Where no python is selected for the target
> you simply won't get a python symlink in $(HOST_DIR)/usr/bin, so the
> configure.py script fails to run since it can't find "python".
>
> Notice that in order to reproduce the issue, you must not have
> python2 installed on your host machine.
>
> Signed-off-by: Avi Shukron <avraham.shukron at gmail.com>
> ---
>  package/ninja/ninja.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/ninja/ninja.mk b/package/ninja/ninja.mk
> index 4897012150..867b92c4c3 100644
> --- a/package/ninja/ninja.mk
> +++ b/package/ninja/ninja.mk
> @@ -17,7 +17,7 @@ NINJA_LICENSE_FILES = COPYING
>  HOST_NINJA_DEPENDENCIES = host-python3
>
>  define HOST_NINJA_BUILD_CMDS
> -       (cd $(@D); ./configure.py --bootstrap)
> +       (cd $(@D); $(HOST_DIR)/bin/python3 ./configure.py --bootstrap)

Well, I guess you can now remove the "./" before configure.py
as now you explicitly call python3 before it.

>  endef
>
>  define HOST_NINJA_INSTALL_CMDS
> --
> 2.17.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

Asaf.