mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Daniel Kriesten <daniel.kriesten@etit.tu-chemnitz.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] Using PTXdist on Mac OS
Date: Sat, 8 Dec 2012 16:31:34 +0100	[thread overview]
Message-ID: <1AD430B7-0904-4E5D-BEA8-D359221672AA@etit.tu-chemnitz.de> (raw)
In-Reply-To: <4F90042A.6060804@corscience.de>


[-- 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

  parent reply	other threads:[~2012-12-08 15:31 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-09 17:11 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 [this message]
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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1AD430B7-0904-4E5D-BEA8-D359221672AA@etit.tu-chemnitz.de \
    --to=daniel.kriesten@etit.tu-chemnitz.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox