[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    [PATCH] build system: do not feed too many files to sed at once
From:       Daniel Borca <dborca () yahoo ! com>
Date:       2013-11-26 20:21:19
Message-ID: 529502BF.8010602 () yahoo ! com
[Download RAW message or body]

Currently, sed pre-opens all files, which may cause EMFILE errors
on systems with low ulimit -n.  Prevent that during build process.
-- 
-dborca

["07-gen-sed.patch" (text/plain)]

Signed-off-by: Daniel Borca <dborca@yahoo.com>
---
 scripts/gen_build_files.sh |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/gen_build_files.sh b/scripts/gen_build_files.sh
index 0989b2f..bf7959b 100755
--- a/scripts/gen_build_files.sh
+++ b/scripts/gen_build_files.sh
@@ -42,7 +42,7 @@ generate()
 }
 
 # (Re)generate include/applets.h
-sed -n 's@^//applet:@@p' "$srctree"/*/*.c "$srctree"/*/*/*.c \
+cat "$srctree"/*/*.c "$srctree"/*/*/*.c | sed -n 's@^//applet:@@p' \
 | generate \
 	"$srctree/include/applets.src.h" \
 	"include/applets.h" \
@@ -52,8 +52,8 @@ sed -n 's@^//applet:@@p' "$srctree"/*/*.c "$srctree"/*/*/*.c \
 # We add line continuation backslash after each line,
 # and insert empty line before each line which doesn't start
 # with space or tab
-sed -n -e 's@^//usage:\([ \t].*\)$@\1 \\@p' -e 's@^//usage:\([^ \t].*\)$@\n\1 \\@p' \
-	"$srctree"/*/*.c "$srctree"/*/*/*.c \
+cat "$srctree"/*/*.c "$srctree"/*/*/*.c | \
+	sed -n -e 's@^//usage:\([ \t].*\)$@\1 \\@p' -e 's@^//usage:\([^ \t].*\)$@\n\1 \\@p' \
 | generate \
 	"$srctree/include/usage.src.h" \
 	"include/usage.h" \
-- 
1.7.4.4



_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

[prev in list] [next in list] [prev in thread] [next in thread]