[Buildroot] [PATCH 1/1] package/gesftpserver: Add host-python as dependency

[Andreas Naumann]

Hi,

On 23.06.21 11:35, Peter Korsgaard wrote:
>>>>>> "Baruch" == Baruch Siach <baruch at tkos.co.il> writes:
> 
>   > Hi Andreas,
>   > On Wed, Jun 23 2021, Andreas Naumann wrote:
>   >> gesftpserver needs python for the configure step.
>   >>
>   >> Signed-off-by: Andreas Naumann <anaumann at ultratronik.de>
>   >> ---
>   >> package/gesftpserver/gesftpserver.mk | 2 +-
>   >> 1 file changed, 1 insertion(+), 1 deletion(-)
>   >>
>   >> diff --git a/package/gesftpserver/gesftpserver.mk b/package/gesftpserver/gesftpserver.mk
>   >> index 85511a3c97..c67fdb3f79 100644
>   >> --- a/package/gesftpserver/gesftpserver.mk
>   >> +++ b/package/gesftpserver/gesftpserver.mk
>   >> @@ -19,7 +19,7 @@ GESFTPSERVER_CPE_ID_PRODUCT = sftpserver
>   >> GESFTPSERVER_CONF_ENV = LIBS=-lpthread
>   >>
>   >> # overwrite openssh version if enabled
>   >> -GESFTPSERVER_DEPENDENCIES += \
>   >> +GESFTPSERVER_DEPENDENCIES += host-python \
> 
>   > Does host-python3 also work? We would probably drop python2 in the near
>   > future.

So I just tried and yes, host-python3 does the job.

> 
> Exactly. Looking at it, python also only seems to be used for the tests
> that we do not need to run, so it is a bit silly to have to pull in
> host-python3.

Ok, i didnt look why Python is needed. Also, I never saw the problem 
until using per-package build. So buildtime-wise it's not a problem in 
my case because we pull in host-python for some other package anyway.

I will check if I can get rid of the tests though.

regards,
Andreas

>