mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] Using PTXdist on Mac OS
@ 2012-04-09 17:11 Bernhard Walle
  2012-04-09 17:48 ` Juergen Beisert
  2012-04-09 18:01 ` Andreas Bießmann
  0 siblings, 2 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-09 17:11 UTC (permalink / raw)
  To: ptxdist

Hello,

on a rainy Easter Monday I wrote some small HOWTO that describes the
pitfalls of using PTXdist on Mac OS. It is based on the Generic SDK
using Qemu.

The HOWTO is available at http://www.bwalle.de/docs/ptxdist_mac.pdf, the
LaTeX sources are at https://bitbucket.org/bwalle/ptxdist-macos-doc.

Maybe it can be linked somewhere in the official documentation or at the
website? If something is wrong or missing, drop me a note.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 17:11 [ptxdist] Using PTXdist on Mac OS Bernhard Walle
@ 2012-04-09 17:48 ` Juergen Beisert
  2012-04-10  9:37   ` Bernhard Walle
  2012-04-09 18:01 ` Andreas Bießmann
  1 sibling, 1 reply; 37+ messages in thread
From: Juergen Beisert @ 2012-04-09 17:48 UTC (permalink / raw)
  To: ptxdist; +Cc: Bernhard Walle

Hi Bernhard,

Bernhard Walle wrote:
> on a rainy Easter Monday I wrote some small HOWTO that describes the
> pitfalls of using PTXdist on Mac OS. It is based on the Generic SDK
> using Qemu.

Rainy weather seems to be a motivator for writing documentation. :)

> The HOWTO is available at http://www.bwalle.de/docs/ptxdist_mac.pdf, the
> LaTeX sources are at https://bitbucket.org/bwalle/ptxdist-macos-doc.

"[...] The reader may ask why I use Mac OS to build Linux systems. [...]  
just “because I can”."

\o/

> Maybe it can be linked somewhere in the official documentation or at the
> website?

If you promise to keep the document at your website at the given URL, I would 
add it to the Guru manual. 

You mentioned your 'https://bitbucket.org/bwalle/ptxdist-arm-boards' BSP in 
you document. I would like to add it to our list of free BSPs 
at "http://www.pengutronix.de/oselas/bsp/pengutronix/index_en.html" if you 
send me some information about its features (to give the reader the same 
overview like with our free BSPs).

> If something is wrong or missing, drop me a note.

Just a few things that confuse me when I read it the first time. I will send 
you a patch (offlist) with my suggestions.

Regards,
Juergen

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 17:11 [ptxdist] Using PTXdist on Mac OS Bernhard Walle
  2012-04-09 17:48 ` Juergen Beisert
@ 2012-04-09 18:01 ` Andreas Bießmann
  2012-04-09 18:59   ` Bernhard Walle
  1 sibling, 1 reply; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-09 18:01 UTC (permalink / raw)
  To: ptxdist; +Cc: Bernhard Walle

Dear Bernhard,

On 09.04.12 19:11, Bernhard Walle wrote:

> The HOWTO is available at http://www.bwalle.de/docs/ptxdist_mac.pdf, the
> LaTeX sources are at https://bitbucket.org/bwalle/ptxdist-macos-doc.

I have some things to say about your HowTo (some patches may follow).

a) Fink is _not_ Debian based but uses the great Debian package format
(*.deb ;)
b) FSF GCC is _not_ required for GNU Toolchain bootstrap, I did compile
my arm-cortexa8, arm-v4t, arm-v5te, x86_64 and avr32 OSELAS.Toolchain with:

---8<---
andreas@andreas-mbp % ls -l /usr/bin/gcc
lrwxr-xr-x  1 root  wheel  12 21 Nov 07:39 /usr/bin/gcc@ -> llvm-gcc-4.2
andreas@andreas-mbp % gcc --version
i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1 (Based on Apple Inc. build
5658) (LLVM build 2336.1.00)
Copyright (C) 2007 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.
--->8---

(But maybe there are some Fink stuff leaking in and let it build on my
system? Did not investigate it further ...)

c) you forget Fink package names (patches follow ;)

regards

Andreas Bießmann

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 18:01 ` Andreas Bießmann
@ 2012-04-09 18:59   ` Bernhard Walle
  2012-04-09 19:47     ` Juergen Beisert
                       ` (2 more replies)
  0 siblings, 3 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-09 18:59 UTC (permalink / raw)
  To: Andreas Bießmann; +Cc: ptxdist

Hi,

* Andreas Bießmann <andreas@biessmann.de> [2012-04-09 20:01]:
> 
> a) Fink is _not_ Debian based but uses the great Debian package format
> (*.deb ;)

Well, depends on what you call 'Debian-based'. I didn't say that the
packages are from Debian (the MacPorts packages are also not from
Free/Open/NetBSD).

finkproject.org says "Fink uses Debian tools like dpkg and apt-get to
provide powerful binary package management. " So is the wording

Is following acceptable?

------------------------------------ 8< ------------------------------------
  \item \textbf{Fink} (\url{http://www.finkproject.org}) uses the package
    management tools and the package format from Debian.  Contrary to
    MacPorts, it has both binaries and can be built from source (but as far as
    I know, the binaries are less up-to-date compared the source packages).
------------------------------------ >8 ------------------------------------

> b) FSF GCC is _not_ required for GNU Toolchain bootstrap, I did compile
> my arm-cortexa8, arm-v4t, arm-v5te, x86_64 and avr32 OSELAS.Toolchain with:

This experience is from the time when I used Xcode 4.2 (I had to wait
for the update until MacPorts got support for it). I got llvm-gcc
segmentation faults and this was also "documented" in the bug tracker of
FSF GCC. And I think that in LLVM bugtracker was a fix already in SVN of
LLVM GCC.

I'm just running a build of a toolchain with Xcode 4.3(.2) LLVM GCC and
if that build succeeds then I will remove the restriction to FSF GCC and
mention Xcode 4.3 somewhere in the requirements.

bwalle@euler % gcc --version
i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.9.00)

> (But maybe there are some Fink stuff leaking in and let it build on my
> system? Did not investigate it further ...)

I don't think that some Fink environment settings fixes GCC segmentation faults.
But we will see.

> c) you forget Fink package names (patches follow ;)

Patch is applied, thanks. But I will wait for Jürgens comments until I
upload a new PDF.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 18:59   ` Bernhard Walle
@ 2012-04-09 19:47     ` Juergen Beisert
  2012-04-09 20:04     ` Andreas Bießmann
  2012-04-09 20:48     ` Andreas Bießmann
  2 siblings, 0 replies; 37+ messages in thread
From: Juergen Beisert @ 2012-04-09 19:47 UTC (permalink / raw)
  To: ptxdist; +Cc: Bernhard Walle, Andreas Bießmann

Bernhard Walle wrote:
> [...]
> > c) you forget Fink package names (patches follow ;)
>
> Patch is applied, thanks. But I will wait for Jürgens comments until I
> upload a new PDF.

