From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-ee0-x232.google.com ([2a00:1450:4013:c00::232]) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1WQwKA-0008Mz-AA for ptxdist@pengutronix.de; Fri, 21 Mar 2014 11:05:15 +0100 Received: by mail-ee0-f50.google.com with SMTP id c13so1590202eek.37 for ; Fri, 21 Mar 2014 03:05:08 -0700 (PDT) Received: from omega (p20030064AA051101E2CB4EFFFE1BB546.dip0.t-ipconnect.de. [2003:64:aa05:1101:e2cb:4eff:fe1b:b546]) by mx.google.com with ESMTPSA id a4sm10546908eep.12.2014.03.21.03.05.07 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 21 Mar 2014 03:05:07 -0700 (PDT) Date: Fri, 21 Mar 2014 11:05:03 +0100 From: Alexander Aring Message-ID: <20140321100501.GA15409@omega> References: <1394716005-18738-1-git-send-email-alex.aring@gmail.com> <20140321090054.GM23331@pengutronix.de> <20140321093200.GD25178@omega> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20140321093200.GD25178@omega> Subject: Re: [ptxdist] [PATCH 1/2] host-cython: add new package 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="us-ascii" Content-Transfer-Encoding: 7bit Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist@pengutronix.de Hi Michael, On Fri, Mar 21, 2014 at 10:32:00AM +0100, Alexander Aring wrote: > On Fri, Mar 21, 2014 at 10:00:55AM +0100, Michael Olbrich wrote: > > On Thu, Mar 13, 2014 at 02:06:44PM +0100, Alexander Aring wrote: > > > Signed-off-by: Alexander Aring > > > --- > > > Hi, > > > > > > I am not sure about the following points: > > > > > > i) helptext: it's a host package with noprompt, but I saw a helptext in > > > host-python also. That's why I add a help text here. > > > > > > ii) Cython installs some site-packages which is needed by $(CROSS_PYTHON). I > > > install things in --root=$(PTXCONF_SYSROOT_TARGET) because $(CROSS_PYTHON) > > > used this as site-packages search path. > > > > Both patches applied. > > > > I changed this to "--root=$(HOST_CYTHON_PKGDIR)" and --prefix=. This way > > 'cython' runs just fine. I'm not sure anything else is needed. 'pyzmq' > > builds without error even if cython is not installed here. Please test. > > > > I read too fast the README on github. Cython is only needed if we build > from git repository but I need cython for other packages. I will test if > this still works. Thanks Michael. Now I understand a little bit more. I installed it into PTXCONF_SYSROOT_TARGET because CROSS_PYTHON has the default site-packages search path there. I know this was wrong because it is a host tool which should installed into PTXCONF_SYSROOT_HOST. I always search a way to change the default site-packages search path and now I found it! [1] Your solution to install it into HOST_CYTHON_PKGDIR will install Cython into PTXCONF_SYSROOT_HOST which is correct, but then it doesn't work out of the box, because CROSS_PYTHON doesn't search in PTXCONF_SYSROOT_HOST. With solution of [1] I can change the site-packages search path with: "$(CROSS_ENV) PYTHONUSERBASE=$(PTXCONF_SYSROOT_HOST) $(CROSS_PYTHON)" and then it works with a installed Cython in PTXCONF_SYSROOT_HOST. Please let me know if this is okay for you, maybe we can put this into the default $(CROSS_PYTHON) path. It seems Cython is some special thing, we need it only at compiletime and it have some cross compiled things in there. I don't know what happens if we set PYTHONUSERBASE and python uses still his default site-packages path or not. - Alex [1] http://pymotw.com/2/site/ -- ptxdist mailing list ptxdist@pengutronix.de