From: "Tim Sander" <tim.sander@hbm.com>
To: barebox@lists.infradead.org
Subject: [PATCH] Re: [PATCH] fix genenv when using svn based enviroment directory
Date: Mon, 23 May 2011 09:47:40 +0200 [thread overview]
Message-ID: <201105230947.40484.tim.sander@hbm.com> (raw)
In-Reply-To: <20110523072159.GE20715@pengutronix.de>
Hi Sascha
> > $objtree/scripts/bareboxenv -s $tempdir $objtree/barebox_default_env
> >
> > -rm -r $tempdir
> > +rm -rf $tempdir
>
> Is this second change intended? If yes, why?
If the barebox environment managed with subversion there are some read only
directories within this dir, which make rm wait for a confirmation, which is
more annoying when building with ptxdist.
But i made this change before the first one, i just checked that the -f option
is not needed anymore since the environment clobbering .svn directories have
been removed earlier.
So the more minimalistic patch version is attached below.
Best Regards
Tim
Signed-of-by: Tim Sander <tim.sander@hbm.com>
---
scripts/genenv | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/scripts/genenv b/scripts/genenv
index de8b4f1..65c074e 100755
--- a/scripts/genenv
+++ b/scripts/genenv
@@ -13,6 +13,8 @@ tempdir=$(mktemp -d)
for i in $*; do
cp -r $i/* $tempdir
done
+find $tempdir -name .svn |xargs rm -r
+
$objtree/scripts/bareboxenv -s $tempdir $objtree/barebox_default_env
rm -r $tempdir
--
1.7.0.4
PS: Forgot the list in reply, this is the manual forward.
Hottinger Baldwin Messtechnik GmbH, Im Tiefen See 45, 64293 Darmstadt, Germany | www.hbm.com
Registered as GmbH (German limited liability corporation) in the commercial register at the local court of Darmstadt, HRB 1147
Company domiciled in Darmstadt | CEO: Andreas Huellhorst | Chairman of the board: James Charles Webster
Als Gesellschaft mit beschraenkter Haftung eingetragen im Handelsregister des Amtsgerichts Darmstadt unter HRB 1147
Sitz der Gesellschaft: Darmstadt | Geschaeftsfuehrung: Andreas Huellhorst | Aufsichtsratsvorsitzender: James Charles Webster
The information in this email is confidential. It is intended solely for the addressee. If you are not the intended recipient, please let me know and delete this email.
Die in dieser E-Mail enthaltene Information ist vertraulich und lediglich für den Empfaenger bestimmt. Sollten Sie nicht der eigentliche Empfaenger sein, informieren Sie mich bitte kurz und loeschen diese E-Mail.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
prev parent reply other threads:[~2011-05-23 7:47 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-20 13:00 Tim Sander
2011-05-23 7:21 ` Sascha Hauer
2011-05-23 7:47 ` Tim Sander [this message]
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=201105230947.40484.tim.sander@hbm.com \
--to=tim.sander@hbm.com \
--cc=barebox@lists.infradead.org \
/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