And /me is still fighting with git $&§&%&&$/%($

Regards,
Juergen

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 18:59   ` Bernhard Walle
  2012-04-09 19:47     ` Juergen Beisert
@ 2012-04-09 20:04     ` Andreas Bießmann
  2012-04-09 20:48     ` Andreas Bießmann
  2 siblings, 0 replies; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-09 20:04 UTC (permalink / raw)
  To: ptxdist; +Cc: Bernhard Walle

Hi,

On 09.04.12 20:59, Bernhard Walle wrote:
> Hi,
> 
> * Andreas Bießmann <andreas@biessmann.de> [2012-04-09 20:01]:
>>
>> a) Fink is _not_ Debian based but uses the great Debian package format
>> (*.deb ;)
> 
> Well, depends on what you call 'Debian-based'. I didn't say that the
> packages are from Debian (the MacPorts packages are also not from
> Free/Open/NetBSD).
> 
> finkproject.org says "Fink uses Debian tools like dpkg and apt-get to
> provide powerful binary package management. " So is the wording
> 
> Is following acceptable?
> 
> ------------------------------------ 8< ------------------------------------
>   \item \textbf{Fink} (\url{http://www.finkproject.org}) uses the package
>     management tools and the package format from Debian.  Contrary to
>     MacPorts, it has both binaries and can be built from source (but as far as
>     I know, the binaries are less up-to-date compared the source packages).
> ------------------------------------ >8 ------------------------------------

sounds way better than 'Fink is based on Debian.'

best regards

Andreas

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 18:59   ` Bernhard Walle
  2012-04-09 19:47     ` Juergen Beisert
  2012-04-09 20:04     ` Andreas Bießmann
@ 2012-04-09 20:48     ` Andreas Bießmann
  2012-04-10  9:04       ` Bernhard Walle
  2 siblings, 1 reply; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-09 20:48 UTC (permalink / raw)
  To: ptxdist; +Cc: Bernhard Walle

Hi,

On 09.04.12 20:59, Bernhard Walle wrote:
> Hi,
> 
> * Andreas Bießmann <andreas@biessmann.de> [2012-04-09 20:01]:

>> b) FSF GCC is _not_ required for GNU Toolchain bootstrap, I did compile
>> my arm-cortexa8, arm-v4t, arm-v5te, x86_64 and avr32 OSELAS.Toolchain with:
> 
> This experience is from the time when I used Xcode 4.2 (I had to wait
> for the update until MacPorts got support for it). I got llvm-gcc
> segmentation faults and this was also "documented" in the bug tracker of
> FSF GCC. And I think that in LLVM bugtracker was a fix already in SVN of
> LLVM GCC.
> 
> I'm just running a build of a toolchain with Xcode 4.3(.2) LLVM GCC and
> if that build succeeds then I will remove the restriction to FSF GCC and
> mention Xcode 4.3 somewhere in the requirements.
> 
> bwalle@euler % gcc --version
> i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.9.00)

well my '(LLVM build 2336.1.00)' used to work. This f*** App Store does
not automagically update X Code as it used to in prior versions of Mac OS X.
Cause I have not encountered llvm-gcc segfaults in my X Code 4.2.1
(build 4D502), I think it is a specific problem to your version. Maybe
mention your version and/or the specific error in your doc?
BTW: Thank you for the pointer to X Code 4.3.2, download is running.
Hopefully no faulty toolchain ...

>> (But maybe there are some Fink stuff leaking in and let it build on my
>> system? Did not investigate it further ...)
> 
> I don't think that some Fink environment settings fixes GCC segmentation faults.

me too.

best regards

Andreas

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 20:48     ` Andreas Bießmann
@ 2012-04-10  9:04       ` Bernhard Walle
  2012-04-10 18:35         ` Bernhard Walle
  0 siblings, 1 reply; 37+ messages in thread
From: Bernhard Walle @ 2012-04-10  9:04 UTC (permalink / raw)
  To: Andreas Bießmann; +Cc: ptxdist

* Andreas Bießmann <andreas@biessmann.de> [2012-04-09 22:48]:
> > 
> > I'm just running a build of a toolchain with Xcode 4.3(.2) LLVM GCC and
> > if that build succeeds then I will remove the restriction to FSF GCC and
> > mention Xcode 4.3 somewhere in the requirements.
> > 
> > bwalle@euler % gcc --version
> > i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.9.00)
> 
> well my '(LLVM build 2336.1.00)' used to work. This f*** App Store does
> not automagically update X Code as it used to in prior versions of Mac OS X.
> Cause I have not encountered llvm-gcc segfaults in my X Code 4.2.1
> (build 4D502), I think it is a specific problem to your version. Maybe
> mention your version and/or the specific error in your doc?
> BTW: Thank you for the pointer to X Code 4.3.2, download is running.
> Hopefully no faulty toolchain ...

With Xcode 4.3.2 compiler it works. I checked the armv5 toolchain and
cortexa8 which uses gcc-linaro.

So I'll remove the section about gcc-apple, but I need to check first if
Qemu also works with llvm-gcc or maybe clang.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-09 17:48 ` Juergen Beisert
@ 2012-04-10  9:37   ` Bernhard Walle
  0 siblings, 0 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-10  9:37 UTC (permalink / raw)
  To: ptxdist

Hi,

* Juergen Beisert <jbe@pengutronix.de> [2012-04-09 19:48]:
> 
> > Maybe it can be linked somewhere in the official documentation or at the
> > website?
> 
> If you promise to keep the document at your website at the given URL, I would 
> add it to the Guru manual. 

I promise.

> You mentioned your 'https://bitbucket.org/bwalle/ptxdist-arm-boards' BSP in 
> you document. I would like to add it to our list of free BSPs 
> at "http://www.pengutronix.de/oselas/bsp/pengutronix/index_en.html" if you 
> send me some information about its features (to give the reader the same 
> overview like with our free BSPs).

On my TODO list, but I need to do some improvements on the BSP before it
can be "advertised".

> > If something is wrong or missing, drop me a note.
> 
> Just a few things that confuse me when I read it the first time. I will send 
> you a patch (offlist) with my suggestions.

Thanks for the review.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-10  9:04       ` Bernhard Walle
@ 2012-04-10 18:35         ` Bernhard Walle
  2012-04-17 11:08           ` Daniel Kriesten
  0 siblings, 1 reply; 37+ messages in thread
From: Bernhard Walle @ 2012-04-10 18:35 UTC (permalink / raw)
  To: ptxdist

* Bernhard Walle <bernhard@bwalle.de> [2012-04-10 11:04]:
> 
> With Xcode 4.3.2 compiler it works. I checked the armv5 toolchain and
> cortexa8 which uses gcc-linaro.
> 
> So I'll remove the section about gcc-apple, but I need to check first if
> Qemu also works with llvm-gcc or maybe clang.

I think all comments are integrated now. I uploaded a new PDF version
at http://www.bwalle.de/docs/ptxdist_mac.pdf. Thanks to Jürgen and
Andreas for the feedback. The Changelog can be viewed with GIT
at https://bitbucket.org/bwalle/ptxdist-macos-doc, the PDFs are tagged.

Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-10 18:35         ` Bernhard Walle
@ 2012-04-17 11:08           ` Daniel Kriesten
  2012-04-17 11:24             ` Michael Olbrich
  2012-04-17 11:48             ` Bernhard Walle
  0 siblings, 2 replies; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-17 11:08 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1.1: Type: text/plain, Size: 2005 bytes --]

Dear Bernhard,

First of all thanks for the great work on porting ptxdist to mac! Since 2010 I tried using it from time to time on my mac, but alway went back to our Linux compute servers for one reason or an other.

I'm using a "quiet modified" version of 10.6 (python27, ncurses, coreutils, git, and several others form macports). So my system is probably not a reference system at all. But 

So, following your your document I end up with a working tool-chain, a working qemu-arm-softmmu and a working kernel, but no hdd img.
I had to fix 
.../lib/ptxdist-2012.04.0/scripts/genhdimg
replacing seq by gseq. But the better approach may be a symlink in .../lib/ptxdist-2012.04.0/bin/

The next point is concerning the files system on a Mac. AFAIR the default filesystem is the case insensitive version of HFS+ (this is definitely true up to 10.6). So the OSELAS tool-chain rejects to build. And further on some of the packages will also fail. Obvious, there are two solutions. Using an external disk with a case sensitive file system (e.g. HFSX, ext*) ore using a (sparse) image as a container. I decided to use the latter approach.

The last point I tumbled over is the git command on page 10:
% git am git clone git://git.qemu.org/qemu.git
Whats the purpose of this command? Is it really needed?

An other thing is xargs ... without the gnu version make distclean fails. The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.

Finally I added two patches to reflect these changes. May be the second patch (Makefile.in) still needs some work to be in its final state. I am not sure, in which make targets the 

Am 10.04.2012 um 20:35 schrieb Bernhard Walle:

> I think all comments are integrated now. I uploaded a new PDF version
> at http://www.bwalle.de/docs/ptxdist_mac.pdf. Thanks to Jürgen and
> Andreas for the feedback. The Changelog can be viewed with GIT
> at https://bitbucket.org/bwalle/ptxdist-macos-doc, the PDFs are tagged.
> 

Regards
	Daniel


[-- Attachment #1.1.2: 0001-check-for-GNU-seq-and-GNU-xargs.patch --]
[-- Type: application/octet-stream, Size: 856 bytes --]

From 895b6a946aa928aa62b2a328dab3fe55cf0b99a3 Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Tue, 17 Apr 2012 11:32:23 +0200
Subject: [PATCH 1/2] check for GNU seq and GNU xargs

On Mac OS X 10.6 seq is not available by default. And MacPorts creates
no symlink to /opt/local/bin.
BSD xargs does not accept the '-r' option.

Signed-off-by: Daniel Kriesten <krid@tu-chemnitz.eu>
---
 configure.ac |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/configure.ac b/configure.ac
index 44c2448..d83059a 100755
--- a/configure.ac
+++ b/configure.ac
@@ -147,6 +147,8 @@ GNU_TOOL(install, coreutils)
 GNU_TOOL(stat, coreutils)
 GNU_TOOL(mknod, coreutils)
 GNU_TOOL(readlink, coreutils)
+GNU_TOOL(seq, coreutils)
+GNU_TOOL(xargs, findutils)
 GNU_TOOL(tar, tar)
 GNU_TOOL(find, findutils)
 
-- 
1.7.10


[-- Attachment #1.1.3: 0002-Use-tools-as-identified-by-configure.patch --]
[-- Type: application/octet-stream, Size: 1465 bytes --]

From 53c92e68491e4405193e240e69b3b1dff0405dd2 Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Tue, 17 Apr 2012 11:44:45 +0200
Subject: [PATCH 2/2] Use tools as identified by configure

The configure script checks for several (GNU) tools to exist and saves
their PATH. Thus the Makefile should use the saved PATH as well.

Signed-off-by: Daniel Kriesten <krid@tu-chemnitz.eu>
---
 Makefile.in |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/Makefile.in b/Makefile.in
index 8382402..02a9e5f 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -62,8 +62,8 @@ environment:
 	@echo " done"
 
 clean:
-	@rm -f .done
-	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+	@"$(abs_srcdir)/bin/rm" -f .done
+	@"$(abs_srcdir)/bin/find" "$(abs_srcdir)/bin" -type l -print0 | "$(abs_srcdir)/bin/xargs" -r -0 rm
 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
 
 dirty-check:
@@ -113,12 +113,12 @@ dist: dirty-check
 	md5sum "${project}.tar.bz2" > "${project}.tar.bz2.md5"
 
 distclean: clean
-	rm -fr Makefile
-	rm -fr build-stamp
-	rm -fr config.log config.status autom4te.cache
+	"$(abs_srcdir)/bin/rm" -fr Makefile
+	"$(abs_srcdir)/bin/rm" -fr build-stamp
+	"$(abs_srcdir)/bin/rm" -fr config.log config.status autom4te.cache
 
 maintainer-clean: distclean
-	rm -f configure
+	"$(abs_srcdir)/bin/rm" -f configure
 
 release:
 	@scripts/make_$@.sh
-- 
1.7.10


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:08           ` Daniel Kriesten
@ 2012-04-17 11:24             ` Michael Olbrich
  2012-04-17 11:45               ` Bernhard Walle
  2012-04-17 11:48             ` Bernhard Walle
  1 sibling, 1 reply; 37+ messages in thread
From: Michael Olbrich @ 2012-04-17 11:24 UTC (permalink / raw)
  To: ptxdist

Hi,

On Tue, Apr 17, 2012 at 01:08:12PM +0200, Daniel Kriesten wrote:
[...]
> So, following your your document I end up with a working tool-chain, a
> working qemu-arm-softmmu and a working kernel, but no hdd img.
> I had to fix 
> .../lib/ptxdist-2012.04.0/scripts/genhdimg
> replacing seq by gseq. But the better approach may be a symlink in
> .../lib/ptxdist-2012.04.0/bin/

Well, genhdimg is the only user of 'seq', so I'd rather fix that. Would you
care to send a patch?

[...]
> An other thing is xargs ... without the gnu version make distclean fails.
> The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.

It's "make clean" actually and that will remove the relevant symlink.
Would this work:
------------------------------------------------------------------
diff --git a/Makefile.in b/Makefile.in
index 8382402..32784c8 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -63,7 +63,7 @@ environment:
 
 clean:
 	@rm -f .done
-	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
 
 dirty-check:
------------------------------------------------------------------

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:24             ` Michael Olbrich
@ 2012-04-17 11:45               ` Bernhard Walle
  2012-04-17 11:59                 ` Daniel Kriesten
  2012-04-18  8:29                 ` Michael Olbrich
  0 siblings, 2 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-17 11:45 UTC (permalink / raw)
  To: ptxdist

Hi,

* Michael Olbrich <m.olbrich@pengutronix.de> [2012-04-17 13:24]:
> On Tue, Apr 17, 2012 at 01:08:12PM +0200, Daniel Kriesten wrote:
> [...]
> > So, following your your document I end up with a working tool-chain, a
> > working qemu-arm-softmmu and a working kernel, but no hdd img.
> > I had to fix 
> > .../lib/ptxdist-2012.04.0/scripts/genhdimg
> > replacing seq by gseq. But the better approach may be a symlink in
> > .../lib/ptxdist-2012.04.0/bin/
> 
> Well, genhdimg is the only user of 'seq', so I'd rather fix that. Would you
> care to send a patch?

But gseq is not available on Linux. So I think adding it to the
configure check is the only reasonable approach?

> [...]
> > An other thing is xargs ... without the gnu version make distclean fails.
> > The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
> 
> It's "make clean" actually and that will remove the relevant symlink.
> Would this work:

That one is already in ptxdist, after 2012.04:

http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7

> ------------------------------------------------------------------
> diff --git a/Makefile.in b/Makefile.in
> index 8382402..32784c8 100644
> --- a/Makefile.in
> +++ b/Makefile.in
> @@ -63,7 +63,7 @@ environment:
>  
>  clean:
>  	@rm -f .done
> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
>  	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean

Maybe just -exec ? Maybe it is a bit slower? But see above.



Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:08           ` Daniel Kriesten
  2012-04-17 11:24             ` Michael Olbrich
@ 2012-04-17 11:48             ` Bernhard Walle
  2012-04-17 12:04               ` Daniel Kriesten
  1 sibling, 1 reply; 37+ messages in thread
From: Bernhard Walle @ 2012-04-17 11:48 UTC (permalink / raw)
  To: ptxdist

Hi Daniel,

* Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-17 13:08]:

> The next point is concerning the files system on a Mac. AFAIR the
> default filesystem is the case insensitive version of HFS+ (this is
> definitely true up to 10.6). So the OSELAS tool-chain rejects to
> build. And further on some of the packages will also fail. Obvious,
> there are two solutions. Using an external disk with a case sensitive
> file system (e.g. HFSX, ext*) ore using a (sparse) image as a
> container. I decided to use the latter approach.

Indeed, I missed that in the documentation. I'll add it soon. I
personally use the 3rd approach: A data partition on the internal disk
with a case insensitive file system.

I'll send you a personal mail for proofreading if that is okay.

> The last point I tumbled over is the git command on page 10:
> % git am git clone git://git.qemu.org/qemu.git
> Whats the purpose of this command? Is it really needed?

You only need that if you want to use Qemu to start the image.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:45               ` Bernhard Walle
@ 2012-04-17 11:59                 ` Daniel Kriesten
  2012-04-17 12:03                   ` Bernhard Walle
  2012-04-18  8:29                 ` Michael Olbrich
  1 sibling, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-17 11:59 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 2363 bytes --]

Hi,

Am 17.04.2012 um 13:45 schrieb Bernhard Walle:

> Hi,
> 
> * Michael Olbrich <m.olbrich@pengutronix.de> [2012-04-17 13:24]:
>> On Tue, Apr 17, 2012 at 01:08:12PM +0200, Daniel Kriesten wrote:
>> [...]
>>> So, following your your document I end up with a working tool-chain, a
>>> working qemu-arm-softmmu and a working kernel, but no hdd img.
>>> I had to fix 
>>> .../lib/ptxdist-2012.04.0/scripts/genhdimg
>>> replacing seq by gseq. But the better approach may be a symlink in
>>> .../lib/ptxdist-2012.04.0/bin/
>> 
>> Well, genhdimg is the only user of 'seq', so I'd rather fix that. Would you
>> care to send a patch?
> 
> But gseq is not available on Linux. So I think adding it to the
> configure check is the only reasonable approach?

Well, whats the advantage of $(seq 4) over "for i in 1 2 3 4; do ..." ?

Alternatively, it is possible in just a few lines of code to check for seq or gseq within genhdimg.
SEQ=$(which seq 2>/dev/null ||which gseq 2>/dev/null)
if [[ -z $SEQ ]]; then
some error
exit 1
fi

> 
>> [...]
>>> An other thing is xargs ... without the gnu version make distclean fails.
>>> The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
>> 
>> It's "make clean" actually and that will remove the relevant symlink.
>> Would this work:
> 
> That one is already in ptxdist, after 2012.04:
> 
> http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7
> 
>> ------------------------------------------------------------------
>> diff --git a/Makefile.in b/Makefile.in
>> index 8382402..32784c8 100644
>> --- a/Makefile.in
>> +++ b/Makefile.in
>> @@ -63,7 +63,7 @@ environment:
>> 
>> clean:
>> 	@rm -f .done
>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>> +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
> 
> Maybe just -exec ? Maybe it is a bit slower? But see above.
> 
> 
> 
> Regards,
> Bernhard
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

Mit freundlichen Grüßen
	Daniel Kriesten

-- 
Daniel Kriesten
Professur Schaltkreis- und Systementwurf
Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
Tel: +49 371 531 33058  Fax: +49 371 531 833058
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:59                 ` Daniel Kriesten
@ 2012-04-17 12:03                   ` Bernhard Walle
  2012-04-17 12:32                     ` Daniel Kriesten
  0 siblings, 1 reply; 37+ messages in thread
From: Bernhard Walle @ 2012-04-17 12:03 UTC (permalink / raw)
  To: ptxdist

* Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-17 13:59]:
> >> care to send a patch?
> > 
> > But gseq is not available on Linux. So I think adding it to the
> > configure check is the only reasonable approach?
> 
> Well, whats the advantage of $(seq 4) over "for i in 1 2 3 4; do ..." ?

