* [PATCH 0/2] Fixes for diffutils and u-boot @ 2016-12-05 14:54 Robert Yang 2016-12-05 14:54 ` [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" Robert Yang 2016-12-05 14:54 ` [PATCH 2/2] image.bbclass: inherit image_types_uboot when needed Robert Yang 0 siblings, 2 replies; 6+ messages in thread From: Robert Yang @ 2016-12-05 14:54 UTC (permalink / raw) To: openembedded-core The following changes since commit 11063a01d4511b2688ea7ba2d7359e4e07328c66: ruby: upgrade to 2.3.1 (2016-11-30 15:47:17 +0000) are available in the git repository at: git://git.openembedded.org/openembedded-core-contrib rbt/2fixes http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/2fixes Robert Yang (2): diffutils: do_configure: fix "Argument list too long" image.bbclass: inherit image_types_uboot when needed meta/classes/image.bbclass | 9 +++++++++ meta/recipes-extended/diffutils/diffutils_3.4.bb | 3 +++ 2 files changed, 12 insertions(+) -- 2.9.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" 2016-12-05 14:54 [PATCH 0/2] Fixes for diffutils and u-boot Robert Yang @ 2016-12-05 14:54 ` Robert Yang 2016-12-05 18:23 ` Khem Raj 2016-12-05 14:54 ` [PATCH 2/2] image.bbclass: inherit image_types_uboot when needed Robert Yang 1 sibling, 1 reply; 6+ messages in thread From: Robert Yang @ 2016-12-05 14:54 UTC (permalink / raw) To: openembedded-core Fixed when len(TMPDIR) = 410: aclocal: error: cannot open echo [snip]: Argument list too long This is becuase it has a lot of m4 files, use relative path for them can fix the problem. It doesn't happen when MACHINE="qemux86", I think it is because intel-x86-64 is longer than qemux86. Signed-off-by: Robert Yang <liezhi.yang@windriver.com> --- meta/recipes-extended/diffutils/diffutils_3.4.bb | 3 +++ 1 file changed, 3 insertions(+) diff --git a/meta/recipes-extended/diffutils/diffutils_3.4.bb b/meta/recipes-extended/diffutils/diffutils_3.4.bb index cb7092b..be280ec 100644 --- a/meta/recipes-extended/diffutils/diffutils_3.4.bb +++ b/meta/recipes-extended/diffutils/diffutils_3.4.bb @@ -10,6 +10,9 @@ SRC_URI = "${GNU_MIRROR}/diffutils/diffutils-${PV}.tar.xz \ EXTRA_OECONF += "--without-libsigsegv-prefix" +# Fix "Argument list too long" error when len(TMPDIR) = 410 +acpaths = "-I ./m4" + do_configure_prepend () { # Need to remove gettext macros with weird mix of versions for i in codeset.m4 gettext_gl.m4 intlmacosx.m4 inttypes-pri.m4 lib-ld_gl.m4 lib-prefix_gl.m4 po_gl.m4 ssize_t.m4 wchar_t.m4 wint_t.m4; do -- 2.9.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" 2016-12-05 14:54 ` [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" Robert Yang @ 2016-12-05 18:23 ` Khem Raj 2016-12-06 2:11 ` Robert Yang 0 siblings, 1 reply; 6+ messages in thread From: Khem Raj @ 2016-12-05 18:23 UTC (permalink / raw) To: Robert Yang; +Cc: Patches and discussions about the oe-core layer On Mon, Dec 5, 2016 at 6:54 AM, Robert Yang <liezhi.yang@windriver.com> wrote: > Fixed when len(TMPDIR) = 410: > aclocal: error: cannot open echo [snip]: Argument list too long > > This is becuase it has a lot of m4 files, use relative path for them > can fix the problem. > > It doesn't happen when MACHINE="qemux86", I think it is because > intel-x86-64 is longer than qemux86. > > Signed-off-by: Robert Yang <liezhi.yang@windriver.com> > --- > meta/recipes-extended/diffutils/diffutils_3.4.bb | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/meta/recipes-extended/diffutils/diffutils_3.4.bb b/meta/recipes-extended/diffutils/diffutils_3.4.bb > index cb7092b..be280ec 100644 > --- a/meta/recipes-extended/diffutils/diffutils_3.4.bb > +++ b/meta/recipes-extended/diffutils/diffutils_3.4.bb > @@ -10,6 +10,9 @@ SRC_URI = "${GNU_MIRROR}/diffutils/diffutils-${PV}.tar.xz \ > > EXTRA_OECONF += "--without-libsigsegv-prefix" > > +# Fix "Argument list too long" error when len(TMPDIR) = 410 > +acpaths = "-I ./m4" it looks ok if this work ok with externalsrc > + > do_configure_prepend () { > # Need to remove gettext macros with weird mix of versions > for i in codeset.m4 gettext_gl.m4 intlmacosx.m4 inttypes-pri.m4 lib-ld_gl.m4 lib-prefix_gl.m4 po_gl.m4 ssize_t.m4 wchar_t.m4 wint_t.m4; do > -- > 2.9.0 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" 2016-12-05 18:23 ` Khem Raj @ 2016-12-06 2:11 ` Robert Yang 2016-12-06 2:38 ` Khem Raj 0 siblings, 1 reply; 6+ messages in thread From: Robert Yang @ 2016-12-06 2:11 UTC (permalink / raw) To: Khem Raj; +Cc: Patches and discussions about the oe-core layer On 12/06/2016 02:23 AM, Khem Raj wrote: > On Mon, Dec 5, 2016 at 6:54 AM, Robert Yang <liezhi.yang@windriver.com> wrote: >> Fixed when len(TMPDIR) = 410: >> aclocal: error: cannot open echo [snip]: Argument list too long >> >> This is becuase it has a lot of m4 files, use relative path for them >> can fix the problem. >> >> It doesn't happen when MACHINE="qemux86", I think it is because >> intel-x86-64 is longer than qemux86. >> >> Signed-off-by: Robert Yang <liezhi.yang@windriver.com> >> --- >> meta/recipes-extended/diffutils/diffutils_3.4.bb | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/meta/recipes-extended/diffutils/diffutils_3.4.bb b/meta/recipes-extended/diffutils/diffutils_3.4.bb >> index cb7092b..be280ec 100644 >> --- a/meta/recipes-extended/diffutils/diffutils_3.4.bb >> +++ b/meta/recipes-extended/diffutils/diffutils_3.4.bb >> @@ -10,6 +10,9 @@ SRC_URI = "${GNU_MIRROR}/diffutils/diffutils-${PV}.tar.xz \ >> >> EXTRA_OECONF += "--without-libsigsegv-prefix" >> >> +# Fix "Argument list too long" error when len(TMPDIR) = 410 >> +acpaths = "-I ./m4" > > it looks ok if this work ok with externalsrc Hi Khem, What did mean about externalsrc, please ? Did you mean when B != S? If yes, it works well since it is for aclocal, which always runs in S. We have already such a fix in several recipes. meta/recipes-extended/tar/tar.inc:acpaths = "-I ./m4" meta/recipes-extended/wget/wget.inc:acpaths = "-I ./m4" meta/recipes-core/coreutils/coreutils_6.9.bb:acpaths = "-I ./m4" meta/recipes-core/coreutils/coreutils_8.25.bb:acpaths = "-I ./m4" // Robert > >> + >> do_configure_prepend () { >> # Need to remove gettext macros with weird mix of versions >> for i in codeset.m4 gettext_gl.m4 intlmacosx.m4 inttypes-pri.m4 lib-ld_gl.m4 lib-prefix_gl.m4 po_gl.m4 ssize_t.m4 wchar_t.m4 wint_t.m4; do >> -- >> 2.9.0 >> >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" 2016-12-06 2:11 ` Robert Yang @ 2016-12-06 2:38 ` Khem Raj 0 siblings, 0 replies; 6+ messages in thread From: Khem Raj @ 2016-12-06 2:38 UTC (permalink / raw) To: Robert Yang; +Cc: Patches and discussions about the oe-core layer > On Dec 5, 2016, at 6:11 PM, Robert Yang <liezhi.yang@windriver.com> wrote: > > > > On 12/06/2016 02:23 AM, Khem Raj wrote: >> On Mon, Dec 5, 2016 at 6:54 AM, Robert Yang <liezhi.yang@windriver.com> wrote: >>> Fixed when len(TMPDIR) = 410: >>> aclocal: error: cannot open echo [snip]: Argument list too long >>> >>> This is becuase it has a lot of m4 files, use relative path for them >>> can fix the problem. >>> >>> It doesn't happen when MACHINE="qemux86", I think it is because >>> intel-x86-64 is longer than qemux86. >>> >>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com> >>> --- >>> meta/recipes-extended/diffutils/diffutils_3.4.bb | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/meta/recipes-extended/diffutils/diffutils_3.4.bb b/meta/recipes-extended/diffutils/diffutils_3.4.bb >>> index cb7092b..be280ec 100644 >>> --- a/meta/recipes-extended/diffutils/diffutils_3.4.bb >>> +++ b/meta/recipes-extended/diffutils/diffutils_3.4.bb >>> @@ -10,6 +10,9 @@ SRC_URI = "${GNU_MIRROR}/diffutils/diffutils-${PV}.tar.xz \ >>> >>> EXTRA_OECONF += "--without-libsigsegv-prefix" >>> >>> +# Fix "Argument list too long" error when len(TMPDIR) = 410 >>> +acpaths = "-I ./m4" >> >> it looks ok if this work ok with externalsrc > > Hi Khem, > > What did mean about externalsrc, please ? Did you mean when B != S? yeah, > If yes, it works well since it is for aclocal, which always runs in S. seems to be the case. Then we should be fine. > We have already such a fix in several recipes. > > meta/recipes-extended/tar/tar.inc:acpaths = "-I ./m4" > meta/recipes-extended/wget/wget.inc:acpaths = "-I ./m4" > meta/recipes-core/coreutils/coreutils_6.9.bb:acpaths = "-I ./m4" > meta/recipes-core/coreutils/coreutils_8.25.bb:acpaths = "-I ./m4" > > // Robert > >> >>> + >>> do_configure_prepend () { >>> # Need to remove gettext macros with weird mix of versions >>> for i in codeset.m4 gettext_gl.m4 intlmacosx.m4 inttypes-pri.m4 lib-ld_gl.m4 lib-prefix_gl.m4 po_gl.m4 ssize_t.m4 wchar_t.m4 wint_t.m4; do >>> -- >>> 2.9.0 >>> >>> -- >>> _______________________________________________ >>> Openembedded-core mailing list >>> Openembedded-core@lists.openembedded.org >>> http://lists.openembedded.org/mailman/listinfo/openembedded-core >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] image.bbclass: inherit image_types_uboot when needed 2016-12-05 14:54 [PATCH 0/2] Fixes for diffutils and u-boot Robert Yang 2016-12-05 14:54 ` [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" Robert Yang @ 2016-12-05 14:54 ` Robert Yang 1 sibling, 0 replies; 6+ messages in thread From: Robert Yang @ 2016-12-05 14:54 UTC (permalink / raw) To: openembedded-core Fixed: MACHINE = "qemuarm" IMAGE_FSTYPES += "ext3.bz2.u-boot" [snip] No IMAGE_CMD defined for IMAGE_FSTYPES entry 'ext3.bz2.u-boot' - possibly invalid type name or missing support class [snip] This is because image_types_uboot is not inherited, inherit it when needed will fix the problem. Signed-off-by: Robert Yang <liezhi.yang@windriver.com> --- meta/classes/image.bbclass | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass index 6111f6d..b10272a 100644 --- a/meta/classes/image.bbclass +++ b/meta/classes/image.bbclass @@ -144,6 +144,15 @@ inherit ${IMAGE_TYPE_live} IMAGE_TYPE_vm = '${@bb.utils.contains_any("IMAGE_FSTYPES", ["vmdk", "vdi", "qcow2", "hdddirect"], "image-vm", "", d)}' inherit ${IMAGE_TYPE_vm} +def build_uboot(d): + if 'u-boot' in (d.getVar('IMAGE_FSTYPES', True) or ''): + return "image_types_uboot" + else: + return "" + +IMAGE_TYPE_uboot = "${@build_uboot(d)}" +inherit ${IMAGE_TYPE_uboot} + python () { deps = " " + imagetypes_getdepends(d) d.appendVarFlag('do_rootfs', 'depends', deps) -- 2.9.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-12-06 2:38 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-12-05 14:54 [PATCH 0/2] Fixes for diffutils and u-boot Robert Yang 2016-12-05 14:54 ` [PATCH 1/2] diffutils: do_configure: fix "Argument list too long" Robert Yang 2016-12-05 18:23 ` Khem Raj 2016-12-06 2:11 ` Robert Yang 2016-12-06 2:38 ` Khem Raj 2016-12-05 14:54 ` [PATCH 2/2] image.bbclass: inherit image_types_uboot when needed Robert Yang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox