Re: [ptxdist] usage of new local-kernel method fails
On Tue, Jul 19, 2011 at 10:09PM +0200, Robert Schwebel wrote: This was a real quick answer... thanks !! On Tue, Jul 19, 2011 at 09:59:06PM +0200, Jürgen Kilb wrote: i've tried to use the new 'local-kernel' method, which was introduced by commit cef51e6071 " remove old local-kernel method" ;-) I've created a link 'kernel.phyCARD-L' under local_src which points to my local git repository. Sounds right. But ptxdist is still trying to download the kernel sources... and fails. It downloads the right kernel sources - specified in platformconfig - but still uses your link. OK, now I've entered a version which was available/downloadable and it is working now. I'm not sure if it is a good idea to do so (because if you specify linux-x.y.z you will need to fetch it anyway) or a misfeature because it was easier to implement this way :-) Now that I know how it is working, it is OK. However, I think the new solution is much better than what we had before. Improvements are welcome, of course. rsc I agree, the solution is better now. Normaly I would expect, the packet download would be skipped for local sources. Greetings, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] usage of new local-kernel method fails
Hi, i've tried to use the new 'local-kernel' method, which was introduced by commit cef51e6071 " remove old local-kernel method" ;-) I've created a link 'kernel.phyCARD-L' under local_src which points to my local git repository. But ptxdist is still trying to download the kernel sources... and fails. For other packages like u-boot or barebox it is working. For me it seems ptxd_make_world_common.sh is called in a different way if the kernel package should be build. Any hints ? greetings, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH v1] Quotation fix of image_jffs2.make
Hi Michael, sorry, I was to fast yesterday evening. New patch follows.. > > $(IMAGEDIR)/root.jffs2: $(STATEDIR)/image_working_dir $(STATEDIR)/host-mtd-utils.install.post > > @echo -n "Creating root.jffs2 from working dir... (--eraseblock=$ > (PTXCONF_IMAGE_JFFS2_BLOCKSIZE) $(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS))" > > can you add the remove_quotes here too? I don't think it's needed right > now, but it would be more consistent. Ok, splitted the long line. > > > - @cd $(image/work_dir); \ > > + @cd $(image/work_dir); \ > > (awk -F: $(DOPERMISSIONS) $(image/permissions) && \ > > (\ > >echo -n "$(PTXCONF_SYSROOT_HOST)/sbin/mkfs.jffs2 "; \ > > - echo -n "-d $(image/work_dir) ";\ > > + echo -n "-d $(image/work_dir) "; \ > >echo -n "--eraseblock=$(PTXCONF_IMAGE_JFFS2_BLOCKSIZE) "; \ > > - echo -n "$(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS) "; \ > > + echo -n "$(call remove_quotes,$(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS)) ";\ > >echo -n "-o $@" ) \ > > hmmm, perhaps align all '\' again. I've send the patch with 'stg mail' I will check again. > > > - ) | $(FAKEROOT) -- > > + ) | tee $(STATEDIR)/logfile.txt | $(FAKEROOT) -- > > If we log here this should be 'tee -a "$(PTX_LOGFILE)"', but I'm not > certain if PTX_LOGFILE is defined here. > It seems PTX_LOGFILE is defined, so I added 'tee -a "$(PTX_LOGFILE)"'. Maybe a bit too much output what do you think ? > > @echo "done." > > > > $(IMAGEDIR)/root.sum.jffs2: $(IMAGEDIR)/root.jffs2 > > @echo -n "Creating root.sum.jffs2 with summary... (--eraseblock=$ > (PTXCONF_IMAGE_JFFS2_BLOCKSIZE) $(PTXCONF_IMAGE_JFFS2_SUM_EXTRA_ARGS))" > > dito. OK, done. > > > - @cd $(image/work_dir); \ > > + @cd $(image/work_dir); \ > > (\ > >echo -n "$(PTXCONF_SYSROOT_HOST)/sbin/sumtool "; \ > >echo -n "-i $< "; \ > >echo -n "--eraseblock=$(PTXCONF_IMAGE_JFFS2_BLOCKSIZE) "; \ > > - echo -n "$(PTXCONF_IMAGE_JFFS2_SUM_EXTRA_ARGS) "; \ > > + echo -n "$(call remove_quotes,$(PTXCONF_IMAGE_JFFS2_SUM_EXTRA_ARGS)) "; \ > >echo -n "-o $@" \ > > align. OK, done. > > > ) | $(FAKEROOT) -- > > This should be consistent. Log both or neither. > OK, fixed. -- Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] problems with post rule image_jffs2.make
Hi Michael, > > The problem is that PTXCONF_IMAGE_JFFS2_EXTRA_ARGS already contains the > quotation marks. > echo -n "$(call remove_quotes,$(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS)) ";\ > is probably the correct fix. Or > echo -n $(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS); \ > and add the space in the next line. > Just to make clear I used the quotation marks to show what I have entered in the ptxdist menu->platformconfig->image creation options->Generate images/root.jffs2->extra arguments passed to mkfs.jffs2 So if PTXCONF_IMAGE_JFFS2_EXTRA_ARGS contains already the quotation marks... they are comming from the menu ;-) Isn't it a general problem which has to be fixed also in other rules/makefiles ? Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] problems with post rule image_jffs2.make
Hi, i've encountered a problem with the post rule "image_jffs2.make" and maybe some other places... If I set the PTXCONF_IMAGE_JFFS2_EXTRA_ARGS to "-n -p" the created JFFS2 file is paded but includes JFFS2 clean markers. If I change "-n" to "--no-cleanmarkers" the JFFS2 image is created without clean markers as expected. With "-p -n" the JFFS2 image is padded and has no clean markers... I digged around and found that the parameter "-n" is not passed to mkfs.jffs2. It looks like it has something to do with the expansion of the variable PTXCONF_IMAGE_JFFS2_EXTRA_ARGS in the makefile "image_jffs2.make" If I change the line echo -n "$(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS) "; \ to echo -n ""$(PTXCONF_IMAGE_JFFS2_EXTRA_ARGS) ""; \ it is working... but is this the right solution ? I can do the same at the console: desktop:~$ FOO="-n -p" desktop:~$ echo $FOO -pdesktop:~$FOO="-p -n" desktop:~$ echo $FOO -p -n desktop:~$ Any thoughts on this ? Greetings, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] different project specific settings for different platforms in unified BSP
Hi Robert, > > Any hints/ideas ? > > ptxdist menuconfig platform > That is only half the truth ;-) Now I found out, that I have to create the "platforms" directory in my project, which includes an Kconfig file which source the toplevel Kconfig from "platforms/Kconfig". In that Kconfig file I can create my own platform specific entries. Greetings, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] different project specific settings for different platforms in unified BSP
Hi, I am working on a unified BSP which includes three different platforms i.e. three different CPUs. I wonder what is the best way to handle CPU specific software in my case bootloader tools (x-load for OMAP) ? I can include a rule for every CPU specific tool, but all of them will be shown in the project_specific menu... But I don't want to see the OMAP x-loader if I build the BSP for an i.MX... This would lead to a platform_specific menu ? Any hints/ideas ? Greetings, Jürgen -Entwicklung- PHYTEC Messtechnik GmbH Robert-Koch-Str. 39 55129 Mainz Germany Tel.: +49 (0)6131 9221-45 Web: http://www.phytec.de Geschäftsführer: Dipl.-Ing. Michael Mitezki, Handelsregister Mainz, HRB 4656, Finanzamt Mainz-Mitte, St.Nr. 266500608, DE 149059855 This E-Mail may contain confidential or privileged information. If you are not the intended recipient (or have received this E-Mail in error) please notify the sender immediately and destroy this E-Mail. Any unauthorized copying, disclosure or distribution of the material in this E-Mail is strictly forbidden. -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH 3] Add mpg123 rule/in file.
Hi Michael, On Tue, Jun 01, 2010 at 21:42 PM +0200 Michael Olbrich wrote: > > +## SECTION=multimedia_sound > > + > > +config MPG123 > > + tristate > > + prompt "mpg123" > > select ALSA_LIB > > I don't think this needs to be optional. > What do you mean ? How can I mark the alsa-lib as non optional ? Greetings, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] OSELAS.BSP-Phytec-phyCORE-9 mplay
Hi Jan, Am Samstag, den 21.02.2009, 18:08 +0100 schrieb Jan Weitzel: > cpudetect.c: In function 'sigill_handler_sse': > cpudetect.c:303: error: 'struct sigcontext' has no member named 'eip' > make[1]: *** [cpudetect.o] Error 1 > make[1]: *** Waiting for unfinished jobs 'eip' ? Extended Instruction Pointer on an arm ?? It seems you compile 'mplay' with an x86 compiler ? see you, Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [Patch] openssl.make, corrected 'ifneq' to set the THUD variable correctly.
Hi Robert, [EMAIL PROTECTED] wrote on 01.04.2008 11:16:28: > Could you please try again, with latest trunk (>= 7901)? I've applied > the patch below, which should also work with make 3.80. > It's now OK. -Jürgen-- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [Patch] openssl.make, corrected 'ifneq' to set the THUD variable correctly.
Hi, [EMAIL PROTECTED] wrote on 31.03.2008 18:22:23: > On Mon, Mar 31, 2008 at 05:54:09PM +0200, Jürgen Kilb wrote: > > If i do the same, the result is different... but why ? > > > > [EMAIL PROTECTED]:~> cat Makefile > > BAZ=$(and $(FOO),$(BAR)) > > > > all: > > @echo $(BAZ) > > [EMAIL PROTECTED]:~> make FOO= BAR= > > > > [EMAIL PROTECTED]:~> make FOO=y BAR= > > > > [EMAIL PROTECTED]:~> make FOO= BAR=y > > > > [EMAIL PROTECTED]:~> make FOO=y BAR=y > > > > [EMAIL PROTECTED]:~> > > > > Maybe the 'make' version ? > > > > [EMAIL PROTECTED]:~> make -v > > GNU Make 3.80 > > Copyright (C) 2002 Free Software Foundation, Inc. > > This is free software; see the source for copying conditions. > > There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A > > PARTICULAR PURPOSE. > > [EMAIL PROTECTED]:~> > > Hmm, I have 3.81. Does anyone have a better idea, or should we depend on > 3.81? However, it would be a little bit uggly... > make was the problem :-( It works with a fresh build "make-3.81" [EMAIL PROTECTED]:~/make-3.81> cd .. [EMAIL PROTECTED]:~> make-3.81/make FOO= BAR= [EMAIL PROTECTED]:~> make-3.81/make FOO=y BAR= [EMAIL PROTECTED]:~> make-3.81/make FOO= BAR=y [EMAIL PROTECTED]:~> make-3.81/make FOO=y BAR=y y [EMAIL PROTECTED]:~> -Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [Patch] openssl.make, corrected 'ifneq' to set the THUD variable correctly.
Hi, [EMAIL PROTECTED] wrote on 31.03.2008 17:05:35: > So the arguments are expanded as you described, the $(and) function expands to > the expansion of the last argument (y), which is != nothing. > > Should be right. Stand alone example: > > [EMAIL PROTECTED]:~$ cat Makefile > BAZ=$(and $(FOO),$(BAR)) > > all: > @echo $(BAZ) > [EMAIL PROTECTED]:~$ make FOO= BAR= > > [EMAIL PROTECTED]:~$ make FOO=y BAR= > > [EMAIL PROTECTED]:~$ make FOO= BAR=y > > [EMAIL PROTECTED]:~$ make FOO=y BAR=y > y > > Q.E.D. If i do the same, the result is different... but why ? [EMAIL PROTECTED]:~> cat Makefile BAZ=$(and $(FOO),$(BAR)) all: @echo $(BAZ) [EMAIL PROTECTED]:~> make FOO= BAR= [EMAIL PROTECTED]:~> make FOO=y BAR= [EMAIL PROTECTED]:~> make FOO= BAR=y [EMAIL PROTECTED]:~> make FOO=y BAR=y [EMAIL PROTECTED]:~> Maybe the 'make' version ? [EMAIL PROTECTED]:~> make -v GNU Make 3.80 Copyright (C) 2002 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. [EMAIL PROTECTED]:~> -Jürgen -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] Jürgen Kilb/Mainz/Phytec is out of the office.
Ups..sorry for that Out of Office Mail.I've corrected that ;-)-Juergen -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] Jürgen Kilb/Mainz/Phytec is out of the office.
I will be out of the office starting Fri 06/01/2007 and will not return until Mon 07/02/2007. In urgent cases please contact my substitute [EMAIL PROTECTED] -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] corrected some package sources
Hi,please find attached an patch which will correct some package sources:- Jürgendiff -urN rules.org/mgetty.make rules/mgetty.make--- rules.org/mgetty.make 2007-01-28 22:17:41.0 +0100+++ rules/mgetty.make 2007-04-19 21:02:21.0 +0200@@ -21,7 +21,7 @@ MGETTY_DATE = Dec16 MGETTY = mgetty$(MGETTY_VERSION) MGETTY_SUFFIX = tar.gz-MGETTY_URL = ftp://ftp.leo.org/historic/comp/os/unix/networking/mgetty/$(MGETTY)-$(MGETTY_DATE).$(MGETTY_SUFFIX)+MGETTY_URL = ftp://alpha.greenie.net/pub/mgetty/source/1.1/$(MGETTY)-$(MGETTY_DATE).$(MGETTY_SUFFIX) MGETTY_SOURCE = $(SRCDIR)/$(MGETTY)-$(MGETTY_DATE).$(MGETTY_SUFFIX) MGETTY_DIR = $(BUILDDIR)/mgetty-$(MGETTY_VERSION)diff -urN rules.org/pcre.make rules/pcre.make--- rules.org/pcre.make 2007-01-28 22:17:41.0 +0100+++ rules/pcre.make 2007-04-19 20:37:20.0 +0200@@ -20,8 +20,8 @@ # PCRE_VERSION = 5.0 PCRE = pcre-$(PCRE_VERSION)-PCRE_SUFFIX = tar.gz-PCRE_URL = ftp://ftp.csx.cam.ac.uk/pub/software/programming/pcre/$(PCRE).$(PCRE_SUFFIX)+PCRE_SUFFIX = tar.bz2+PCER_URL = http://mesh.dl.sourceforge.net/sourceforge/pcre/$(PCRE).$(PCRE_SUFFIX) PCRE_SOURCE = $(SRCDIR)/$(PCRE).$(PCRE_SUFFIX) PCRE_DIR = $(BUILDDIR)/$(PCRE)diff -urN rules.org/ppp.make rules/ppp.make--- rules.org/ppp.make 2007-01-28 22:17:41.0 +0100+++ rules/ppp.make 2007-04-19 21:22:25.0 +0200@@ -19,7 +19,7 @@ # PPP_VERSION = 2.4.1 PPP = ppp-$(PPP_VERSION)-PPP_URL = ftp://ftp.samba.org/pub/ppp/$(PPP).tar.gz+PPP_URL = http://samba.org/ftp/ppp/$(PPP).tar.gz PPP_SOURCE = $(SRCDIR)/$(PPP).tar.gz PPP_DIR = $(BUILDDIR)/$(PPP)diff -urN rules.org/rsync.make rules/rsync.make--- rules.org/rsync.make 2007-01-28 22:17:41.0 +0100+++ rules/rsync.make 2007-04-19 21:11:50.0 +0200@@ -20,7 +20,7 @@ RSYNC_VERSION = 2.6.8 RSYNC = rsync-$(RSYNC_VERSION) RSYNC_SUFFIX = tar.gz-RSYNC_URL = http://samba.anu.edu.au/ftp/rsync/$(RSYNC).$(RSYNC_SUFFIX)+RSYNC_URL = http://samba.anu.edu.au/ftp/rsync/old-versions/$(RSYNC).$(RSYNC_SUFFIX) RSYNC_SOURCE = $(SRCDIR)/$(RSYNC).$(RSYNC_SUFFIX) RSYNC_DIR = $(BUILDDIR)/$(RSYNC) -- ptxdist mailing list ptxdist@pengutronix.de