I didn't look in the code, sorry. Of course that's okay for me. ;)


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:48             ` Bernhard Walle
@ 2012-04-17 12:04               ` Daniel Kriesten
  2012-04-17 12:18                 ` Bernhard Walle
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-17 12:04 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 1683 bytes --]


Am 17.04.2012 um 13:48 schrieb Bernhard Walle:

> Hi Daniel,
> 
> * Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-17 13:08]:
> 
>> The next point is concerning the files system on a Mac. AFAIR the
>> default filesystem is the case insensitive version of HFS+ (this is
>> definitely true up to 10.6). So the OSELAS tool-chain rejects to
>> build. And further on some of the packages will also fail. Obvious,
>> there are two solutions. Using an external disk with a case sensitive
>> file system (e.g. HFSX, ext*) ore using a (sparse) image as a
>> container. I decided to use the latter approach.
> 
> Indeed, I missed that in the documentation. I'll add it soon. I
> personally use the 3rd approach: A data partition on the internal disk
> with a case insensitive file system.
> 
> I'll send you a personal mail for proofreading if that is okay.

I would be pleased to do so.

> 
>> The last point I tumbled over is the git command on page 10:
>> % git am git clone git://git.qemu.org/qemu.git
>> Whats the purpose of this command? Is it really needed?
> 
> You only need that if you want to use Qemu to start the image.

For this reason I used the second line, starting with patch ...
But the first line is either syntactically not correct or not needed? If I am right, "git am" requires a patchset?

<---- snip ---->
 \begin{lstlisting}[style=inline]
% git am git clone git://git.qemu.org/qemu.git
% patch -p0 -i \
   <path-to-bsp>/configs/arm-qemu-2011.11.0/patches ...
    .../patch-cocoa-uint16-redefined.diff
\end{lstlisting}
<---- snap ---->

Regards,
	Daniel
-- 
Daniel Kriesten
http://www.die-kriestens.de


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 12:04               ` Daniel Kriesten
@ 2012-04-17 12:18                 ` Bernhard Walle
  0 siblings, 0 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-17 12:18 UTC (permalink / raw)
  To: ptxdist

* Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-17 14:04]:
> 
> For this reason I used the second line, starting with patch ...
> But the first line is either syntactically not correct or not needed? If I am right, "git am" requires a patchset?
> 
> <---- snip ---->
>  \begin{lstlisting}[style=inline]
> % git am git clone git://git.qemu.org/qemu.git
> % patch -p0 -i \
>    <path-to-bsp>/configs/arm-qemu-2011.11.0/patches ...
>     .../patch-cocoa-uint16-redefined.diff
> \end{lstlisting}
> <---- snap ---->

Now I understand the problem, thanks. Will be fixed. Yes, git am
doesn't work in that case.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 12:03                   ` Bernhard Walle
@ 2012-04-17 12:32                     ` Daniel Kriesten
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-17 12:32 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1.1: Type: text/plain, Size: 668 bytes --]

Dear all,

find attached the patch to avoid the use of seq in the genhdimg script.

Am 17.04.2012 um 14:03 schrieb Bernhard Walle:

> * Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-17 13:59]:
>>>> care to send a patch?
>>> 
>>> But gseq is not available on Linux. So I think adding it to the
>>> configure check is the only reasonable approach?
>> 
>> Well, whats the advantage of $(seq 4) over "for i in 1 2 3 4; do ..." ?
> 
> I didn't look in the code, sorry. Of course that's okay for me. ;)
> 
> 
> Regards,
> Bernhard
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

Regards,
	Daniel
-- 
Daniel Kriesten
http://www.die-kriestens.de

[-- Attachment #1.1.2: 0001-Replace-seq-by-the-appropriate-sequence.patch --]
[-- Type: application/octet-stream, Size: 1100 bytes --]

From dee07eb444f3291ee963704af6e95f473c1d4119 Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Tue, 17 Apr 2012 14:15:43 +0200
Subject: [PATCH] Replace seq by the appropriate sequence

The seq command is not availabale on all systems. By replacing the (at
least two) occurences of seq with the corresponding sequence the
dependecy on seq is resolved.

Signed-off-by: Daniel Kriesten <krid@tu-chemnitz.eu>
---
 scripts/genhdimg |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/genhdimg b/scripts/genhdimg
index d3143ac..4c4bc2e 100755
--- a/scripts/genhdimg
+++ b/scripts/genhdimg
@@ -79,7 +79,7 @@ echo "------------------------------------"
 echo "Creating hard disk's partition table"
 echo "------------------------------------"
 
-for i in $(seq 4); do
+for i in 1 2 3 4; do
 	part_opts "$i"
 
 	if [ $i -gt $partcount ]; then
@@ -100,7 +100,7 @@ for i in $(seq 4); do
 	extraarg=""
 done
 
-for i in $(seq 4); do
+for i in 1 2 3 4; do
 	part_opts "$i"
 
 	if [ -z "${pimage}" ]; then
-- 
1.7.10


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-17 11:45               ` Bernhard Walle
  2012-04-17 11:59                 ` Daniel Kriesten
@ 2012-04-18  8:29                 ` Michael Olbrich
  2012-04-18 11:25                   ` Daniel Kriesten
  1 sibling, 1 reply; 37+ messages in thread
From: Michael Olbrich @ 2012-04-18  8:29 UTC (permalink / raw)
  To: ptxdist

On Tue, Apr 17, 2012 at 01:45:04PM +0200, Bernhard Walle wrote:
> Hi,
> 
> * Michael Olbrich <m.olbrich@pengutronix.de> [2012-04-17 13:24]:
> > On Tue, Apr 17, 2012 at 01:08:12PM +0200, Daniel Kriesten wrote:
> > [...]
> > > So, following your your document I end up with a working tool-chain, a
> > > working qemu-arm-softmmu and a working kernel, but no hdd img.
> > > I had to fix 
> > > .../lib/ptxdist-2012.04.0/scripts/genhdimg
> > > replacing seq by gseq. But the better approach may be a symlink in
> > > .../lib/ptxdist-2012.04.0/bin/
> > 
> > Well, genhdimg is the only user of 'seq', so I'd rather fix that. Would you
> > care to send a patch?
> 
> But gseq is not available on Linux. So I think adding it to the
> configure check is the only reasonable approach?
> 
> > [...]
> > > An other thing is xargs ... without the gnu version make distclean fails.
> > > The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
> > 
> > It's "make clean" actually and that will remove the relevant symlink.
> > Would this work:
> 
> That one is already in ptxdist, after 2012.04:
> 
> http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7

Yes it's there, and calling "make clean" twice still breaks.

> > ------------------------------------------------------------------
> > diff --git a/Makefile.in b/Makefile.in
> > index 8382402..32784c8 100644
> > --- a/Makefile.in
> > +++ b/Makefile.in
> > @@ -63,7 +63,7 @@ environment:
> >  
> >  clean:
> >  	@rm -f .done
> > -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> > +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
> >  	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
> 
> Maybe just -exec ? Maybe it is a bit slower? But see above.

I really don't care either way.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-18  8:29                 ` Michael Olbrich
@ 2012-04-18 11:25                   ` Daniel Kriesten
  2012-04-18 14:16                     ` Michael Olbrich
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-18 11:25 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1.1: Type: text/plain, Size: 1375 bytes --]

