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