From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: From: =?iso-8859-1?q?J=FCrgen_Beisert?= Date: Mon, 6 Jan 2014 12:20:54 +0100 References: <1368448690-15034-1-git-send-email-alex.aring@gmail.com> <20140106082507.GA12646@omega> <20140106105057.GF2156@pengutronix.de> In-Reply-To: <20140106105057.GF2156@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline Message-Id: <201401061220.54164.jbe@pengutronix.de> Subject: Re: [ptxdist] [PATCH v2] lowpan-tools: update lowpan-patches Reply-To: ptxdist@pengutronix.de List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist@pengutronix.de Cc: Michael Olbrich , Alexander Aring On Monday 06 January 2014 11:50:57 Michael Olbrich wrote: > On Mon, Jan 06, 2014 at 09:25:08AM +0100, Alexander Aring wrote: > > On Mon, Jan 06, 2014 at 08:40:24AM +0100, J=FCrgen Beisert wrote: > > > On Monday 13 May 2013 17:00:36 Michael Olbrich wrote: > > > > On Mon, May 13, 2013 at 02:38:10PM +0200, Alexander Aring wrote: > > > > > Currently there is no way to make a new lowpan-tools release. > > > > > This patch synchronize the patches from version 0.3 with current > > > > > git master. > > > > > > > > > > Delete Patch 0001-Avoid-crashing-in-izcoordinator.patch. Which is > > > > > replaced by upstream > > > > > 0001-izoordinator-Fixes-to-error-handling.patch patch. > > > > > > > > Thanks, applied. > > > > > > This package fails in the prepare stage when the LOWPAN_TOOLS_TESTS is > > > not set. The 'configure' script always checks for python. > > > > > > I do not know yet how this package depends on python: Should we always > > > build python in order to make this package build again or should we a= dd > > > a new parameter to its configure.ac to switch off this test, when the > > > test tools are disabled? > > > > indeed. The 'configure' script always checks for python and generate a > > python 'site-package module'. > > > > The LOWPAN_TOOLS_TESTS needs this 'module' only. But at the moment there > > is no 'enable/disable' switch in the 'configure' script. > > > > We shall change: > > > > "select PYTHON if LOWPAN_TOOLS_TESTS" to "select PYTHON" > > > > Thanks for the hint! > > That should not be necessary. PTXdist always needs the system python, so > setting ac_cv_path_PYTHON=3Dpython should be enough to get > /bin/python. The result won't be anything usefull, but it should > compile. Yes, it does. The required change looks very simply now: diff --git a/rules/lowpan-tools.make b/rules/lowpan-tools.make index a4919bd..24e7568 100644 --- a/rules/lowpan-tools.make +++ b/rules/lowpan-tools.make @@ -35,7 +35,7 @@ LOWPAN_TOOLS_CONF_ENV =3D \ ifdef PTXCONF_LOWPAN_TOOLS_TESTS LOWPAN_TOOLS_CONF_ENV +=3D ac_cv_path_PYTHON=3D$(CROSS_PYTHON) else -LOWPAN_TOOLS_CONF_ENV +=3D ac_cv_path_PYTHON=3D: +LOWPAN_TOOLS_CONF_ENV +=3D ac_cv_path_PYTHON=3Dpython endif # Tested-by: Juergen Beisert But: "select PYTHON" means Python is required at run-time. Using = the "$(CROSS_PYTHON)" suggests Python is required only at build-time. Am I = wrong? jbe -- = Pengutronix e.K. =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = =A0| Juergen Beisert =A0 =A0 =A0 =A0 =A0 =A0 | Linux Solutions for Science and Industry =A0 =A0 =A0| http://www.pengutroni= x.de/ | -- = ptxdist mailing list ptxdist@pengutronix.de