Hi,

Am 18.04.2012 um 10:29 schrieb Michael Olbrich:
>> [...]
>>> [...]
>>>> An other thing is xargs ... without the gnu version make distclean fails.
>>>> The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
>>> 
>>> It's "make clean" actually and that will remove the relevant symlink.
>>> Would this work:
>> 
>> That one is already in ptxdist, after 2012.04:
>> 
>> http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7
> 
> Yes it's there, and calling "make clean" twice still breaks.
> 
>>> ------------------------------------------------------------------
>>> diff --git a/Makefile.in b/Makefile.in
>>> index 8382402..32784c8 100644
>>> --- a/Makefile.in
>>> +++ b/Makefile.in
>>> @@ -63,7 +63,7 @@ environment:
>>> 
>>> clean:
>>> 	@rm -f .done
>>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>>> +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
>>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>> 
>> Maybe just -exec ? Maybe it is a bit slower? But see above.
> 
> I really don't care either way.

From my point of view removing -r and adding -f is a simple, but sufficient solution. I added the corresponding patch, along with a patch fixing a typo in configure.ac.

Regards,
	Daniel
-- 
Daniel Kriesten
http://www.die-kriestens.de

[-- Attachment #1.1.2: 0001-fix-typo-in-the-xargs-check.patch --]
[-- Type: application/octet-stream, Size: 868 bytes --]

From 1a40577500278fab95562e449f5e597e77d76f7d Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Wed, 18 Apr 2012 11:46:53 +0200
Subject: [PATCH 1/2] fix typo in the xargs check

Signed-off-by: Daniel Kriesten <krid@tu-chemnitz.eu>
---
 configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index 44c2448..fbc6216 100755
--- a/configure.ac
+++ b/configure.ac
@@ -292,7 +292,7 @@ fi
 AC_MSG_CHECKING([if xargs -r works as expected])
 echo | ${XARGS} -r false
 if test $? -ne 0; then
-	AC_MSG_ERROR([xargs with "-r" support needed, please install GNU xargs (unsually located in the "findutils" package])
+	AC_MSG_ERROR([xargs with "-r" support needed, please install GNU xargs (usually located in the "findutils" package])
 fi
 AC_MSG_RESULT([ok])
 
-- 
1.7.10


[-- Attachment #1.1.3: 0002-Remove-r-from-call-to-xargs.patch --]
[-- Type: application/octet-stream, Size: 838 bytes --]

From 78bd4616dc8d05e6e26d9e4544478bd44262010a Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Wed, 18 Apr 2012 11:47:35 +0200
Subject: [PATCH 2/2] Remove -r from call to xargs

The '-r' flag is a GNU extension. To enhance compatibility remove '-r'
and suppress error mesages using 'rm -f'.

Signed-off-by: Daniel Kriesten <krid@tu-chemnitz.eu>
---
 Makefile.in |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.in b/Makefile.in
index 8382402..1d7678e 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -63,7 +63,7 @@ environment:
 
 clean:
 	@rm -f .done
-	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm -f
 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
 
 dirty-check:
-- 
1.7.10


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-18 11:25                   ` Daniel Kriesten
@ 2012-04-18 14:16                     ` Michael Olbrich
  2012-04-18 14:28                       ` Daniel Kriesten
  0 siblings, 1 reply; 37+ messages in thread
From: Michael Olbrich @ 2012-04-18 14:16 UTC (permalink / raw)
  To: ptxdist

Hi,

On Wed, Apr 18, 2012 at 01:25:07PM +0200, Daniel Kriesten wrote:
> Am 18.04.2012 um 10:29 schrieb Michael Olbrich:
> >> [...]
> >>> [...]
> >>>> An other thing is xargs ... without the gnu version make distclean fails.
> >>>> The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
> >>> 
> >>> It's "make clean" actually and that will remove the relevant symlink.
> >>> Would this work:
> >> 
> >> That one is already in ptxdist, after 2012.04:
> >> 
> >> http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7
> > 
> > Yes it's there, and calling "make clean" twice still breaks.
> > 
> >>> ------------------------------------------------------------------
> >>> diff --git a/Makefile.in b/Makefile.in
> >>> index 8382402..32784c8 100644
> >>> --- a/Makefile.in
> >>> +++ b/Makefile.in
> >>> @@ -63,7 +63,7 @@ environment:
> >>> 
> >>> clean:
> >>> 	@rm -f .done
> >>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> >>> +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
> >>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
> >> 
> >> Maybe just -exec ? Maybe it is a bit slower? But see above.
> > 
> > I really don't care either way.
> 
> From my point of view removing -r and adding -f is a simple, but sufficient
> solution. I added the corresponding patch, along with a patch fixing a typo
> in configure.ac.

So "find -print0" works, right? I wasn't sure if that's a GNU extension.
The xargs man-page I have says "... The GNU find -print0 option produces
input suitable for this mode.". So I wasn't sure if this works _only_ with
GNU find.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-18 14:16                     ` Michael Olbrich
@ 2012-04-18 14:28                       ` Daniel Kriesten
  2012-04-19 10:41                         ` Daniel Kriesten
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-18 14:28 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 2039 bytes --]


Am 18.04.2012 um 16:16 schrieb Michael Olbrich:

> On Wed, Apr 18, 2012 at 01:25:07PM +0200, Daniel Kriesten wrote:
>> Am 18.04.2012 um 10:29 schrieb Michael Olbrich:
>>>> [...]
>>>>> [...]
>>>>>> An other thing is xargs ... without the gnu version make distclean fails.
>>>>>> The solution is to symlink in .../lib/ptxdist-2012.04.0/bin/, too.
>>>>> 
>>>>> It's "make clean" actually and that will remove the relevant symlink.
>>>>> Would this work:
>>>> 
>>>> That one is already in ptxdist, after 2012.04:
>>>> 
>>>> http://git.pengutronix.de/?p=ptxdist.git;a=commit;h=394af1de808e6fc4c3cf921d36e1b1f887133eb7
>>> 
>>> Yes it's there, and calling "make clean" twice still breaks.
>>> 
>>>>> ------------------------------------------------------------------
>>>>> diff --git a/Makefile.in b/Makefile.in
>>>>> index 8382402..32784c8 100644
>>>>> --- a/Makefile.in
>>>>> +++ b/Makefile.in
>>>>> @@ -63,7 +63,7 @@ environment:
>>>>> 
>>>>> clean:
>>>>> 	@rm -f .done
>>>>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>>>>> +	@cd "$(abs_srcdir)/bin" && find -type l -print | xargs rm -f
>>>>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>>>> 
>>>> Maybe just -exec ? Maybe it is a bit slower? But see above.
>>> 
>>> I really don't care either way.
>> 
>> From my point of view removing -r and adding -f is a simple, but sufficient
>> solution. I added the corresponding patch, along with a patch fixing a typo
>> in configure.ac.
> 
> So "find -print0" works, right? I wasn't sure if that's a GNU extension.
> The xargs man-page I have says "... The GNU find -print0 option produces
> input suitable for this mode.". So I wasn't sure if this works _only_ with
> GNU find.

Uh - I'll check that.

Mit freundlichen Grüßen
	Daniel Kriesten

-- 
Daniel Kriesten
Professur Schaltkreis- und Systementwurf
Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
Tel: +49 371 531 33058  Fax: +49 371 531 833058
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-18 14:28                       ` Daniel Kriesten
@ 2012-04-19 10:41                         ` Daniel Kriesten
  2012-04-19 11:29                           ` Andreas Bießmann
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-19 10:41 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 1310 bytes --]

Hi Michael,
at leas on a Mac (OS X 10.6.8) the BSD find supports -print0

[me@mymac:ptxdist:master u=] $ ls bin/
awk@      chown@    find@     md5sum@   mknod@    ptxdist*  readlink@ rmdir@    stat@     xargs@
chmod@    cp@       install@  mkdir@    mv@       python@   rm@       sed@      tar@
[me@mymac:ptxdist:master u=] $ which find
/usr/bin/find
[me@mymac:ptxdist:master u=] $ find bin -type l -print0 | xargs -0 rm -vf
bin/awk
bin/chmod
bin/chown
bin/cp
bin/find
bin/install
bin/md5sum
bin/mkdir
bin/mknod
bin/mv
bin/python
bin/readlink
bin/rm
bin/rmdir
bin/sed
bin/stat
bin/tar
bin/xargs
[me@mymac:ptxdist:master u=] $ ls bin
ptxdist*

Am 18.04.2012 um 16:28 schrieb Daniel Kriesten:
> Am 18.04.2012 um 16:16 schrieb Michael Olbrich:
> [...]
>> So "find -print0" works, right? I wasn't sure if that's a GNU extension.
>> The xargs man-page I have says "... The GNU find -print0 option produces
>> input suitable for this mode.". So I wasn't sure if this works _only_ with
>> GNU find.
> 
> Uh - I'll check that.



Regards,
	Daniel
-- 
Daniel Kriesten
Chair for Circuit and System Design
Chemnitz University of Technology, Reichenhainer Str. 31-33, D-09126 Chemnitz
Phone: +49 371 531-33158, Fax: +49 371 531-833158
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 10:41                         ` Daniel Kriesten
@ 2012-04-19 11:29                           ` Andreas Bießmann
  2012-04-19 12:04                             ` Daniel Kriesten
  0 siblings, 1 reply; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-19 11:29 UTC (permalink / raw)
  To: ptxdist

Hi all,

On 19.04.2012 12:41, Daniel Kriesten wrote:
> Hi Michael,
> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0

yes, OS X delivered BSD find supports -print0, but it does _not_ support
omitting the path!

---8<---
andreas@andreas-mbp % /usr/bin/find -type l -print0
/usr/bin/find: illegal option -- t
/usr/bin/find: illegal option -- y
/usr/bin/find: illegal option -- p
/usr/bin/find: illegal option -- e
find: l: No such file or directory
andreas@andreas-mbp % /usr/bin/find . -type l -print0
./awk./chmod./chown./cp./install./md5sum./mkdir./mknod./mv./python./readlink./rm./rmdir./sed./stat./tar./xargs%

                          andreas@andreas-mbp %
--->8---

Therefore the provided patch from Michael will _not_ work with that find.

Beside that that was not the question. I think a portable solution would be:

---8<---
--- a/Makefile.in
+++ b/Makefile.in
@@ -63,7 +63,7 @@ environment:

 clean:
 	@rm -f .done
-	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+	@cd "$(abs_srcdir)/bin" && find . -type l -print | xargs rm -f
 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
--->8---

OR leaving the xargs -r param

---8<---
--- a/Makefile.in
+++ b/Makefile.in
@@ -63,7 +63,7 @@ environment:

 clean:
 	@rm -f .done
-	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
--->8---

best regards

Andreas Bießmann

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 11:29                           ` Andreas Bießmann
@ 2012-04-19 12:04                             ` Daniel Kriesten
  2012-04-19 12:25                               ` Andreas Bießmann
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-19 12:04 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 1074 bytes --]

Hi all,

Am 19.04.2012 um 13:29 schrieb Andreas Bießmann:

> Hi all,
> 
> On 19.04.2012 12:41, Daniel Kriesten wrote:
>> Hi Michael,
>> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0
> 
> yes, OS X delivered BSD find supports -print0, but it does _not_ support
> omitting the path!
> [...]
> OR leaving the xargs -r param
> 
> ---8<---
> --- a/Makefile.in
> +++ b/Makefile.in
> @@ -63,7 +63,7 @@ environment:
> 
> clean:
> 	@rm -f .done
> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> +	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
> --->8---

This one is my promoted solution in yesterdays patch, except I would recommend 'rm -f' to avoid errors if find returns an empty list.

Mit freundlichen Grüßen
	Daniel Kriesten

-- 
Daniel Kriesten
Professur Schaltkreis- und Systementwurf
Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
Tel: +49 371 531 33058  Fax: +49 371 531 833058
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 12:04                             ` Daniel Kriesten
@ 2012-04-19 12:25                               ` Andreas Bießmann
  2012-04-19 13:12                                 ` Daniel Kriesten
  2012-12-08 15:31                                 ` Daniel Kriesten
  0 siblings, 2 replies; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-19 12:25 UTC (permalink / raw)
  To: ptxdist

Hi,

On 19.04.2012 14:04, Daniel Kriesten wrote:
> Hi all,
> 
> Am 19.04.2012 um 13:29 schrieb Andreas Bießmann:
> 
>> Hi all,
>>
>> On 19.04.2012 12:41, Daniel Kriesten wrote:
>>> Hi Michael,
>>> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0
>>
>> yes, OS X delivered BSD find supports -print0, but it does _not_ support
>> omitting the path!
>> [...]
>> OR leaving the xargs -r param
>>
>> ---8<---
>> --- a/Makefile.in
>> +++ b/Makefile.in
>> @@ -63,7 +63,7 @@ environment:
>>
>> clean:
>> 	@rm -f .done
>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>> +	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>> --->8---
> 
> This one is my promoted solution in yesterdays patch, except I would recommend 'rm -f' to avoid errors if find returns an empty list.

well ok, +1 for your solution. I maybe missed that in my mail history
(can you please send patches inline next time?).

best regards

Andreas Bießmann

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 12:25                               ` Andreas Bießmann
@ 2012-04-19 13:12                                 ` Daniel Kriesten
  2012-04-19 13:27                                   ` Michael Olbrich
                                                     ` (2 more replies)
  2012-12-08 15:31                                 ` Daniel Kriesten
  1 sibling, 3 replies; 37+ messages in thread
From: Daniel Kriesten @ 2012-04-19 13:12 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 706 bytes --]

Hello Andreas,

Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:

> well ok, +1 for your solution. I maybe missed that in my mail history
> (can you please send patches inline next time?).

is this the preferred method?

As I'm using Mail.app (Apple Mail) I'm alway fearing broken patches when sending inline, as the program is set up to do automatic line breaks. And up to now I can get git send-email to play with our universities smtp server.


Regards,
	Daniel
-- 
Daniel Kriesten
Chair for Circuit and System Design
Chemnitz University of Technology, Reichenhainer Str. 31-33, D-09126 Chemnitz
Phone: +49 371 531-33158, Fax: +49 371 531-833158
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 13:12                                 ` Daniel Kriesten
@ 2012-04-19 13:27                                   ` Michael Olbrich
  2012-04-19 14:03                                   ` Andreas Bießmann
  2012-04-19 20:13                                   ` Bernhard Walle
  2 siblings, 0 replies; 37+ messages in thread
From: Michael Olbrich @ 2012-04-19 13:27 UTC (permalink / raw)
  To: ptxdist

Hi,

On Thu, Apr 19, 2012 at 03:12:25PM +0200, Daniel Kriesten wrote:
> Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:
> > well ok, +1 for your solution. I maybe missed that in my mail history
> > (can you please send patches inline next time?).
> 
> is this the preferred method?

Yes, it makes reading and commenting the patch much easier.

> As I'm using Mail.app (Apple Mail) I'm alway fearing broken patches when
> sending inline, as the program is set up to do automatic line breaks. And up
> to now I can get git send-email to play with our universities smtp server.

Most patches get at least one round of review before I really test them.
If it fails to apply, then I'll say something and you can attach the next
version.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 13:12                                 ` Daniel Kriesten
  2012-04-19 13:27                                   ` Michael Olbrich
@ 2012-04-19 14:03                                   ` Andreas Bießmann
  2012-04-19 20:13                                   ` Bernhard Walle
  2 siblings, 0 replies; 37+ messages in thread
From: Andreas Bießmann @ 2012-04-19 14:03 UTC (permalink / raw)
  To: ptxdist

Hi Daniel,

On 19.04.2012 15:12, Daniel Kriesten wrote:
> Hello Andreas,
> 
> Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:
> 
>> well ok, +1 for your solution. I maybe missed that in my mail history
>> (can you please send patches inline next time?).
> 
> is this the preferred method?

as Michael said it is much better to read.

> As I'm using Mail.app (Apple Mail) I'm alway fearing broken patches when sending inline, as the program is set up to do automatic line breaks. And up to now I can get git send-email to play with our universities smtp server.

(line break also broken, fix your MUA ;)

You could use a wrapper for git send-email. Some use msmtp to work
around broken/insufficient smtp support in git. There are some git +
msmtp + gmail howto's in the web, maybe that can solve your problem with
your servers?

best regards

Andreas Bießmann

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 13:12                                 ` Daniel Kriesten
  2012-04-19 13:27                                   ` Michael Olbrich
  2012-04-19 14:03                                   ` Andreas Bießmann
@ 2012-04-19 20:13                                   ` Bernhard Walle
  2 siblings, 0 replies; 37+ messages in thread
From: Bernhard Walle @ 2012-04-19 20:13 UTC (permalink / raw)
  To: ptxdist

* Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de> [2012-04-19 15:12]:
> 
> As I'm using Mail.app (Apple Mail) I'm alway fearing broken patches
> when sending inline, as the program is set up to do automatic line
> breaks. And up to now I can get git send-email to play with our
> universities smtp server.

I would also not recommend using Apple Mail for sending patches. Even
for Thunderbird (which can be configured to do everything right) it's a
mess to do so.

If you don't want git send-email directly, you could consider using git
imap-send. This one puts the formatted mails in the drafts folder of
your IMAP server and you can actually review and send the mails using
Apple Mail. However, I would guess that this produces better results
than just using copy and paste.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-04-19 12:25                               ` Andreas Bießmann
  2012-04-19 13:12                                 ` Daniel Kriesten
@ 2012-12-08 15:31                                 ` Daniel Kriesten
  2012-12-08 15:43                                   ` [ptxdist] Using PTXdist on Mac OS 10.7 Daniel Kriesten
  2012-12-10  8:46                                   ` [ptxdist] Using PTXdist on Mac OS Michael Olbrich
  1 sibling, 2 replies; 37+ messages in thread
From: Daniel Kriesten @ 2012-12-08 15:31 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 2383 bytes --]

Hi all,

this is quite a late reply to this mail. But today I struggled about this once again. So here its the patch.

---8<---

From dd717be19e704b71a030b091d2604d7a05ccebb2 Mon Sep 17 00:00:00 2001
From: Daniel Kriesten <krid@tu-chemnitz.eu>
Date: Sat, 8 Dec 2012 16:27:35 +0100
Subject: [PATCH] Remove -r from call to xargs

The '-r' flag is a GNU extension. To enhance compatibility remove '-r'
and suppress error mesages using 'rm -f'.
---
 Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.in b/Makefile.in
index e171045..72509a2 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -71,7 +71,7 @@ environment:
 
 clean:
        @rm -f .done
-       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
+       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm -f
        @$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
 
 dirty-check:
-- 
1.8.0

--->8---

Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:
> Hi,
> 
> On 19.04.2012 14:04, Daniel Kriesten wrote:
>> Hi all,
>> 
>> Am 19.04.2012 um 13:29 schrieb Andreas Bießmann:
>> 
>>> Hi all,
>>> 
>>> On 19.04.2012 12:41, Daniel Kriesten wrote:
>>>> Hi Michael,
>>>> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0
>>> 
>>> yes, OS X delivered BSD find supports -print0, but it does _not_ support
>>> omitting the path!
>>> [...]
>>> OR leaving the xargs -r param
>>> 
>>> ---8<---
>>> --- a/Makefile.in
>>> +++ b/Makefile.in
>>> @@ -63,7 +63,7 @@ environment:
>>> 
>>> clean:
>>> 	@rm -f .done
>>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>>> +	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
>>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>>> --->8---
>> 
>> This one is my promoted solution in yesterdays patch, except I would recommend 'rm -f' to avoid errors if find returns an empty list.
> 
> well ok, +1 for your solution. I maybe missed that in my mail history
> (can you please send patches inline next time?).
> 
> best regards
> 
> Andreas Bießmann
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 


-- 
Daniel Kriesten
Professur Schaltkreis- und Systementwurf
Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
Tel: +49 371 531 33058  Fax: +49 371 531 833058
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS 10.7
  2012-12-08 15:31                                 ` Daniel Kriesten
@ 2012-12-08 15:43                                   ` Daniel Kriesten
  2012-12-10  8:48                                     ` Michael Olbrich
  2012-12-10  8:46                                   ` [ptxdist] Using PTXdist on Mac OS Michael Olbrich
  1 sibling, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-12-08 15:43 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 1945 bytes --]

Dear all,

So today I did a fresh build of ptxdist on my mac using
Mac OS X 10.7.5
XCode 4.5.2 (4G2008a)

and (nearly) following the guide of Bernhard. Only, I use the git-Version as of today (08f738f9).

In addition to the tools mentioned in the guide, I had to install autoconf and automake.

Then I did:
./autogen.sh 
./configure --prefix=/usr/local/ptxdist --with-bash-completion=/usr/local/etc/bash_completion.d
make

This ends in the following error:
----- 8< -----
gcc -g -O2  -DCURSES_LOC="<curses.h>" -DKBUILD_NO_NLS -DPACKAGE='"ptxdist"' -DCONFIG_='"PTXCONF_"' -c mconf.c -o mconf.o
mconf.c: In function ‘search_conf’:
mconf.c:378: error: invalid initializer
make[1]: *** [mconf.o] Error 1
make: *** [kconfig] Error 2
----- >8 -----

Happily, clang is more nosy on the error:
----- 8< -----
clang -g -O2  -DCURSES_LOC="<curses.h>" -DKBUILD_NO_NLS -DPACKAGE='"ptxdist"' -DCONFIG_='"PTXCONF_"' -c mconf.c -o mconf.o
mconf.c:378:25: warning: implicit declaration of function 'CIRCLEQ_HEAD_INITIALIZER' is invalid in C99 [-Wimplicit-function-declaration]
                struct jk_head head = CIRCLEQ_HEAD_INITIALIZER(head);
                                      ^
mconf.c:378:18: error: initializing 'struct jk_head' with an expression of incompatible type 'int'
                struct jk_head head = CIRCLEQ_HEAD_INITIALIZER(head);
                               ^      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning and 1 error generated.
----- >8 -----

So the solution is to add "missing.h" to mconf.c, when compiling on a mac. But I'm not sure, what's the pretty way to do that in a platform independent way. But I'm sure you may lend me a hand here?


Regards
	Daniel

-- 
Daniel Kriesten
Chair for Circuit and System Design
Chemnitz University of Technology, Reichenhainer Str. 31-33, D-09126 Chemnitz
Phone: +49 371 531-33158, Fax: +49 371 531-833158
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-12-08 15:31                                 ` Daniel Kriesten
  2012-12-08 15:43                                   ` [ptxdist] Using PTXdist on Mac OS 10.7 Daniel Kriesten
@ 2012-12-10  8:46                                   ` Michael Olbrich
  2012-12-11 15:00                                     ` Daniel Kriesten
  1 sibling, 1 reply; 37+ messages in thread
From: Michael Olbrich @ 2012-12-10  8:46 UTC (permalink / raw)
  To: ptxdist

On Sat, Dec 08, 2012 at 04:31:34PM +0100, Daniel Kriesten wrote:
> Hi all,
> 
> this is quite a late reply to this mail. But today I struggled about this once again. So here its the patch.

${PTXDIST_TOPDIR}/bin/xargs should be a gnu xargs. Is that not used?

Michael

> ---8<---
> 
> From dd717be19e704b71a030b091d2604d7a05ccebb2 Mon Sep 17 00:00:00 2001
> From: Daniel Kriesten <krid@tu-chemnitz.eu>
> Date: Sat, 8 Dec 2012 16:27:35 +0100
> Subject: [PATCH] Remove -r from call to xargs
> 
> The '-r' flag is a GNU extension. To enhance compatibility remove '-r'
> and suppress error mesages using 'rm -f'.
> ---
>  Makefile.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Makefile.in b/Makefile.in
> index e171045..72509a2 100644
> --- a/Makefile.in
> +++ b/Makefile.in
> @@ -71,7 +71,7 @@ environment:
>  
>  clean:
>         @rm -f .done
> -       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> +       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm -f
>         @$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>  
>  dirty-check:
> -- 
> 1.8.0
> 
> --->8---
> 
> Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:
> > Hi,
> > 
> > On 19.04.2012 14:04, Daniel Kriesten wrote:
> >> Hi all,
> >> 
> >> Am 19.04.2012 um 13:29 schrieb Andreas Bießmann:
> >> 
> >>> Hi all,
> >>> 
> >>> On 19.04.2012 12:41, Daniel Kriesten wrote:
> >>>> Hi Michael,
> >>>> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0
> >>> 
> >>> yes, OS X delivered BSD find supports -print0, but it does _not_ support
> >>> omitting the path!
> >>> [...]
> >>> OR leaving the xargs -r param
> >>> 
> >>> ---8<---
> >>> --- a/Makefile.in
> >>> +++ b/Makefile.in
> >>> @@ -63,7 +63,7 @@ environment:
> >>> 
> >>> clean:
> >>> 	@rm -f .done
> >>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
> >>> +	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
> >>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
> >>> --->8---
> >> 
> >> This one is my promoted solution in yesterdays patch, except I would recommend 'rm -f' to avoid errors if find returns an empty list.
> > 
> > well ok, +1 for your solution. I maybe missed that in my mail history
> > (can you please send patches inline next time?).
> > 
> > best regards
> > 
> > Andreas Bießmann
> > 
> > -- 
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> > 
> 
> 
> -- 
> Daniel Kriesten
> Professur Schaltkreis- und Systementwurf
> Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
> Tel: +49 371 531 33058  Fax: +49 371 531 833058
> http://www.tu-chemnitz.de/etit/sse
> 



> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS 10.7
  2012-12-08 15:43                                   ` [ptxdist] Using PTXdist on Mac OS 10.7 Daniel Kriesten
@ 2012-12-10  8:48                                     ` Michael Olbrich
  0 siblings, 0 replies; 37+ messages in thread
From: Michael Olbrich @ 2012-12-10  8:48 UTC (permalink / raw)
  To: ptxdist

On Sat, Dec 08, 2012 at 04:43:56PM +0100, Daniel Kriesten wrote:
> Dear all,
> 
> So today I did a fresh build of ptxdist on my mac using
> Mac OS X 10.7.5
> XCode 4.5.2 (4G2008a)
> 
> and (nearly) following the guide of Bernhard. Only, I use the git-Version as of today (08f738f9).
> 
> In addition to the tools mentioned in the guide, I had to install autoconf and automake.
> 
> Then I did:
> ./autogen.sh 
> ./configure --prefix=/usr/local/ptxdist --with-bash-completion=/usr/local/etc/bash_completion.d
> make
> 
> This ends in the following error:
> ----- 8< -----
> gcc -g -O2  -DCURSES_LOC="<curses.h>" -DKBUILD_NO_NLS -DPACKAGE='"ptxdist"' -DCONFIG_='"PTXCONF_"' -c mconf.c -o mconf.o
> mconf.c: In function ‘search_conf’:
> mconf.c:378: error: invalid initializer
> make[1]: *** [mconf.o] Error 1
> make: *** [kconfig] Error 2
> ----- >8 -----
> 
> Happily, clang is more nosy on the error:
> ----- 8< -----
> clang -g -O2  -DCURSES_LOC="<curses.h>" -DKBUILD_NO_NLS -DPACKAGE='"ptxdist"' -DCONFIG_='"PTXCONF_"' -c mconf.c -o mconf.o
> mconf.c:378:25: warning: implicit declaration of function 'CIRCLEQ_HEAD_INITIALIZER' is invalid in C99 [-Wimplicit-function-declaration]
>                 struct jk_head head = CIRCLEQ_HEAD_INITIALIZER(head);
>                                       ^
> mconf.c:378:18: error: initializing 'struct jk_head' with an expression of incompatible type 'int'
>                 struct jk_head head = CIRCLEQ_HEAD_INITIALIZER(head);
>                                ^      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 1 warning and 1 error generated.
> ----- >8 -----
> 
> So the solution is to add "missing.h" to mconf.c, when compiling on a mac. But I'm not sure, what's the pretty way to do that in a platform independent way. But I'm sure you may lend me a hand here?

Just include it unconditionally "missing.h" only defines things that are
actually missing. If it breaks anything, then it's a bug in "missing.h".

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-12-10  8:46                                   ` [ptxdist] Using PTXdist on Mac OS Michael Olbrich
@ 2012-12-11 15:00                                     ` Daniel Kriesten
  2012-12-12 11:00                                       ` Michael Olbrich
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Kriesten @ 2012-12-11 15:00 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 3752 bytes --]


Am 10.12.2012 um 09:46 schrieb Michael Olbrich:

> On Sat, Dec 08, 2012 at 04:31:34PM +0100, Daniel Kriesten wrote:
>> 
>> 
>> this is quite a late reply to this mail. But today I struggled about this once again. So here its the patch.
> 
> ${PTXDIST_TOPDIR}/bin/xargs should be a gnu xargs. Is that not used?

It  is used, if it already exists. But on a system, where gnu xargs is not the default (like Mac), something like
./configure
make clean

or
./configure
make
make clean; make clean

fails, due to missing xargs with GNUoption -r

Regards
	Daniel

> Michael
> 
>> ---8<---
>> 
>> From dd717be19e704b71a030b091d2604d7a05ccebb2 Mon Sep 17 00:00:00 2001
>> From: Daniel Kriesten <krid@tu-chemnitz.eu>
>> Date: Sat, 8 Dec 2012 16:27:35 +0100
>> Subject: [PATCH] Remove -r from call to xargs
>> 
>> The '-r' flag is a GNU extension. To enhance compatibility remove '-r'
>> and suppress error mesages using 'rm -f'.
>> ---
>> Makefile.in | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/Makefile.in b/Makefile.in
>> index e171045..72509a2 100644
>> --- a/Makefile.in
>> +++ b/Makefile.in
>> @@ -71,7 +71,7 @@ environment:
>> 
>> clean:
>>        @rm -f .done
>> -       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>> +       @find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm -f
>>        @$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>> 
>> dirty-check:
>> -- 
>> 1.8.0
>> 
>> --->8---
>> 
>> Am 19.04.2012 um 14:25 schrieb Andreas Bießmann:
>>> Hi,
>>> 
>>> On 19.04.2012 14:04, Daniel Kriesten wrote:
>>>> Hi all,
>>>> 
>>>> Am 19.04.2012 um 13:29 schrieb Andreas Bießmann:
>>>> 
>>>>> Hi all,
>>>>> 
>>>>> On 19.04.2012 12:41, Daniel Kriesten wrote:
>>>>>> Hi Michael,
>>>>>> at leas on a Mac (OS X 10.6.8) the BSD find supports -print0
>>>>> 
>>>>> yes, OS X delivered BSD find supports -print0, but it does _not_ support
>>>>> omitting the path!
>>>>> [...]
>>>>> OR leaving the xargs -r param
>>>>> 
>>>>> ---8<---
>>>>> --- a/Makefile.in
>>>>> +++ b/Makefile.in
>>>>> @@ -63,7 +63,7 @@ environment:
>>>>> 
>>>>> clean:
>>>>> 	@rm -f .done
>>>>> -	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -r -0 rm
>>>>> +	@find "$(abs_srcdir)/bin" -type l -print0 | xargs -0 rm
>>>>> 	@$(MAKE) -C "$(abs_srcdir)/scripts/kconfig" clean
>>>>> --->8---
>>>> 
>>>> This one is my promoted solution in yesterdays patch, except I would recommend 'rm -f' to avoid errors if find returns an empty list.
>>> 
>>> well ok, +1 for your solution. I maybe missed that in my mail history
>>> (can you please send patches inline next time?).
>>> 
>>> best regards
>>> 
>>> Andreas Bießmann
>>> 
>>> -- 
>>> ptxdist mailing list
>>> ptxdist@pengutronix.de
>>> 
>> 
>> 
>> -- 
>> Daniel Kriesten
>> Professur Schaltkreis- und Systementwurf
>> Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
>> Tel: +49 371 531 33058  Fax: +49 371 531 833058
>> http://www.tu-chemnitz.de/etit/sse
>> 
> 
> 
> 
>> -- 
>> ptxdist mailing list
>> ptxdist@pengutronix.de
> 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 


-- 
Daniel Kriesten
Professur Schaltkreis- und Systementwurf
Technische Universität Chemnitz, Reichenhainer Str. 31-33, 09126 Chemnitz
Tel: +49 371 531 33058  Fax: +49 371 531 833058
http://www.tu-chemnitz.de/etit/sse


[-- Attachment #1.2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 4964 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [ptxdist] Using PTXdist on Mac OS
  2012-12-11 15:00                                     ` Daniel Kriesten
@ 2012-12-12 11:00                                       ` Michael Olbrich
  0 siblings, 0 replies; 37+ messages in thread
From: Michael Olbrich @ 2012-12-12 11:00 UTC (permalink / raw)
  To: ptxdist

On Tue, Dec 11, 2012 at 04:00:18PM +0100, Daniel Kriesten wrote:
> 
> Am 10.12.2012 um 09:46 schrieb Michael Olbrich:
> 
> > On Sat, Dec 08, 2012 at 04:31:34PM +0100, Daniel Kriesten wrote:
> >> 
> >> 
> >> this is quite a late reply to this mail. But today I struggled about this once again. So here its the patch.
> > 
> > ${PTXDIST_TOPDIR}/bin/xargs should be a gnu xargs. Is that not used?
> 
> It  is used, if it already exists. But on a system, where gnu xargs is not the default (like Mac), something like
> ./configure
> make clean
> 
> or
> ./configure
> make
> make clean; make clean
> 
> fails, due to missing xargs with GNUoption -r

Now I remember the discussion. I guess I forgot to actually make the
change. I've committed it now. Just in time for the release :-).

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 37+ messages in thread

end of thread, other threads:[~2012-12-12 11:00 UTC | newest]

Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-09 17:11 [ptxdist] Using PTXdist on Mac OS Bernhard Walle
2012-04-09 17:48 ` Juergen Beisert
2012-04-10  9:37   ` Bernhard Walle
2012-04-09 18:01 ` Andreas Bießmann
2012-04-09 18:59   ` Bernhard Walle
2012-04-09 19:47     ` Juergen Beisert
2012-04-09 20:04     ` Andreas Bießmann
2012-04-09 20:48     ` Andreas Bießmann
2012-04-10  9:04       ` Bernhard Walle
2012-04-10 18:35         ` Bernhard Walle
2012-04-17 11:08           ` Daniel Kriesten
2012-04-17 11:24             ` Michael Olbrich
2012-04-17 11:45               ` Bernhard Walle
2012-04-17 11:59                 ` Daniel Kriesten
2012-04-17 12:03                   ` Bernhard Walle
2012-04-17 12:32                     ` Daniel Kriesten
2012-04-18  8:29                 ` Michael Olbrich
2012-04-18 11:25                   ` Daniel Kriesten
2012-04-18 14:16                     ` Michael Olbrich
2012-04-18 14:28                       ` Daniel Kriesten
2012-04-19 10:41                         ` Daniel Kriesten
2012-04-19 11:29                           ` Andreas Bießmann
2012-04-19 12:04                             ` Daniel Kriesten
2012-04-19 12:25                               ` Andreas Bießmann
2012-04-19 13:12                                 ` Daniel Kriesten
2012-04-19 13:27                                   ` Michael Olbrich
2012-04-19 14:03                                   ` Andreas Bießmann
2012-04-19 20:13                                   ` Bernhard Walle
2012-12-08 15:31                                 ` Daniel Kriesten
2012-12-08 15:43                                   ` [ptxdist] Using PTXdist on Mac OS 10.7 Daniel Kriesten
2012-12-10  8:48                                     ` Michael Olbrich
2012-12-10  8:46                                   ` [ptxdist] Using PTXdist on Mac OS Michael Olbrich
2012-12-11 15:00                                     ` Daniel Kriesten
2012-12-12 11:00                                       ` Michael Olbrich
2012-04-17 11:48             ` Bernhard Walle
2012-04-17 12:04               ` Daniel Kriesten
2012-04-17 12:18                 ` Bernhard Walle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox