[Buildroot] [PATCH 1/1] package/bind: fix python build

Peter Korsgaard peter at korsgaard.com
Tue Apr 30 07:21:08 UTC 2019


>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice at gmail.com> writes:

 > A check for python-ply has been added as this is a dependency of the
 > dnssec-keymgr script so install host-python-ply to avoid a build failure
 > if python-ply is not installed on host

 > Fixes:
 >  - http://autobuild.buildroot.org/results/96815b1300547c976443bf74b762febdfcc8d3ba

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
 > ---
 >  package/bind/bind.mk | 7 +++++--
 >  1 file changed, 5 insertions(+), 2 deletions(-)

 > diff --git a/package/bind/bind.mk b/package/bind/bind.mk
 > index 356bc259b2..087ce941b0 100644
 > --- a/package/bind/bind.mk
 > +++ b/package/bind/bind.mk
 > @@ -70,8 +70,11 @@ else
 >  BIND_CONF_OPTS += --with-openssl=no
 >  endif
 
 > -# Used by dnssec-checkds and dnssec-coverage
 > -ifeq ($(BR2_PACKAGE_PYTHON)$(BR2_PACKAGE_PYTHON3),)
 > +# Used by dnssec-keymgr
 > +ifeq ($(BR2_PACKAGE_PYTHON_PLY),y)
 > +BIND_DEPENDENCIES += host-python-ply
 > +BIND_CONF_OPTS += --with-python=$(HOST_DIR)/usr/bin/python

Hmm, I somehow missed the host/target mixup. This is not good. You test
if the target python-ply package is enabled, but then add
host-python-ply (which doesn't exist) to _DEPENDENCIES:

http://autobuild.buildroot.net/results/2615ebfdcb068594d028af4823260e613d37e867/build-end.log

If I understand it right, we need ply for the host to satisfy the
configure check and ply for the target for runtime, so I guess the
solution is to add a host version of the python-ply package.

-- 
Bye, Peter Korsgaard


More information about the buildroot mailing list