* [PATCH 0/1] V2 gcc-cross: argument list too long when do_install
@ 2012-06-14 12:18 Robert Yang
2012-06-14 12:18 ` [PATCH 1/1] " Robert Yang
2012-06-18 16:51 ` [PATCH 0/1] V2 " Saul Wold
0 siblings, 2 replies; 3+ messages in thread
From: Robert Yang @ 2012-06-14 12:18 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenfeng.Zhao
Changes of V2:
* Use the patch rather than sed command as Khem suggested.
// Robert
The following changes since commit 7f017cab0b3e4d5a4fc356eccd8eb8429b0531d0:
connman: bump PR and add some comments (2012-06-13 13:11:23 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib robert/too_long
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/too_long
Robert Yang (1):
gcc-cross: argument list too long when do_install
meta/recipes-devtools/gcc/gcc-4.6.inc | 1 +
.../gcc/gcc-4.6/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
meta/recipes-devtools/gcc/gcc-4.7.inc | 1 +
.../gcc/gcc-4.7/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
4 files changed, 68 insertions(+), 0 deletions(-)
create mode 100644 meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch
create mode 100644 meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 1/1] gcc-cross: argument list too long when do_install
2012-06-14 12:18 [PATCH 0/1] V2 gcc-cross: argument list too long when do_install Robert Yang
@ 2012-06-14 12:18 ` Robert Yang
2012-06-18 16:51 ` [PATCH 0/1] V2 " Saul Wold
1 sibling, 0 replies; 3+ messages in thread
From: Robert Yang @ 2012-06-14 12:18 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenfeng.Zhao
There would be an "Argument list too long" error when the length of
tmpdir is longer than 190, this is caused by:
headers=`echo $(PLUGIN_HEADERS) | tr ' ' '\012' | sort -u`
The PLUGIN_HEADERS is too long before sort, so the "echo" can't handle
it, use the $(sort list) of GNU make which can handle the too long list
would fix the problem, the header would be short enough after sorted.
The "tr ' ' '\012'" was used for translating the space to "\n", the
$(sort list) doesn't need this.
This doesn't impact the output, so it doesn't need the PR bump.
[YOCTO #2591]
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
meta/recipes-devtools/gcc/gcc-4.6.inc | 1 +
.../gcc/gcc-4.6/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
meta/recipes-devtools/gcc/gcc-4.7.inc | 1 +
.../gcc/gcc-4.7/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
4 files changed, 68 insertions(+), 0 deletions(-)
create mode 100644 meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch
create mode 100644 meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch
diff --git a/meta/recipes-devtools/gcc/gcc-4.6.inc b/meta/recipes-devtools/gcc/gcc-4.6.inc
index 7a70a38..03d8871 100644
--- a/meta/recipes-devtools/gcc/gcc-4.6.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.6.inc
@@ -75,6 +75,7 @@ SRC_URI = "svn://gcc.gnu.org/svn/gcc/branches;module=${BRANCH};proto=http \
file://fortran-cross-compile-hack.patch \
file://cpp-honour-sysroot.patch \
file://mips64-default-n64.patch \
+ file://gcc-argument-list-too-long.patch \
"
SRC_URI_append_sh3 = " file://sh3-installfix-fixheaders.patch "
diff --git a/meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch b/meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch
new file mode 100644
index 0000000..70d3c53
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch
@@ -0,0 +1,33 @@
+There would be an "Argument list too long" error when the
+build directory is longer than 200, this is caused by:
+
+headers=`echo $(PLUGIN_HEADERS) | tr ' ' '\012' | sort -u`
+
+The PLUGIN_HEADERS is too long before sort, so the "echo" can't handle
+it, use the $(sort list) of GNU make which can handle the too long list
+would fix the problem, the header would be short enough after sorted.
+The "tr ' ' '\012'" was used for translating the space to "\n", the
+$(sort list) doesn't need this.
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
+
+Upstream-Status: Pending
+---
+ gcc/Makefile.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/gcc/Makefile.in b/gcc/Makefile.in
+--- a/gcc/Makefile.in
++++ b/gcc/Makefile.in
+@@ -4553,7 +4553,7 @@ install-plugin: installdirs lang.install-plugin s-header-vars install-gengtype
+ # We keep the directory structure for files in config or c-family and .def
+ # files. All other files are flattened to a single directory.
+ $(mkinstalldirs) $(DESTDIR)$(plugin_includedir)
+- headers=`echo $(PLUGIN_HEADERS) | tr ' ' '\012' | sort -u`; \
++ headers="$(sort $(PLUGIN_HEADERS))"; \
+ srcdirstrip=`echo "$(srcdir)" | sed 's/[].[^$$\\*|]/\\\\&/g'`; \
+ for file in $$headers; do \
+ if [ -f $$file ] ; then \
+--
+1.7.10.2
+
diff --git a/meta/recipes-devtools/gcc/gcc-4.7.inc b/meta/recipes-devtools/gcc/gcc-4.7.inc
index 0321776..c02195b 100644
--- a/meta/recipes-devtools/gcc/gcc-4.7.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.7.inc
@@ -66,6 +66,7 @@ SRC_URI = "svn://gcc.gnu.org/svn/gcc/branches;module=${BRANCH};proto=http \
file://cpp-honor-sysroot.patch \
file://mips64-default-n64.patch \
file://arm-hard-float-loader.patch \
+ file://gcc-argument-list-too-long.patch \
"
S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/${BRANCH}"
diff --git a/meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch b/meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch
new file mode 100644
index 0000000..70d3c53
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch
@@ -0,0 +1,33 @@
+There would be an "Argument list too long" error when the
+build directory is longer than 200, this is caused by:
+
+headers=`echo $(PLUGIN_HEADERS) | tr ' ' '\012' | sort -u`
+
+The PLUGIN_HEADERS is too long before sort, so the "echo" can't handle
+it, use the $(sort list) of GNU make which can handle the too long list
+would fix the problem, the header would be short enough after sorted.
+The "tr ' ' '\012'" was used for translating the space to "\n", the
+$(sort list) doesn't need this.
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
+
+Upstream-Status: Pending
+---
+ gcc/Makefile.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/gcc/Makefile.in b/gcc/Makefile.in
+--- a/gcc/Makefile.in
++++ b/gcc/Makefile.in
+@@ -4553,7 +4553,7 @@ install-plugin: installdirs lang.install-plugin s-header-vars install-gengtype
+ # We keep the directory structure for files in config or c-family and .def
+ # files. All other files are flattened to a single directory.
+ $(mkinstalldirs) $(DESTDIR)$(plugin_includedir)
+- headers=`echo $(PLUGIN_HEADERS) | tr ' ' '\012' | sort -u`; \
++ headers="$(sort $(PLUGIN_HEADERS))"; \
+ srcdirstrip=`echo "$(srcdir)" | sed 's/[].[^$$\\*|]/\\\\&/g'`; \
+ for file in $$headers; do \
+ if [ -f $$file ] ; then \
+--
+1.7.10.2
+
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 0/1] V2 gcc-cross: argument list too long when do_install
2012-06-14 12:18 [PATCH 0/1] V2 gcc-cross: argument list too long when do_install Robert Yang
2012-06-14 12:18 ` [PATCH 1/1] " Robert Yang
@ 2012-06-18 16:51 ` Saul Wold
1 sibling, 0 replies; 3+ messages in thread
From: Saul Wold @ 2012-06-18 16:51 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Zhenfeng.Zhao
On 06/14/2012 05:18 AM, Robert Yang wrote:
> Changes of V2:
> * Use the patch rather than sed command as Khem suggested.
>
> // Robert
>
> The following changes since commit 7f017cab0b3e4d5a4fc356eccd8eb8429b0531d0:
>
> connman: bump PR and add some comments (2012-06-13 13:11:23 +0100)
>
> are available in the git repository at:
> git://git.pokylinux.org/poky-contrib robert/too_long
> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/too_long
>
> Robert Yang (1):
> gcc-cross: argument list too long when do_install
>
> meta/recipes-devtools/gcc/gcc-4.6.inc | 1 +
> .../gcc/gcc-4.6/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
> meta/recipes-devtools/gcc/gcc-4.7.inc | 1 +
> .../gcc/gcc-4.7/gcc-argument-list-too-long.patch | 33 ++++++++++++++++++++
> 4 files changed, 68 insertions(+), 0 deletions(-)
> create mode 100644 meta/recipes-devtools/gcc/gcc-4.6/gcc-argument-list-too-long.patch
> create mode 100644 meta/recipes-devtools/gcc/gcc-4.7/gcc-argument-list-too-long.patch
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
Merged into OE-Core
Thanks
Sau!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-06-18 17:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-14 12:18 [PATCH 0/1] V2 gcc-cross: argument list too long when do_install Robert Yang
2012-06-14 12:18 ` [PATCH 1/1] " Robert Yang
2012-06-18 16:51 ` [PATCH 0/1] V2 " Saul Wold
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.