Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 00/17] uclibc support in oe-core v2
@ 2011-04-25 18:54 Khem Raj
  2011-04-25 18:54 ` [PATCH 01/17] gettext.bbclass: Use _append instead of =+ Khem Raj
                   ` (16 more replies)
  0 siblings, 17 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Here are patches needed on top of oe-core to get uclibc based images
building. It incorporates the feedback from last set of patches. On
top there is a new virtual package virtual/gettext introduced which
is preferred to be gettext but can be overridden e.g. for uclibc it
can be set to proxy-libintl and thereby getting rid of gettext in images

libintl comes from three sources glibc/eglibc, gettext, proxy-libintl
when using eglibc/glibc we chose glibc/eglibc to provide it and not gettext

but when using uclibc one could use gettext to provide it or one could choose
proxy-libintl which stubs out gettext functionality and reduces the image
size.

Virtual/gettext can be provided by gettext or proxy-libintl, when we build
uclibc systems with out NLS then we can get rid of gettext fully by selecting
proxy-libintl to provide both virtual/libintl as well as virtual/gettext

Thanks
-Khem

Pull URL: git://git.openembedded.org/openembedded-core-contrib
  Branch: kraj/uclibc
  Browse: http://git.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=kraj/uclibc

Thanks,
    Khem Raj <raj.khem@gmail.com>
---


Khem Raj (17):
  gettext.bbclass: Use _append instead of =+
  insane.bbclass: Move code to add function to tasks toward the end
  insane.bbclass: Checking for NLS too when checking gettext dependency
  autotools.bbclass: Copy gettext files only if --disable-nls is not
    set
  sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak
  gettext: Divide packaging into gettext-runtime and rest
  gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with
    proxy-libintl
  poky-default.inc: Weakly assign gettext to provide virtual/gettext
  libiconv: update from 1.9.2 -> 1.11.1
  glib-2.0: Inherit gettext should provide right libintl so remove from
    DEPENDS
  util-linux.inc: remove virtual/libintl from DEPENDS
  elfutils_0.148.bb: Fix compilation issues on uclibc
  perl_5.12.2.bb: Undefine features not found in uclibc
  xorg-lib: inherit gettext class instead of adding gettext to DEPENDS
    directly
  attr: Fix compilation on uclibc
  linux-tools.inc: Dummify do_compile_perf and do_install_perf for
    uclibc
  alsa-utils_1.0.23.bb: Fix compilation for uclibc targets

 meta/classes/autotools.bbclass                     |   19 ++--
 meta/classes/gettext.bbclass                       |   31 ++++---
 meta/classes/insane.bbclass                        |   24 +++--
 meta/classes/sstate.bbclass                        |    2 +-
 meta/conf/distro/include/poky-default.inc          |    1 +
 meta/recipes-core/gettext/gettext_0.17.bb          |   39 ++++++-
 meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch |  104 ++++++++++++++++++++
 meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb      |    4 +-
 meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb      |    5 +-
 meta/recipes-core/glib-2.0/glib.inc                |    2 +-
 meta/recipes-core/util-linux/util-linux.inc        |    6 +-
 .../elfutils/elfutils-0.148/mempcpy.patch          |   23 +++++
 meta/recipes-devtools/elfutils/elfutils_0.148.bb   |    6 +-
 .../perl/perl-5.12.2/native-ssp.patch              |   16 ---
 meta/recipes-devtools/perl/perl_5.12.2.bb          |   16 +++
 meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb     |    4 +-
 meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb   |    4 +-
 meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb     |    3 +-
 meta/recipes-kernel/linux/linux-tools.inc          |   17 +++-
 meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb  |    9 +-
 meta/recipes-support/attr/attr.inc                 |    4 +
 meta/recipes-support/attr/ea-acl.inc               |    4 +-
 .../libiconv/libiconv-1.11.1/autoconf.patch        |   48 +++++++++
 .../shared_preloadable_libiconv_linux.patch        |   24 +++++
 meta/recipes-support/libiconv/libiconv_1.11.1.bb   |   30 ++++++
 meta/recipes-support/libiconv/libiconv_1.9.2.bb    |   21 ----
 26 files changed, 366 insertions(+), 100 deletions(-)
 create mode 100644 meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch
 create mode 100644 meta/recipes-devtools/elfutils/elfutils-0.148/mempcpy.patch
 delete mode 100644 meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
 create mode 100644 meta/recipes-support/libiconv/libiconv-1.11.1/autoconf.patch
 create mode 100644 meta/recipes-support/libiconv/libiconv-1.11.1/shared_preloadable_libiconv_linux.patch
 create mode 100644 meta/recipes-support/libiconv/libiconv_1.11.1.bb
 delete mode 100644 meta/recipes-support/libiconv/libiconv_1.9.2.bb

-- 
1.7.4.1




^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-26 19:48   ` Saul Wold
  2011-04-25 18:54 ` [PATCH 02/17] insane.bbclass: Move code to add function to tasks toward the end Khem Raj
                   ` (15 subsequent siblings)
  16 siblings, 1 reply; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Ensure gettext and gettext-native are removed from DEPENDS when
not using NLS

Use append instead of += to get gettext dependecies processed
correctly in all cases

Dont remove gettext-native for native recipes as ENABLE_NLS is
only for target and not for native recipes

Replace using 1 for a boolean type with True

Honor INHIBIT_DEFAULT_DEPS

Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is non
null

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
 1 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/meta/classes/gettext.bbclass b/meta/classes/gettext.bbclass
index a40e74f..a6f80f2 100644
--- a/meta/classes/gettext.bbclass
+++ b/meta/classes/gettext.bbclass
@@ -1,17 +1,24 @@
 def gettext_after_parse(d):
-    # Remove the NLS bits if USE_NLS is no.
-    if bb.data.getVar('USE_NLS', d, 1) == 'no':
-        cfg = oe_filter_out('^--(dis|en)able-nls$', bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
-        cfg += " --disable-nls"
-        depends = bb.data.getVar('DEPENDS', d, 1) or ""
-        bb.data.setVar('DEPENDS', oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
-        bb.data.setVar('EXTRA_OECONF', cfg, d)
-
+   # Remove the NLS bits if USE_NLS is no.
+   if bb.data.getVar('USE_NLS', d, True) == 'no':
+       cfg = oe_filter_out('^--(dis|en)able-nls$', bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
+       cfg += " --disable-nls"
+       depends = bb.data.getVar('DEPENDS', d, True) or ""
+       depends = oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$', depends, d)
+       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross', 'crosssdk'):
+           depends = oe_filter_out('^(gettext-native)$', depends, d)
+       bb.data.setVar('DEPENDS', depends, d)
+       bb.data.setVar('EXTRA_OECONF', cfg, d)
+   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove dependencies
+   # added by this class through DEPENDS_GETTEXT
+   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
+       depends = bb.data.getVar('DEPENDS', d, True) or ""
+       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d, True) + ')$'
+       depends = oe_filter_out(gettext_deps, depends, d)
+       bb.data.setVar('DEPENDS', depends, d)
 python () {
     gettext_after_parse(d)
 }
-
-DEPENDS_GETTEXT = "gettext gettext-native"
-
-DEPENDS =+ "${DEPENDS_GETTEXT}"
 EXTRA_OECONF += "--enable-nls"
+DEPENDS_GETTEXT ?= "virtual/gettext"
+DEPENDS_append = " ${DEPENDS_GETTEXT} "
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 02/17] insane.bbclass: Move code to add function to tasks toward the end
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
  2011-04-25 18:54 ` [PATCH 01/17] gettext.bbclass: Use _append instead of =+ Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 03/17] insane.bbclass: Checking for NLS too when checking gettext dependency Khem Raj
                   ` (14 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Cosmetic change to make syntax highlighters happy

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/insane.bbclass |   15 ++++++++-------
 1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 22a5086..98acf7f 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -554,9 +554,6 @@ python do_package_qa () {
 }
 
 
-# The Staging Func, to check all staging
-#addtask qa_staging after do_populate_sysroot before do_build
-do_populate_sysroot[postfuncs] += "do_qa_staging "
 python do_qa_staging() {
     bb.note("QA checking staging")
 
@@ -564,10 +561,6 @@ python do_qa_staging() {
         bb.fatal("QA staging was broken by the package built above")
 }
 
-# Check broken config.log files, for packages requiring Gettext which don't
-# have it in DEPENDS and for correct LIC_FILES_CHKSUM
-#addtask qa_configure after do_configure before do_compile
-do_configure[postfuncs] += "do_qa_configure "
 python do_qa_configure() {
     configs = []
     workdir = bb.data.getVar('WORKDIR', d, True)
@@ -603,3 +596,11 @@ Missing inherit gettext?""" % config)
     if not package_qa_check_license(workdir, d):
         bb.fatal("Licensing Error: LIC_FILES_CHKSUM does not match, please fix")
 }
+# The Staging Func, to check all staging
+#addtask qa_staging after do_populate_sysroot before do_build
+do_populate_sysroot[postfuncs] += "do_qa_staging "
+
+# Check broken config.log files, for packages requiring Gettext which don't
+# have it in DEPENDS and for correct LIC_FILES_CHKSUM
+#addtask qa_configure after do_configure before do_compile
+do_configure[postfuncs] += "do_qa_configure "
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 03/17] insane.bbclass: Checking for NLS too when checking gettext dependency
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
  2011-04-25 18:54 ` [PATCH 01/17] gettext.bbclass: Use _append instead of =+ Khem Raj
  2011-04-25 18:54 ` [PATCH 02/17] insane.bbclass: Move code to add function to tasks toward the end Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 04/17] autotools.bbclass: Copy gettext files only if --disable-nls is not set Khem Raj
                   ` (13 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Checking for gettext is not needed when --disable-nls is used

Let user know what variant of gettext is missing e.g. gettext-native,
gettext-nativesdk etc, reveals a bit more for user

Check for virtual/gettext

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/insane.bbclass |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 98acf7f..742be5e 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -578,20 +578,21 @@ Rerun configure task after fixing this. The path was '%s'""" % root)
         if "configure.in" in files:
             configs.append(os.path.join(root, "configure.in"))
 
-    if "gettext" not in bb.data.getVar('P', d, True) and "gcc-runtime" not in bb.data.getVar('P', d, True):
+    cnf = bb.data.getVar('EXTRA_OECONF', d, True) or ""
+    if "gettext" not in bb.data.getVar('P', d, True) and "gcc-runtime" not in bb.data.getVar('P', d, True) and "--disable-nls" not in cnf:
        if bb.data.inherits_class('native', d) or bb.data.inherits_class('cross', d) or bb.data.inherits_class('crosssdk', d) or bb.data.inherits_class('nativesdk', d):
           gt = "gettext-native"
        elif bb.data.inherits_class('cross-canadian', d):
           gt = "gettext-nativesdk"
        else:
-          gt = "gettext"
+          gt = "virtual/gettext"
        deps = bb.utils.explode_deps(bb.data.getVar('DEPENDS', d, True) or "")
        if gt not in deps:
           for config in configs:
               gnu = "grep \"^[[:space:]]*AM_GNU_GETTEXT\" %s >/dev/null" % config
               if os.system(gnu) == 0:
-                 bb.fatal("""Gettext required but not in DEPENDS for file %s.
-Missing inherit gettext?""" % config)
+                 bb.fatal("""%s required but not in DEPENDS for file %s.
+Missing inherit gettext?""" % (gt, config))
 
     if not package_qa_check_license(workdir, d):
         bb.fatal("Licensing Error: LIC_FILES_CHKSUM does not match, please fix")
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 04/17] autotools.bbclass: Copy gettext files only if --disable-nls is not set
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (2 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 03/17] insane.bbclass: Checking for NLS too when checking gettext dependency Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak Khem Raj
                   ` (12 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/autotools.bbclass |   19 ++++++++++---------
 1 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/meta/classes/autotools.bbclass b/meta/classes/autotools.bbclass
index 7ebf833..cbd2e96 100644
--- a/meta/classes/autotools.bbclass
+++ b/meta/classes/autotools.bbclass
@@ -127,17 +127,18 @@ autotools_do_configure() {
 			else
 			  CONFIGURE_AC=configure.ac
 			fi
-			if grep "^[[:space:]]*AM_GLIB_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then
-			  if grep "sed.*POTFILES" $CONFIGURE_AC >/dev/null; then
-			    : do nothing -- we still have an old unmodified configure.ac
-			  else
-			    oenote Executing glib-gettextize --force --copy
-			    echo "no" | glib-gettextize --force --copy
+			if ! echo ${EXTRA_OECONF} | grep -q "\-\-disable-nls"; then
+			  if grep "^[[:space:]]*AM_GLIB_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then
+			    if grep "sed.*POTFILES" $CONFIGURE_AC >/dev/null; then
+			      : do nothing -- we still have an old unmodified configure.ac
+			    else
+			      oenote Executing glib-gettextize --force --copy
+			      echo "no" | glib-gettextize --force --copy
+			    fi
+			  else if grep "^[[:space:]]*AM_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then
+			    cp ${STAGING_DATADIR}/gettext/config.rpath ${S}/
 			  fi
-			else if grep "^[[:space:]]*AM_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then
-			  cp ${STAGING_DATADIR}/gettext/config.rpath ${S}/
 			fi
-
 			fi
 			mkdir -p m4
 			if grep "^[[:space:]]*[AI][CT]_PROG_INTLTOOL" $CONFIGURE_AC >/dev/null; then
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (3 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 04/17] autotools.bbclass: Copy gettext files only if --disable-nls is not set Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 06/17] gettext: Divide packaging into gettext-runtime and rest Khem Raj
                   ` (11 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Distributions can then override this variable if needed
useful e.g. when building uclibc/eglibc both using
same tmpdir

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/sstate.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index be650c4..553c6a2 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -1,6 +1,6 @@
 SSTATE_VERSION = "2"
 
-SSTATE_MANIFESTS = "${TMPDIR}/sstate-control"
+SSTATE_MANIFESTS ?= "${TMPDIR}/sstate-control"
 SSTATE_MANFILEBASE = "${SSTATE_MANIFESTS}/manifest-${SSTATE_MANMACH}-"
 SSTATE_MANFILEPREFIX = "${SSTATE_MANFILEBASE}${PN}"
 
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 06/17] gettext: Divide packaging into gettext-runtime and rest
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (4 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 07/17] gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with proxy-libintl Khem Raj
                   ` (10 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

This helps in footprint on space constrained systems e.g.
uclibc based systems where packaging full gettext just for
libintl can be waste of space.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-core/gettext/gettext_0.17.bb |   35 +++++++++++++++++++++++++---
 1 files changed, 31 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/gettext/gettext_0.17.bb b/meta/recipes-core/gettext/gettext_0.17.bb
index b9c4e1c..bfcb5ea 100644
--- a/meta/recipes-core/gettext/gettext_0.17.bb
+++ b/meta/recipes-core/gettext/gettext_0.17.bb
@@ -53,9 +53,36 @@ acpaths = '-I ${S}/autoconf-lib-link/m4/ \
 # 1300    KiB /ep93xx/uclibc-dev_0.9.29-r8_ep93xx.ipk
 # 140     KiB /armv4t/gettext-dev_0.14.1-r6_armv4t.ipk
 # 4       KiB /ep93xx/libgcc-s-dev_4.2.2-r2_ep93xx.ipk
+PACKAGES =+ "gettext-runtime gettext-runtime-dev gettext-runtime-doc"
 
-PACKAGES =+ "libgettextlib libgettextsrc"
-FILES_libgettextlib = "${libdir}/libgettextlib-*.so*"
-FILES_libgettextsrc = "${libdir}/libgettextsrc-*.so*"
-
+FILES_gettext-runtime = "${bindir}/gettext \
+                         ${bindir}/ngettext \
+                         ${bindir}/envsubst \
+                         ${bindir}/gettext.sh \
+                         ${libdir}/libasprintf.so* \
+                         ${libdir}/GNU.Gettext.dll \
+                        "
+FILES_gettext-runtime_append_libc-uclibc = " ${libdir}/libintl.so* \
+                                             ${libdir}/charset.alias \
+                                           "
+FILES_gettext-runtime-dev += "${libdir}/libasprintf.a \
+                      ${includedir}/autosprintf.h \
+                     "
+FILES_gettext-runtime-dev_append_libc-uclibc = " ${libdir}/libintl.a \
+                                                 ${includedir}/libintl.h \
+                                               "
+FILES_gettext-runtime-doc = "${mandir}/man1/gettext.* \
+                             ${mandir}/man1/ngettext.* \
+                             ${mandir}/man1/envsubst.* \
+                             ${mandir}/man1/.* \
+                             ${mandir}/man3/* \
+                             ${docdir}/gettext/gettext.* \
+                             ${docdir}/gettext/ngettext.* \
+                             ${docdir}/gettext/envsubst.* \
+                             ${docdir}/gettext/*.3.html \
+                             ${datadir}/gettext/ABOUT-NLS \
+                             ${docdir}/gettext/csharpdoc/* \
+                             ${docdir}/libasprintf/autosprintf.html \
+                             ${infodir}/autosprintf.info \
+                            "
 BBCLASSEXTEND = "native nativesdk"
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 07/17] gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with proxy-libintl
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (5 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 06/17] gettext: Divide packaging into gettext-runtime and rest Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:54 ` [PATCH 08/17] poky-default.inc: Weakly assign gettext to provide virtual/gettext Khem Raj
                   ` (9 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-core/gettext/gettext_0.17.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/gettext/gettext_0.17.bb b/meta/recipes-core/gettext/gettext_0.17.bb
index bfcb5ea..ce5fc3e 100644
--- a/meta/recipes-core/gettext/gettext_0.17.bb
+++ b/meta/recipes-core/gettext/gettext_0.17.bb
@@ -8,9 +8,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 PR = "r5"
 DEPENDS = "gettext-native virtual/libiconv ncurses expat"
 DEPENDS_virtclass-native = ""
-PROVIDES = "virtual/libintl"
+PROVIDES = "virtual/libintl virtual/gettext"
 PROVIDES_virtclass-native = ""
-
+CONFLICTS_${PN} = "proxy-libintl"
 SRC_URI = "${GNU_MIRROR}/gettext/gettext-${PV}.tar.gz \
            file://autotools.patch;patch=1 \
            file://wchar-uclibc.patch;patch=1 \
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 08/17] poky-default.inc: Weakly assign gettext to provide virtual/gettext
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (6 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 07/17] gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with proxy-libintl Khem Raj
@ 2011-04-25 18:54 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1 Khem Raj
                   ` (8 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:54 UTC (permalink / raw)
  To: OE core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/conf/distro/include/poky-default.inc |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/meta/conf/distro/include/poky-default.inc b/meta/conf/distro/include/poky-default.inc
index 3ec5e1d..91985b3 100644
--- a/meta/conf/distro/include/poky-default.inc
+++ b/meta/conf/distro/include/poky-default.inc
@@ -14,6 +14,7 @@ PREFERRED_PROVIDER_virtual/${TARGET_PREFIX}compilerlibs = "gcc-runtime"
 PREFERRED_PROVIDER_virtual/${TARGET_PREFIX}libc-initial = "${POKYLIBC}-initial"
 PREFERRED_PROVIDER_virtual/${SDK_PREFIX}libc-for-gcc-nativesdk ?= "${POKYLIBC}-nativesdk"
 
+PREFERRED_PROVIDER_virtual/gettext ??= "gettext"
 
 GCCVERSION ?= "4.5.1"
 SDKGCCVERSION ?= "4.5.1"
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (7 preceding siblings ...)
  2011-04-25 18:54 ` [PATCH 08/17] poky-default.inc: Weakly assign gettext to provide virtual/gettext Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 10/17] glib-2.0: Inherit gettext should provide right libintl so remove from DEPENDS Khem Raj
                   ` (7 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

1.11.1 was the last LGPLv2 release
shared_preloadable_libiconv_linux.patch is needed to make libtool 2.4
happy

Autoconf patch is similarly made to point m4 directory and update to
newer autoconf

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../libiconv/libiconv-1.11.1/autoconf.patch        |   48 ++++++++++++++++++++
 .../shared_preloadable_libiconv_linux.patch        |   24 ++++++++++
 meta/recipes-support/libiconv/libiconv_1.11.1.bb   |   30 ++++++++++++
 meta/recipes-support/libiconv/libiconv_1.9.2.bb    |   21 ---------
 4 files changed, 102 insertions(+), 21 deletions(-)
 create mode 100644 meta/recipes-support/libiconv/libiconv-1.11.1/autoconf.patch
 create mode 100644 meta/recipes-support/libiconv/libiconv-1.11.1/shared_preloadable_libiconv_linux.patch
 create mode 100644 meta/recipes-support/libiconv/libiconv_1.11.1.bb
 delete mode 100644 meta/recipes-support/libiconv/libiconv_1.9.2.bb

diff --git a/meta/recipes-support/libiconv/libiconv-1.11.1/autoconf.patch b/meta/recipes-support/libiconv/libiconv-1.11.1/autoconf.patch
new file mode 100644
index 0000000..2b24894
--- /dev/null
+++ b/meta/recipes-support/libiconv/libiconv-1.11.1/autoconf.patch
@@ -0,0 +1,48 @@
+It adds the variables that are needed
+for autoconf 2.65 to reconfigure libiconv and defines the m4 macros 
+directory. Its imported from OE.
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+Index: libiconv-1.11.1/configure.ac
+===================================================================
+--- libiconv-1.11.1.orig/configure.ac
++++ libiconv-1.11.1/configure.ac
+@@ -23,7 +23,7 @@ AC_CONFIG_AUX_DIR(build-aux)
+ AM_INIT_AUTOMAKE(libiconv, 1.11)
+ AC_CONFIG_HEADERS(config.h lib/config.h)
+ AC_PROG_MAKE_SET
+-
++AC_CONFIG_MACRO_DIR([m4])
+ dnl           checks for basic programs
+ 
+ AC_PROG_CC
+Index: libiconv-1.11.1/libcharset/configure.ac
+===================================================================
+--- libiconv-1.11.1.orig/libcharset/configure.ac
++++ libiconv-1.11.1/libcharset/configure.ac
+@@ -16,17 +16,17 @@ dnl along with the GNU CHARSET Library;
+ dnl write to the Free Software Foundation, Inc., 51 Franklin Street,
+ dnl Fifth Floor, Boston, MA 02110-1301, USA.
+ 
+-AC_PREREQ(2.13)
++AC_PREREQ(2.61)
++AC_INIT([libcharset],[1.4] )
++AC_CONFIG_SRCDIR([lib/localcharset.c])
+ 
+-PACKAGE=libcharset
+-VERSION=1.4
+-
+-AC_INIT(lib/localcharset.c)
+ AC_CONFIG_AUX_DIR(build-aux)
+ AC_CONFIG_HEADER(config.h)
+ AC_PROG_MAKE_SET
+-AC_SUBST(PACKAGE)
+-AC_SUBST(VERSION)
++dnl AC_SUBST(PACKAGE)
++dnl AC_SUBST(VERSION)
++
++AC_CONFIG_MACRO_DIR([m4])
+ 
+ dnl           checks for basic programs
+ 
diff --git a/meta/recipes-support/libiconv/libiconv-1.11.1/shared_preloadable_libiconv_linux.patch b/meta/recipes-support/libiconv/libiconv-1.11.1/shared_preloadable_libiconv_linux.patch
new file mode 100644
index 0000000..a913dad
--- /dev/null
+++ b/meta/recipes-support/libiconv/libiconv-1.11.1/shared_preloadable_libiconv_linux.patch
@@ -0,0 +1,24 @@
+With libtool generating shared and static version of libraries needs -fPIC flags
+without this it will not generate the commands to create shared linked library
+Its more enforced by libtool 2.4. I have not checked it with older libtool
+libiconv 1.11.x is relatively old release and libtool 2.4 did not exist when it
+was released these kind of problem are more likely
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+Index: libiconv-1.11.1/lib/Makefile.in
+===================================================================
+--- libiconv-1.11.1.orig/lib/Makefile.in
++++ libiconv-1.11.1/lib/Makefile.in
+@@ -70,9 +70,9 @@ preloadable_libiconv.so : preloadable_li
+ 
+ preloadable_libiconv_linux.so : $(SOURCES)
+ 	if test -n "@GCC@"; then \
+-	  $(LIBTOOL_LINK) $(CC) $(LDFLAGS) $(INCLUDES) $(CFLAGS) $(CPPFLAGS) $(DEFS) -fPIC -DPIC -DLIBICONV_PLUG $(SOURCES) -shared -o preloadable_libiconv_linux.so; \
++	  $(LIBTOOL_LINK) $(CC) $(LDFLAGS) $(INCLUDES) $(CFLAGS) $(CPPFLAGS) $(DEFS) -fPIC -DPIC -DLIBICONV_PLUG $(SOURCES) -rpath $(libdir) -o libpreload_iconv.la && cp .libs/libpreload_iconv.so preloadable_libiconv_linux.so; \
+ 	else \
+-	  $(LIBTOOL_LINK) $(CC) $(LDFLAGS) $(INCLUDES) $(CFLAGS) $(CPPFLAGS) $(DEFS) -KPIC -DPIC -DLIBICONV_PLUG $(SOURCES) -shared -o preloadable_libiconv_linux.so; \
++	  $(LIBTOOL_LINK) $(CC) $(LDFLAGS) $(INCLUDES) $(CFLAGS) $(CPPFLAGS) $(DEFS) -KPIC -DPIC -DLIBICONV_PLUG $(SOURCES) -rpath $(libdir) -o libpreload_iconv.la && cp .libs/libpreload_iconv.so preloadable_libiconv_linux.so; \
+ 	fi
+ 
+ preloadable_libiconv_solaris.so : $(SOURCES)
diff --git a/meta/recipes-support/libiconv/libiconv_1.11.1.bb b/meta/recipes-support/libiconv/libiconv_1.11.1.bb
new file mode 100644
index 0000000..28c70c8
--- /dev/null
+++ b/meta/recipes-support/libiconv/libiconv_1.11.1.bb
@@ -0,0 +1,30 @@
+DESCRIPTION = "GNU libiconv - libiconv is for you if your application needs to support \
+multiple character encodings, but that support lacks from your system."
+HOMEPAGE = "http://www.gnu.org/software/libiconv"
+SECTION = "libs"
+PRIORITY = "optional"
+NOTES = "Needs to be stripped down to: ascii iso8859-1 eucjp iso-2022jp gb utf8"
+PROVIDES = "virtual/libiconv"
+PR = "r0"
+LICENSE = "LGPL"
+LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=9f604d8a4f8e74f4f5140845a21b6674 \
+                    file://libcharset/COPYING.LIB;md5=9f604d8a4f8e74f4f5140845a21b6674"
+SRC_URI = "${GNU_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz \
+           file://autoconf.patch \
+	   file://shared_preloadable_libiconv_linux.patch \
+          "
+
+SRC_URI[md5sum] = "d42b97f6ef5dd0ba4469d520ed732fed"
+SRC_URI[sha256sum] = "e78c347a1a0cb15f2648519e9799151f4b4a934b61ad9ee7424478efe2b8257f"
+
+S = "${WORKDIR}/libiconv-${PV}"
+
+inherit autotools pkgconfig gettext
+
+EXTRA_OECONF += "--enable-shared --enable-static --enable-relocatable"
+
+LEAD_SONAME = "libiconv.so"
+
+do_configure_prepend () {
+	rm -f m4/libtool.m4 libcharset/m4/libtool.m4
+}
diff --git a/meta/recipes-support/libiconv/libiconv_1.9.2.bb b/meta/recipes-support/libiconv/libiconv_1.9.2.bb
deleted file mode 100644
index dac9e3c..0000000
--- a/meta/recipes-support/libiconv/libiconv_1.9.2.bb
+++ /dev/null
@@ -1,21 +0,0 @@
-DESCRIPTION = "GNU libiconv - libiconv is for you if your application needs to support \
-multiple character encodings, but that support lacks from your system."
-HOMEPAGE = "http://www.gnu.org/software/libiconv"
-SECTION = "libs"
-PRIORITY = "optional"
-NOTES = "Needs to be stripped down to: ascii iso8859-1 eucjp iso-2022jp gb utf8"
-PROVIDES = "virtual/libiconv"
-PR = "r4"
-LICENSE = "LGPL"
-SRC_URI = "${GNU_MIRROR}/libiconv/libiconv-${PV}.tar.gz"
-
-S = "${WORKDIR}/libiconv-${PV}"
-
-inherit autotools pkgconfig
-
-EXTRA_OECONF += "--enable-shared --enable-static"
-
-do_configure () {
-	rm -f m4/libtool.m4 libcharset/m4/libtool.m4
-	autotools_do_configure
-}
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 10/17] glib-2.0: Inherit gettext should provide right libintl so remove from DEPENDS
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (8 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1 Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 11/17] util-linux.inc: remove virtual/libintl " Khem Raj
                   ` (6 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

Add no-iconv.patch to provide dummy libiconv in case of uclibc

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch |  104 ++++++++++++++++++++
 meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb      |    4 +-
 meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb      |    5 +-
 meta/recipes-core/glib-2.0/glib.inc                |    2 +-
 4 files changed, 112 insertions(+), 3 deletions(-)
 create mode 100644 meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch

diff --git a/meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch b/meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch
new file mode 100644
index 0000000..cca57d4
--- /dev/null
+++ b/meta/recipes-core/glib-2.0/glib-2.0/no-iconv.patch
@@ -0,0 +1,104 @@
+We provide dummy implementation of iconv functionality if it does not find
+iconv which will be case with uclibc. We remove the check for iconv in configure
+
+This is needed when libintl and libiconv is not available. It wont have any
+impact on case when they are available.
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+
+Index: glib-2.26.1/glib/gconvert.c
+===================================================================
+--- glib-2.26.1.orig/glib/gconvert.c
++++ glib-2.26.1/glib/gconvert.c
+@@ -23,7 +23,7 @@
+ #include "config.h"
+ #include "glibconfig.h"
+ 
+-#ifndef G_OS_WIN32
++#if !defined G_OS_WIN32 && defined HAVE_ICONV_H
+ #include <iconv.h>
+ #endif
+ #include <errno.h>
+@@ -65,6 +65,28 @@
+ #error GNU libiconv not in use but included iconv.h is from libiconv
+ #endif
+ 
++#ifndef HAVE_ICONV_H
++typedef void *iconv_t;
++
++iconv_t iconv_open(const char *tocode, const char *fromcode)
++{
++	return (iconv_t)(-1);
++}
++
++int iconv_close(iconv_t cd)
++{
++	free(cd);
++
++	return 0;
++}
++size_t iconv (iconv_t __cd, char **__inbuf,
++                     size_t *__inbytesleft,
++                     char **__outbuf,
++                     size_t *__outbytesleft)
++{
++	return 0;
++}
++#endif
+ 
+ /**
+  * SECTION:conversions
+Index: glib-2.26.1/configure.ac
+===================================================================
+--- glib-2.26.1.orig/configure.ac
++++ glib-2.26.1/configure.ac
+@@ -373,46 +373,8 @@ dnl *** Tests for iconv ***
+ dnl ***********************
+ dnl
+ dnl We do this before the gettext checks, to avoid distortion
+-
+-dnl On Windows we use a native implementation
+-
+-if test x"$glib_native_win32" = xyes; then
+-  with_libiconv=native
+-else
+-  AC_ARG_WITH(libiconv,
+-	      [AC_HELP_STRING([--with-libiconv=@<:@no/gnu/native@:>@],
+-			      [use the libiconv library])],,
+-	      [with_libiconv=maybe])
+-
+-  found_iconv=no
+-  case $with_libiconv in
+-    maybe)
+-      # Check in the C library first
+-      AC_CHECK_FUNC(iconv_open, [with_libiconv=no; found_iconv=yes])
+-      # Check if we have GNU libiconv
+-      if test $found_iconv = "no"; then
+-	AC_CHECK_LIB(iconv, libiconv_open, [with_libiconv=gnu; found_iconv=yes])
+-      fi
+-      # Check if we have a iconv in -liconv, possibly from vendor
+-      if test $found_iconv = "no"; then
+-	AC_CHECK_LIB(iconv, iconv_open, [with_libiconv=native; found_iconv=yes])
+-      fi
+-      ;;
+-    no)
+-      AC_CHECK_FUNC(iconv_open, [with_libiconv=no; found_iconv=yes])
+-      ;;
+-    gnu|yes)
+-      AC_CHECK_LIB(iconv, libiconv_open, [with_libiconv=gnu; found_iconv=yes])
+-      ;;
+-    native)
+-      AC_CHECK_LIB(iconv, iconv_open, [with_libiconv=native; found_iconv=yes])
+-      ;;
+-  esac
+-
+-  if test "x$found_iconv" = "xno" ; then
+-     AC_MSG_ERROR([*** No iconv() implementation found in C library or libiconv])
+-  fi
+-fi
++found_iconv=yes
++with_libiconv=no
+ 
+ gl_GLIBC21
+ AC_ARG_ENABLE(iconv-cache, 
diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb b/meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb
index 6a1fb6c..256abc1 100644
--- a/meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb
+++ b/meta/recipes-core/glib-2.0/glib-2.0_2.26.1.bb
@@ -1,13 +1,15 @@
 require glib.inc
 
 PE = "1"
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "${GNOME_MIRROR}/glib/2.26/glib-${PV}.tar.bz2 \
            file://configure-libtool.patch \
            file://60_wait-longer-for-threads-to-die.patch \
            file://g_once_init_enter.patch \
           "
+# Only apply this patch for target recipe on uclibc
+SRC_URI_append_libc-uclibc = " ${@['', 'file://no-iconv.patch']['${PN}' == '${BPN}']}"
 
 SRC_URI[md5sum] = "17535accceef55bcb17a74d73f9c2aef"
 SRC_URI[sha256sum] = "7a74ff12b6b9dee1f2d0e520b56b68b621920c4f4250bdf23468e515625c28d5"
diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb b/meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb
index 4014a1b..776284e 100644
--- a/meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb
+++ b/meta/recipes-core/glib-2.0/glib-2.0_2.27.5.bb
@@ -1,6 +1,6 @@
 require glib.inc
 
-PR = "r3"
+PR = "r4"
 
 SRC_URI = "${GNOME_MIRROR}/glib/2.27/glib-${PV}.tar.bz2 \
            file://configure-libtool.patch \
@@ -13,6 +13,9 @@ SRC_URI[sha256sum] = "aad3038db865b762e01b1dc455ffd601b4083c069018d290e5fdfe1a61
 
 SRC_URI_append_virtclass-native = " file://glib-gettextize-dir.patch"
 
+# Only apply this patch for target recipe on uclibc
+SRC_URI_append_libc-uclibc = " ${@['', 'file://no-iconv.patch']['${PN}' == '${BPN}']}"
+
 BBCLASSEXTEND = "native nativesdk"
 
 DEFAULT_PREFERENCE = "-1"
diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-2.0/glib.inc
index 5ebff68..756b9ef 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -15,7 +15,7 @@ SECTION = "libs"
 
 require glib-2.0.inc
 
-DEPENDS = "glib-2.0-native gtk-doc-native virtual/libiconv virtual/libintl zip dbus"
+DEPENDS = "glib-2.0-native gtk-doc-native zip dbus"
 DEPENDS_virtclass-native = "gtk-doc-native pkgconfig-native gettext-native dbus-native"
 DEPENDS_virtclass-nativesdk = "libtool-nativesdk"
 PACKAGES =+ "glib-2.0-utils "
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 11/17] util-linux.inc: remove virtual/libintl from DEPENDS
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (9 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 10/17] glib-2.0: Inherit gettext should provide right libintl so remove from DEPENDS Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc Khem Raj
                   ` (5 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

inherit gettext should do it.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-core/util-linux/util-linux.inc |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-core/util-linux/util-linux.inc b/meta/recipes-core/util-linux/util-linux.inc
index 913bb1b..447e5b7 100644
--- a/meta/recipes-core/util-linux/util-linux.inc
+++ b/meta/recipes-core/util-linux/util-linux.inc
@@ -13,10 +13,10 @@ LIC_FILES_CHKSUM = "file://README.licensing;md5=1530e36fe1304d4535513de90a290df9
                     file://licenses/COPYING.UCB;md5=263860f8968d8bafa5392cab74285262 \
                     file://getopt/COPYING;md5=8ca43cbc842c2336e835926c2166c28b"
 
-DEPENDS = "zlib ncurses virtual/libintl"
-DEPENDS_virtclass-native = "zlib-native ncurses-native lzo-native gettext-native"
-
 inherit autotools gettext
+DEPENDS = "zlib ncurses"
+DEPENDS_virtclass-native_append = " lzo-native"
+
 
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/util-linux-ng/v${MAJOR_VERSION}/util-linux-ng-${PV}.tar.bz2 \
            file://MCONFIG \
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (10 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 11/17] util-linux.inc: remove virtual/libintl " Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc Khem Raj
                   ` (4 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

* Add -luargp and -lintl to linker flags
* __mempcpy is not there on uclibc so define it to mempcpy

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../elfutils/elfutils-0.148/mempcpy.patch          |   23 ++++++++++++++++++++
 meta/recipes-devtools/elfutils/elfutils_0.148.bb   |    6 +++-
 2 files changed, 27 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-devtools/elfutils/elfutils-0.148/mempcpy.patch

diff --git a/meta/recipes-devtools/elfutils/elfutils-0.148/mempcpy.patch b/meta/recipes-devtools/elfutils/elfutils-0.148/mempcpy.patch
new file mode 100644
index 0000000..e1e4ad0
--- /dev/null
+++ b/meta/recipes-devtools/elfutils/elfutils-0.148/mempcpy.patch
@@ -0,0 +1,23 @@
+uclibc does not export __mempcpy like glibc so we alias it here.
+This patch may make sense for upstream but elfutils uses more
+glibc specific features like obstack_printf which are missing in
+uclibc they need to be fixed along to make it work all the way
+
+Upstream-status: Inappropriate[Elfutils uses more glibc specific features]
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+Index: elfutils-0.148/libelf/elf_begin.c
+===================================================================
+--- elfutils-0.148.orig/libelf/elf_begin.c
++++ elfutils-0.148/libelf/elf_begin.c
+@@ -68,6 +68,9 @@
+ #include "libelfP.h"
+ #include "common.h"
+ 
++#ifdef __UCLIBC__
++#define __mempcpy mempcpy
++#endif
+ 
+ /* Create descriptor for archive in memory.  */
+ static inline Elf *
diff --git a/meta/recipes-devtools/elfutils/elfutils_0.148.bb b/meta/recipes-devtools/elfutils/elfutils_0.148.bb
index a55b98d..df933f6 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.148.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.148.bb
@@ -28,8 +28,8 @@ SRC_URI += "\
         file://elf_additions.diff \
 	file://elfutils-fsize.patch \
 	file://remove-unused.patch \
+	file://mempcpy.patch \
 "
-
 # The buildsystem wants to generate 2 .h files from source using a binary it just built,
 # which can not pass the cross compiling, so let's work around it by adding 2 .h files
 # along with the do_configure_prepend()
@@ -38,7 +38,6 @@ SRC_URI += "\
         file://i386_dis.h \
         file://x86_64_dis.h \
 "
-
 inherit autotools
 
 EXTRA_OECONF = "--program-prefix=eu-"
@@ -49,6 +48,9 @@ do_configure_prepend() {
 	cp ${WORKDIR}/*dis.h ${S}/libcpu
 }
 
+# Only append ldflags for target recipe
+TARGET_LDFLAGS_libc-uclibc += "${@['', '-lintl -luargp']['${PN}' == '${BPN}']}"
+
 BBCLASSEXTEND = "native nativesdk"
 
 # Package utilities separately
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (11 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-26 14:38   ` Saul Wold
  2011-04-26 16:18   ` Saul Wold
  2011-04-25 18:55 ` [PATCH 14/17] xorg-lib: inherit gettext class instead of adding gettext to DEPENDS directly Khem Raj
                   ` (3 subsequent siblings)
  16 siblings, 2 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../perl/perl-5.12.2/native-ssp.patch              |   16 ----------------
 meta/recipes-devtools/perl/perl_5.12.2.bb          |   16 ++++++++++++++++
 2 files changed, 16 insertions(+), 16 deletions(-)
 delete mode 100644 meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch

diff --git a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch b/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
deleted file mode 100644
index e1e6f08..0000000
--- a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
+++ /dev/null
@@ -1,16 +0,0 @@
-Upstream-Status:Inappropriate [embedded specific]
-
-Fix for compiling with ssp enabled gcc:
-See http://bugs.openembedded.net/show_bug.cgi?id=1980
-
-diff -Naur perl-5.8.7.orig/cflags.SH perl-5.8.7/cflags.SH
---- perl-5.8.7.orig/cflags.SH	2002-09-30 10:59:07.000000000 +0000
-+++ perl-5.8.7/cflags.SH	2005-10-02 04:08:39.000000000 +0000
-@@ -165,6 +165,8 @@
- esac
- 
-     : Can we perhaps use $ansi2knr here
-+    [[ $file == regcomp ]] && export ccflags="${ccflags} -fno-stack-protector"
-+    [[ $file == regexec ]] && export ccflags="${ccflags} -fno-stack-protector"
-     echo "$cc -c -DPERL_CORE $ccflags $optimize $warn"
-     eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $warn"'
diff --git a/meta/recipes-devtools/perl/perl_5.12.2.bb b/meta/recipes-devtools/perl/perl_5.12.2.bb
index 9ce8156..b841e7d 100644
--- a/meta/recipes-devtools/perl/perl_5.12.2.bb
+++ b/meta/recipes-devtools/perl/perl_5.12.2.bb
@@ -131,6 +131,12 @@ do_configure() {
                        -e "s,\(getnetent_r_proto=\)'\w+',\1'0',g" \
                        -e "s,\(d_sockatmark=\)'define',\1'undef',g" \
                        -e "s,\(d_sockatmarkproto=\)'\w+',\1'0',g" \
+                       -e "s,\(d_eaccess=\)'define',\1'undef',g" \
+                       -e "s,\(d_stdio_ptr_lval=\)'define',\1'undef',g" \
+                       -e "s,\(d_stdio_ptr_lval_sets_cnt=\)'define',\1'undef',g" \
+                       -e "s,\(d_stdiobase=\)'define',\1'undef',g" \
+                       -e "s,\(d_stdstdio=\)'define',\1'undef',g" \
+                       -e "s,-fstack-protector,-fno-stack-protector,g" \
                     config.sh-${TARGET_ARCH}-${TARGET_OS}
         fi
 
@@ -145,6 +151,16 @@ do_configure() {
 	       -e 's,/perl5,/perl,g' \
             config.sh-${TARGET_ARCH}-${TARGET_OS}
 
+       case "${TARGET_ARCH}" in                                                                      
+                x86_64 | powerpc | s390)                                                              
+                        sed -i -e "s,\(need_va_copy=\)'undef',\1'define',g" \                         
+                                config.sh-${TARGET_ARCH}-${TARGET_OS}                                 
+                        ;;                                                                            
+                arm)                                                                                  
+                        sed -i -e "s,\(d_u32align=\)'undef',\1'define',g" \                           
+                                config.sh-${TARGET_ARCH}-${TARGET_OS}                                 
+                        ;;                                                                            
+        esac                                                                                          
         # These are strewn all over the source tree
         for foo in `grep -I -m1 \/usr\/include\/.*\\.h ${WORKDIR}/* -r | cut -f 1 -d ":"` ; do
             echo Fixing: $foo
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 14/17] xorg-lib: inherit gettext class instead of adding gettext to DEPENDS directly
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (12 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 15/17] attr: Fix compilation on uclibc Khem Raj
                   ` (2 subsequent siblings)
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb   |    4 +++-
 meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb |    4 ++--
 meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb   |    3 ++-
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb b/meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb
index 79078ef..793008b 100644
--- a/meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb
+++ b/meta/recipes-graphics/xorg-lib/libxau_1.0.6.bb
@@ -6,10 +6,12 @@ both client-side and server-side."
 
 require xorg-lib-common.inc
 
+inherit gettext
+
 LICENSE = "MIT-style"
 LIC_FILES_CHKSUM = "file://COPYING;md5=7908e342491198401321cec1956807ec"
 
-DEPENDS += " xproto gettext"
+DEPENDS += " xproto"
 PROVIDES = "xau"
 
 PR = "r0"
diff --git a/meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb b/meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb
index d897dab..483dce3 100644
--- a/meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb
+++ b/meta/recipes-graphics/xorg-lib/libxdmcp_1.1.0.bb
@@ -8,6 +8,8 @@ autonomous display."
 
 require xorg-lib-common.inc
 
+inherit gettext
+
 LICENSE = "MIT-style"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d559fb26e129626022e052a5e6e0e123"
 
@@ -17,8 +19,6 @@ PROVIDES = "xdmcp"
 PR = "r0"
 PE = "1"
 
-DEPENDS += "gettext"
-
 XORG_PN = "libXdmcp"
 
 BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb b/meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb
index 335f020..19abe5a 100644
--- a/meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb
+++ b/meta/recipes-graphics/xorg-lib/xtrans_1.2.6.bb
@@ -16,7 +16,8 @@ PE = "1"
 PR = "r0"
 
 RDEPENDS_${PN}-dev = ""
-DEPENDS += "gettext"
+
+inherit gettext
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 15/17] attr: Fix compilation on uclibc
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (13 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 14/17] xorg-lib: inherit gettext class instead of adding gettext to DEPENDS directly Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and do_install_perf for uclibc Khem Raj
  2011-04-25 18:55 ` [PATCH 17/17] alsa-utils_1.0.23.bb: Fix compilation for uclibc targets Khem Raj
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

It needs -lintl added to LDFLAGS for uclibc

inherit gettext class instead of adding gettext to DEPENDS directly

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-support/attr/attr.inc   |    4 ++++
 meta/recipes-support/attr/ea-acl.inc |    4 +---
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-support/attr/attr.inc b/meta/recipes-support/attr/attr.inc
index c110340..b2ed9a1 100644
--- a/meta/recipes-support/attr/attr.inc
+++ b/meta/recipes-support/attr/attr.inc
@@ -20,3 +20,7 @@ do_install_append() {
 	sed -i ${D}${libdir}/libattr.la -e \
 	    s,^libdir=\'${base_libdir}\'$,libdir=\'${libdir}\',
 }
+
+# Only append ldflags for target recipe
+LDFLAGS_libc-uclibc += "${@['', '-lintl']['${PN}' == '${BPN}']}"
+
diff --git a/meta/recipes-support/attr/ea-acl.inc b/meta/recipes-support/attr/ea-acl.inc
index bc93107..a09c31c 100644
--- a/meta/recipes-support/attr/ea-acl.inc
+++ b/meta/recipes-support/attr/ea-acl.inc
@@ -1,11 +1,9 @@
 # this build system is mostly shared by attr and acl
 
-DEPENDS =+ "gettext"
-
 SRC_URI += "file://relative-libdir.patch;striplevel=0 \
             file://no-fixed-prog-path.patch"
 
-inherit autotools
+inherit autotools gettext
 
 # the package comes with a custom config.h.in, it cannot be
 # overwritten by autoheader
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and do_install_perf for uclibc
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (14 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 15/17] attr: Fix compilation on uclibc Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  2011-04-25 18:55 ` [PATCH 17/17] alsa-utils_1.0.23.bb: Fix compilation for uclibc targets Khem Raj
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

Perf wants elfutils and elfutils on uclibc does not work due to
its usage of obstack_printf functionality missing from uclibc.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-kernel/linux/linux-tools.inc |   17 +++++++++++++----
 1 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-tools.inc b/meta/recipes-kernel/linux/linux-tools.inc
index d64c9be..ef81650 100644
--- a/meta/recipes-kernel/linux/linux-tools.inc
+++ b/meta/recipes-kernel/linux/linux-tools.inc
@@ -1,6 +1,11 @@
 # included by kernel recipes if they want to build/provide
 # perf functionality from their tree.
-
+do_compile_perf_libc-uclibc () {
+	:
+}
+do_install_perf_libc-uclibc () {
+	:
+}
 do_compile_perf() {
 	oe_runmake -C ${S}/tools/perf CC="${CC}" LD="${LD}" prefix=${prefix} NO_NEWT=1 NO_DWARF=1
 }
@@ -14,6 +19,10 @@ do_install_perf() {
 addtask compile_perf after do_compile before do_install
 addtask install_perf after do_install before do_package
 
-do_compile_perf[depends] =  "virtual/libc:do_populate_sysroot"
-do_compile_perf[depends] =+ "elfutils:do_populate_sysroot"
-RDEPENDS_perf += "python perl elfutils"
+
+PERFDEPENDS = "virtual/libc:do_populate_sysroot elfutils:do_populate_sysroot"
+PERFDEPENDS_libc-uclibc = ""
+PERFRDEPENDS = "python perl elfutils"
+PERFRDEPENDS_libc-uclibc = ""
+do_compile_perf[depends] = "${PERFDEPENDS}"
+RDEPENDS_perf += "${PERFRDEPENDS}"
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH 17/17] alsa-utils_1.0.23.bb: Fix compilation for uclibc targets
  2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
                   ` (15 preceding siblings ...)
  2011-04-25 18:55 ` [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and do_install_perf for uclibc Khem Raj
@ 2011-04-25 18:55 ` Khem Raj
  16 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-25 18:55 UTC (permalink / raw)
  To: OE core

Use libc-uclibc override to cover all uclibc cases

inherit gettext class instead of adding gettext to DEPENDS directly

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb b/meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb
index 46283cc..46b96ae 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.0.23.bb
@@ -5,11 +5,11 @@ SECTION = "console/utils"
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=393a5ca445f6965873eca0259a17f833 \
                     file://alsactl/utils.c;beginline=1;endline=20;md5=fe9526b055e246b5558809a5ae25c0b9"
-DEPENDS = "alsa-lib ncurses gettext"
+DEPENDS = "alsa-lib ncurses"
 PR = "r0"
 
 SRC_URI = "ftp://ftp.alsa-project.org/pub/utils/alsa-utils-${PV}.tar.bz2 \
-           file://ncursesfix.patch;patch=1 "
+           file://ncursesfix.patch"
 
 SRC_URI[md5sum] = "cb0cf46029ac9549cf3a31bff6a4f4e1"
 SRC_URI[sha256sum] = "b7d05e915e25f8540ad151c6dd558cf0fc1e9bb0ee23052d531c983666a1f7b0"
@@ -19,10 +19,9 @@ SRC_URI[sha256sum] = "b7d05e915e25f8540ad151c6dd558cf0fc1e9bb0ee23052d531c983666
 # please close bug and remove this comment when properly fixed
 #
 EXTRA_OECONF = "--disable-xmlto"
-EXTRA_OECONF_linux-uclibc = "--disable-nls"
-EXTRA_OECONF_linux-uclibcgnueabi = "--disable-nls"
+EXTRA_OECONF_append_libc-uclibc = " --disable-nls"
 
-inherit autotools
+inherit autotools gettext
 
 # This are all packages that we need to make. Also, the now empty alsa-utils
 # ipk depend on them.
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 27+ messages in thread

* Re: [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
  2011-04-25 18:55 ` [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc Khem Raj
@ 2011-04-26 14:38   ` Saul Wold
  2011-04-26 16:18   ` Saul Wold
  1 sibling, 0 replies; 27+ messages in thread
From: Saul Wold @ 2011-04-26 14:38 UTC (permalink / raw)
  To: openembedded-core

On 04/25/2011 11:55 AM, Khem Raj wrote:
> Signed-off-by: Khem Raj<raj.khem@gmail.com>
> ---
>   .../perl/perl-5.12.2/native-ssp.patch              |   16 ----------------
>   meta/recipes-devtools/perl/perl_5.12.2.bb          |   16 ++++++++++++++++
>   2 files changed, 16 insertions(+), 16 deletions(-)
>   delete mode 100644 meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>
Khem, Nitin:

With perl being updated to 5.12.3 at this point, can you confirm this 
patch removal is still correct

Sau!

> diff --git a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch b/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
> deleted file mode 100644
> index e1e6f08..0000000
> --- a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
> +++ /dev/null
> @@ -1,16 +0,0 @@
> -Upstream-Status:Inappropriate [embedded specific]
> -
> -Fix for compiling with ssp enabled gcc:
> -See http://bugs.openembedded.net/show_bug.cgi?id=1980
> -
> -diff -Naur perl-5.8.7.orig/cflags.SH perl-5.8.7/cflags.SH
> ---- perl-5.8.7.orig/cflags.SH	2002-09-30 10:59:07.000000000 +0000
> -+++ perl-5.8.7/cflags.SH	2005-10-02 04:08:39.000000000 +0000
> -@@ -165,6 +165,8 @@
> - esac
> -
> -     : Can we perhaps use $ansi2knr here
> -+    [[ $file == regcomp ]]&&  export ccflags="${ccflags} -fno-stack-protector"
> -+    [[ $file == regexec ]]&&  export ccflags="${ccflags} -fno-stack-protector"
> -     echo "$cc -c -DPERL_CORE $ccflags $optimize $warn"
> -     eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $warn"'
> diff --git a/meta/recipes-devtools/perl/perl_5.12.2.bb b/meta/recipes-devtools/perl/perl_5.12.2.bb
> index 9ce8156..b841e7d 100644
> --- a/meta/recipes-devtools/perl/perl_5.12.2.bb
> +++ b/meta/recipes-devtools/perl/perl_5.12.2.bb
> @@ -131,6 +131,12 @@ do_configure() {
>                          -e "s,\(getnetent_r_proto=\)'\w+',\1'0',g" \
>                          -e "s,\(d_sockatmark=\)'define',\1'undef',g" \
>                          -e "s,\(d_sockatmarkproto=\)'\w+',\1'0',g" \
> +                       -e "s,\(d_eaccess=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdio_ptr_lval=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdio_ptr_lval_sets_cnt=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdiobase=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdstdio=\)'define',\1'undef',g" \
> +                       -e "s,-fstack-protector,-fno-stack-protector,g" \
>                       config.sh-${TARGET_ARCH}-${TARGET_OS}
>           fi
>
> @@ -145,6 +151,16 @@ do_configure() {
>   	       -e 's,/perl5,/perl,g' \
>               config.sh-${TARGET_ARCH}-${TARGET_OS}
>
> +       case "${TARGET_ARCH}" in
> +                x86_64 | powerpc | s390)
> +                        sed -i -e "s,\(need_va_copy=\)'undef',\1'define',g" \
> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
> +                        ;;
> +                arm)
> +                        sed -i -e "s,\(d_u32align=\)'undef',\1'define',g" \
> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
> +                        ;;
> +        esac
>           # These are strewn all over the source tree
>           for foo in `grep -I -m1 \/usr\/include\/.*\\.h ${WORKDIR}/* -r | cut -f 1 -d ":"` ; do
>               echo Fixing: $foo



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
  2011-04-25 18:55 ` [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc Khem Raj
  2011-04-26 14:38   ` Saul Wold
@ 2011-04-26 16:18   ` Saul Wold
  2011-04-26 17:47     ` Khem Raj
  1 sibling, 1 reply; 27+ messages in thread
From: Saul Wold @ 2011-04-26 16:18 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 04/25/2011 11:55 AM, Khem Raj wrote:
> Signed-off-by: Khem Raj<raj.khem@gmail.com>
> ---
>   .../perl/perl-5.12.2/native-ssp.patch              |   16 ----------------
>   meta/recipes-devtools/perl/perl_5.12.2.bb          |   16 ++++++++++++++++
>   2 files changed, 16 insertions(+), 16 deletions(-)
>   delete mode 100644 meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>
Khem,

Looking further at this patch, what affects will removing these features 
have on LSB for example?

What's the bigger affect of removing the stack-protector flags, in the 
do_configure?

Sau!


> diff --git a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch b/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
> deleted file mode 100644
> index e1e6f08..0000000
> --- a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
> +++ /dev/null
> @@ -1,16 +0,0 @@
> -Upstream-Status:Inappropriate [embedded specific]
> -
> -Fix for compiling with ssp enabled gcc:
> -See http://bugs.openembedded.net/show_bug.cgi?id=1980
> -
> -diff -Naur perl-5.8.7.orig/cflags.SH perl-5.8.7/cflags.SH
> ---- perl-5.8.7.orig/cflags.SH	2002-09-30 10:59:07.000000000 +0000
> -+++ perl-5.8.7/cflags.SH	2005-10-02 04:08:39.000000000 +0000
> -@@ -165,6 +165,8 @@
> - esac
> -
> -     : Can we perhaps use $ansi2knr here
> -+    [[ $file == regcomp ]]&&  export ccflags="${ccflags} -fno-stack-protector"
> -+    [[ $file == regexec ]]&&  export ccflags="${ccflags} -fno-stack-protector"
> -     echo "$cc -c -DPERL_CORE $ccflags $optimize $warn"
> -     eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $warn"'
> diff --git a/meta/recipes-devtools/perl/perl_5.12.2.bb b/meta/recipes-devtools/perl/perl_5.12.2.bb
> index 9ce8156..b841e7d 100644
> --- a/meta/recipes-devtools/perl/perl_5.12.2.bb
> +++ b/meta/recipes-devtools/perl/perl_5.12.2.bb
> @@ -131,6 +131,12 @@ do_configure() {
>                          -e "s,\(getnetent_r_proto=\)'\w+',\1'0',g" \
>                          -e "s,\(d_sockatmark=\)'define',\1'undef',g" \
>                          -e "s,\(d_sockatmarkproto=\)'\w+',\1'0',g" \
> +                       -e "s,\(d_eaccess=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdio_ptr_lval=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdio_ptr_lval_sets_cnt=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdiobase=\)'define',\1'undef',g" \
> +                       -e "s,\(d_stdstdio=\)'define',\1'undef',g" \
> +                       -e "s,-fstack-protector,-fno-stack-protector,g" \
>                       config.sh-${TARGET_ARCH}-${TARGET_OS}
>           fi
>
> @@ -145,6 +151,16 @@ do_configure() {
>   	       -e 's,/perl5,/perl,g' \
>               config.sh-${TARGET_ARCH}-${TARGET_OS}
>
> +       case "${TARGET_ARCH}" in
> +                x86_64 | powerpc | s390)
> +                        sed -i -e "s,\(need_va_copy=\)'undef',\1'define',g" \
> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
> +                        ;;
> +                arm)
> +                        sed -i -e "s,\(d_u32align=\)'undef',\1'define',g" \
> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
> +                        ;;
> +        esac
>           # These are strewn all over the source tree
>           for foo in `grep -I -m1 \/usr\/include\/.*\\.h ${WORKDIR}/* -r | cut -f 1 -d ":"` ; do
>               echo Fixing: $foo



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
  2011-04-26 16:18   ` Saul Wold
@ 2011-04-26 17:47     ` Khem Raj
  0 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2011-04-26 17:47 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer

On Tue, Apr 26, 2011 at 9:18 AM, Saul Wold <sgw@linux.intel.com> wrote:
> On 04/25/2011 11:55 AM, Khem Raj wrote:
>>
>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>> ---
>>  .../perl/perl-5.12.2/native-ssp.patch              |   16
>> ----------------
>>  meta/recipes-devtools/perl/perl_5.12.2.bb          |   16
>> ++++++++++++++++
>>  2 files changed, 16 insertions(+), 16 deletions(-)
>>  delete mode 100644
>> meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>>
> Khem,
>
> Looking further at this patch, what affects will removing these features
> have on LSB for example?
>
> What's the bigger affect of removing the stack-protector flags, in the
> do_configure?
>

No effect. Patch was not applied. It is useless to keep unapplied
patches. and the real changes are only for uclibc based targets
so it has no effect for eglibc builds

> Sau!
>
>
>> diff --git a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>> b/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>> deleted file mode 100644
>> index e1e6f08..0000000
>> --- a/meta/recipes-devtools/perl/perl-5.12.2/native-ssp.patch
>> +++ /dev/null
>> @@ -1,16 +0,0 @@
>> -Upstream-Status:Inappropriate [embedded specific]
>> -
>> -Fix for compiling with ssp enabled gcc:
>> -See http://bugs.openembedded.net/show_bug.cgi?id=1980
>> -
>> -diff -Naur perl-5.8.7.orig/cflags.SH perl-5.8.7/cflags.SH
>> ---- perl-5.8.7.orig/cflags.SH  2002-09-30 10:59:07.000000000 +0000
>> -+++ perl-5.8.7/cflags.SH       2005-10-02 04:08:39.000000000 +0000
>> -@@ -165,6 +165,8 @@
>> - esac
>> -
>> -     : Can we perhaps use $ansi2knr here
>> -+    [[ $file == regcomp ]]&&  export ccflags="${ccflags}
>> -fno-stack-protector"
>> -+    [[ $file == regexec ]]&&  export ccflags="${ccflags}
>> -fno-stack-protector"
>> -     echo "$cc -c -DPERL_CORE $ccflags $optimize $warn"
>> -     eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $warn"'
>> diff --git a/meta/recipes-devtools/perl/perl_5.12.2.bb
>> b/meta/recipes-devtools/perl/perl_5.12.2.bb
>> index 9ce8156..b841e7d 100644
>> --- a/meta/recipes-devtools/perl/perl_5.12.2.bb
>> +++ b/meta/recipes-devtools/perl/perl_5.12.2.bb
>> @@ -131,6 +131,12 @@ do_configure() {
>>                         -e "s,\(getnetent_r_proto=\)'\w+',\1'0',g" \
>>                         -e "s,\(d_sockatmark=\)'define',\1'undef',g" \
>>                         -e "s,\(d_sockatmarkproto=\)'\w+',\1'0',g" \
>> +                       -e "s,\(d_eaccess=\)'define',\1'undef',g" \
>> +                       -e "s,\(d_stdio_ptr_lval=\)'define',\1'undef',g" \
>> +                       -e
>> "s,\(d_stdio_ptr_lval_sets_cnt=\)'define',\1'undef',g" \
>> +                       -e "s,\(d_stdiobase=\)'define',\1'undef',g" \
>> +                       -e "s,\(d_stdstdio=\)'define',\1'undef',g" \
>> +                       -e "s,-fstack-protector,-fno-stack-protector,g" \
>>                      config.sh-${TARGET_ARCH}-${TARGET_OS}
>>          fi
>>
>> @@ -145,6 +151,16 @@ do_configure() {
>>               -e 's,/perl5,/perl,g' \
>>              config.sh-${TARGET_ARCH}-${TARGET_OS}
>>
>> +       case "${TARGET_ARCH}" in
>> +                x86_64 | powerpc | s390)
>> +                        sed -i -e
>> "s,\(need_va_copy=\)'undef',\1'define',g" \
>> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
>> +                        ;;
>> +                arm)
>> +                        sed -i -e "s,\(d_u32align=\)'undef',\1'define',g"
>> \
>> +                                config.sh-${TARGET_ARCH}-${TARGET_OS}
>> +                        ;;
>> +        esac
>>          # These are strewn all over the source tree
>>          for foo in `grep -I -m1 \/usr\/include\/.*\\.h ${WORKDIR}/* -r |
>> cut -f 1 -d ":"` ; do
>>              echo Fixing: $foo
>



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-25 18:54 ` [PATCH 01/17] gettext.bbclass: Use _append instead of =+ Khem Raj
@ 2011-04-26 19:48   ` Saul Wold
  2011-04-26 20:02     ` Khem Raj
  0 siblings, 1 reply; 27+ messages in thread
From: Saul Wold @ 2011-04-26 19:48 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 04/25/2011 11:54 AM, Khem Raj wrote:
> Ensure gettext and gettext-native are removed from DEPENDS when
> not using NLS
>
> Use append instead of += to get gettext dependecies processed
> correctly in all cases
>
> Dont remove gettext-native for native recipes as ENABLE_NLS is
> only for target and not for native recipes
>
> Replace using 1 for a boolean type with True
>
> Honor INHIBIT_DEFAULT_DEPS
>
> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is non
> null
>
Khem,

This patch seems to be causing a circular dependency when you try to 
build meta-toolchain-sdk, not sure if you tested that target.

% bitbake meta-toolchain-sdk -n
NOTE: Out of date cache found, rebuilding...
WARNING: No bb files matched BBFILE_PATTERN_yocto 
'^/intel/poky/distro/meta-yocto/'
Parsing recipes: 100% |#########################################| Time: 
00:00:15
Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets, 
6 skipped, 0 masked, 0 errors.

OE Build Configuration:
BB_VERSION        = "1.11.0"
METADATA_BRANCH   = "stage"
METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
TARGET_ARCH       = "i586"
TARGET_OS         = "linux"
MACHINE           = "qemux86"
DISTRO            = "poky-lsb"
DISTRO_VERSION    = "1.0+snapshot-20110426"
TARGET_FPU        = ""

NOTE: Resolving any missing task queue dependencies
NOTE: Preparing runqueue
ERROR: Task 
virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb 
(do_package) has circular dependency on 
virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb 
(do_package)

Is there dependencies on this patch with the other patches? (ie can I 
take part of your other patches or just hold off)?

Sau!

> Signed-off-by: Khem Raj<raj.khem@gmail.com>
> ---
>   meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>   1 files changed, 19 insertions(+), 12 deletions(-)
>
> diff --git a/meta/classes/gettext.bbclass b/meta/classes/gettext.bbclass
> index a40e74f..a6f80f2 100644
> --- a/meta/classes/gettext.bbclass
> +++ b/meta/classes/gettext.bbclass
> @@ -1,17 +1,24 @@
>   def gettext_after_parse(d):
> -    # Remove the NLS bits if USE_NLS is no.
> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
> -        cfg = oe_filter_out('^--(dis|en)able-nls$', bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
> -        cfg += " --disable-nls"
> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
> -        bb.data.setVar('DEPENDS', oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
> -
> +   # Remove the NLS bits if USE_NLS is no.
> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
> +       cfg = oe_filter_out('^--(dis|en)able-nls$', bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
> +       cfg += " --disable-nls"
> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
> +       depends = oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$', depends, d)
> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross', 'crosssdk'):
> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
> +       bb.data.setVar('DEPENDS', depends, d)
> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove dependencies
> +   # added by this class through DEPENDS_GETTEXT
> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d, True) + ')$'
> +       depends = oe_filter_out(gettext_deps, depends, d)
> +       bb.data.setVar('DEPENDS', depends, d)
>   python () {
>       gettext_after_parse(d)
>   }
> -
> -DEPENDS_GETTEXT = "gettext gettext-native"
> -
> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>   EXTRA_OECONF += "--enable-nls"
> +DEPENDS_GETTEXT ?= "virtual/gettext"
> +DEPENDS_append = " ${DEPENDS_GETTEXT} "



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-26 19:48   ` Saul Wold
@ 2011-04-26 20:02     ` Khem Raj
  2011-04-26 22:17       ` Khem Raj
  0 siblings, 1 reply; 27+ messages in thread
From: Khem Raj @ 2011-04-26 20:02 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Tue, Apr 26, 2011 at 12:48 PM, Saul Wold <sgw@linux.intel.com> wrote:
> On 04/25/2011 11:54 AM, Khem Raj wrote:
>>
>> Ensure gettext and gettext-native are removed from DEPENDS when
>> not using NLS
>>
>> Use append instead of += to get gettext dependecies processed
>> correctly in all cases
>>
>> Dont remove gettext-native for native recipes as ENABLE_NLS is
>> only for target and not for native recipes
>>
>> Replace using 1 for a boolean type with True
>>
>> Honor INHIBIT_DEFAULT_DEPS
>>
>> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is non
>> null
>>
> Khem,
>
> This patch seems to be causing a circular dependency when you try to build
> meta-toolchain-sdk, not sure if you tested that target.
>

No I did not. Let me try it out.

> % bitbake meta-toolchain-sdk -n
> NOTE: Out of date cache found, rebuilding...
> WARNING: No bb files matched BBFILE_PATTERN_yocto
> '^/intel/poky/distro/meta-yocto/'
> Parsing recipes: 100% |#########################################| Time:
> 00:00:15
> Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets, 6
> skipped, 0 masked, 0 errors.
>
> OE Build Configuration:
> BB_VERSION        = "1.11.0"
> METADATA_BRANCH   = "stage"
> METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
> TARGET_ARCH       = "i586"
> TARGET_OS         = "linux"
> MACHINE           = "qemux86"
> DISTRO            = "poky-lsb"
> DISTRO_VERSION    = "1.0+snapshot-20110426"
> TARGET_FPU        = ""
>
> NOTE: Resolving any missing task queue dependencies
> NOTE: Preparing runqueue
> ERROR: Task
> virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb
> (do_package) has circular dependency on
> virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb
> (do_package)
>
> Is there dependencies on this patch with the other patches? (ie can I take
> part of your other patches or just hold off)?
>

This patch is kind of needed for few of them for parsing sake.
some of them dont depend on this e.g.

[PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak
[PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1
[PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
[PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
[PATCH 14/17] xorg-lib: inherit gettext class instead of adding
gettext to DEPENDS directly
[PATCH 15/17] attr: Fix compilation on uclibc
[PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and
do_install_perf for uclibc
> Sau!
>
>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>> ---
>>  meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>>  1 files changed, 19 insertions(+), 12 deletions(-)
>>
>> diff --git a/meta/classes/gettext.bbclass b/meta/classes/gettext.bbclass
>> index a40e74f..a6f80f2 100644
>> --- a/meta/classes/gettext.bbclass
>> +++ b/meta/classes/gettext.bbclass
>> @@ -1,17 +1,24 @@
>>  def gettext_after_parse(d):
>> -    # Remove the NLS bits if USE_NLS is no.
>> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
>> -        cfg = oe_filter_out('^--(dis|en)able-nls$',
>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>> -        cfg += " --disable-nls"
>> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
>> -        bb.data.setVar('DEPENDS',
>> oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
>> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
>> -
>> +   # Remove the NLS bits if USE_NLS is no.
>> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
>> +       cfg = oe_filter_out('^--(dis|en)able-nls$',
>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>> +       cfg += " --disable-nls"
>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>> +       depends =
>> oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$',
>> depends, d)
>> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross',
>> 'crosssdk'):
>> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
>> +       bb.data.setVar('DEPENDS', depends, d)
>> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
>> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove
>> dependencies
>> +   # added by this class through DEPENDS_GETTEXT
>> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d, True) +
>> ')$'
>> +       depends = oe_filter_out(gettext_deps, depends, d)
>> +       bb.data.setVar('DEPENDS', depends, d)
>>  python () {
>>      gettext_after_parse(d)
>>  }
>> -
>> -DEPENDS_GETTEXT = "gettext gettext-native"
>> -
>> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>>  EXTRA_OECONF += "--enable-nls"
>> +DEPENDS_GETTEXT ?= "virtual/gettext"
>> +DEPENDS_append = " ${DEPENDS_GETTEXT} "
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-26 20:02     ` Khem Raj
@ 2011-04-26 22:17       ` Khem Raj
  2011-04-27 22:51         ` Saul Wold
  0 siblings, 1 reply; 27+ messages in thread
From: Khem Raj @ 2011-04-26 22:17 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Tue, Apr 26, 2011 at 1:02 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Tue, Apr 26, 2011 at 12:48 PM, Saul Wold <sgw@linux.intel.com> wrote:
>> On 04/25/2011 11:54 AM, Khem Raj wrote:
>>>
>>> Ensure gettext and gettext-native are removed from DEPENDS when
>>> not using NLS
>>>
>>> Use append instead of += to get gettext dependecies processed
>>> correctly in all cases
>>>
>>> Dont remove gettext-native for native recipes as ENABLE_NLS is
>>> only for target and not for native recipes
>>>
>>> Replace using 1 for a boolean type with True
>>>
>>> Honor INHIBIT_DEFAULT_DEPS
>>>
>>> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is non
>>> null
>>>
>> Khem,
>>
>> This patch seems to be causing a circular dependency when you try to build
>> meta-toolchain-sdk, not sure if you tested that target.
>>
>
> No I did not. Let me try it out.
>
>> % bitbake meta-toolchain-sdk -n
>> NOTE: Out of date cache found, rebuilding...
>> WARNING: No bb files matched BBFILE_PATTERN_yocto
>> '^/intel/poky/distro/meta-yocto/'
>> Parsing recipes: 100% |#########################################| Time:
>> 00:00:15
>> Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets, 6
>> skipped, 0 masked, 0 errors.
>>
>> OE Build Configuration:
>> BB_VERSION        = "1.11.0"
>> METADATA_BRANCH   = "stage"
>> METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
>> TARGET_ARCH       = "i586"
>> TARGET_OS         = "linux"
>> MACHINE           = "qemux86"
>> DISTRO            = "poky-lsb"
>> DISTRO_VERSION    = "1.0+snapshot-20110426"
>> TARGET_FPU        = ""
>>
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Preparing runqueue
>> ERROR: Task
>> virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb
>> (do_package) has circular dependency on
>> virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb
>> (do_package)
>>
>> Is there dependencies on this patch with the other patches? (ie can I take
>> part of your other patches or just hold off)?
>>
>
> This patch is kind of needed for few of them for parsing sake.
> some of them dont depend on this e.g.
>
> [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak
> [PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1
> [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
> [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
> [PATCH 14/17] xorg-lib: inherit gettext class instead of adding
> gettext to DEPENDS directly
> [PATCH 15/17] attr: Fix compilation on uclibc
> [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and
> do_install_perf for uclibc


Saul

I have fixed this patch and pushed it to the pull tree. Can you pull fresh
from the pull request tree and give it a shot ? Let me know how it goes

>> Sau!
>>
>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>> ---
>>>  meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>>>  1 files changed, 19 insertions(+), 12 deletions(-)
>>>
>>> diff --git a/meta/classes/gettext.bbclass b/meta/classes/gettext.bbclass
>>> index a40e74f..a6f80f2 100644
>>> --- a/meta/classes/gettext.bbclass
>>> +++ b/meta/classes/gettext.bbclass
>>> @@ -1,17 +1,24 @@
>>>  def gettext_after_parse(d):
>>> -    # Remove the NLS bits if USE_NLS is no.
>>> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
>>> -        cfg = oe_filter_out('^--(dis|en)able-nls$',
>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>> -        cfg += " --disable-nls"
>>> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
>>> -        bb.data.setVar('DEPENDS',
>>> oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
>>> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
>>> -
>>> +   # Remove the NLS bits if USE_NLS is no.
>>> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
>>> +       cfg = oe_filter_out('^--(dis|en)able-nls$',
>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>> +       cfg += " --disable-nls"
>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>> +       depends =
>>> oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$',
>>> depends, d)
>>> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross',
>>> 'crosssdk'):
>>> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
>>> +       bb.data.setVar('DEPENDS', depends, d)
>>> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
>>> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove
>>> dependencies
>>> +   # added by this class through DEPENDS_GETTEXT
>>> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d, True) +
>>> ')$'
>>> +       depends = oe_filter_out(gettext_deps, depends, d)
>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>  python () {
>>>      gettext_after_parse(d)
>>>  }
>>> -
>>> -DEPENDS_GETTEXT = "gettext gettext-native"
>>> -
>>> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>>>  EXTRA_OECONF += "--enable-nls"
>>> +DEPENDS_GETTEXT ?= "virtual/gettext"
>>> +DEPENDS_append = " ${DEPENDS_GETTEXT} "
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-26 22:17       ` Khem Raj
@ 2011-04-27 22:51         ` Saul Wold
  2011-04-27 23:59           ` Khem Raj
  0 siblings, 1 reply; 27+ messages in thread
From: Saul Wold @ 2011-04-27 22:51 UTC (permalink / raw)
  To: openembedded-core

On 04/26/2011 03:17 PM, Khem Raj wrote:
> On Tue, Apr 26, 2011 at 1:02 PM, Khem Raj<raj.khem@gmail.com>  wrote:
>> On Tue, Apr 26, 2011 at 12:48 PM, Saul Wold<sgw@linux.intel.com>  wrote:
>>> On 04/25/2011 11:54 AM, Khem Raj wrote:
>>>>
>>>> Ensure gettext and gettext-native are removed from DEPENDS when
>>>> not using NLS
>>>>
>>>> Use append instead of += to get gettext dependecies processed
>>>> correctly in all cases
>>>>
>>>> Dont remove gettext-native for native recipes as ENABLE_NLS is
>>>> only for target and not for native recipes
>>>>
>>>> Replace using 1 for a boolean type with True
>>>>
>>>> Honor INHIBIT_DEFAULT_DEPS
>>>>
>>>> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is non
>>>> null
>>>>
>>> Khem,
>>>
>>> This patch seems to be causing a circular dependency when you try to build
>>> meta-toolchain-sdk, not sure if you tested that target.
>>>
>>
>> No I did not. Let me try it out.

Khem,

There still seems to be a problem with binutil-cross-canadian and 
gcc-cross-canadian not finding the gettext-native, it suggests to have 
these recipe inherit gettext, which binutils does via it's .inc file, 
but gcc-cross-canadian did not seem to do.  I tried to add inherit 
gettext to the cross-canadian.bbclass but that did not solve the issue.

Currently the meta-toolchain gmae builds show this problem.

Thanks
	Sau!

>>
>>> % bitbake meta-toolchain-sdk -n
>>> NOTE: Out of date cache found, rebuilding...
>>> WARNING: No bb files matched BBFILE_PATTERN_yocto
>>> '^/intel/poky/distro/meta-yocto/'
>>> Parsing recipes: 100% |#########################################| Time:
>>> 00:00:15
>>> Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets, 6
>>> skipped, 0 masked, 0 errors.
>>>
>>> OE Build Configuration:
>>> BB_VERSION        = "1.11.0"
>>> METADATA_BRANCH   = "stage"
>>> METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
>>> TARGET_ARCH       = "i586"
>>> TARGET_OS         = "linux"
>>> MACHINE           = "qemux86"
>>> DISTRO            = "poky-lsb"
>>> DISTRO_VERSION    = "1.0+snapshot-20110426"
>>> TARGET_FPU        = ""
>>>
>>> NOTE: Resolving any missing task queue dependencies
>>> NOTE: Preparing runqueue
>>> ERROR: Task
>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb
>>> (do_package) has circular dependency on
>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb
>>> (do_package)
>>>
>>> Is there dependencies on this patch with the other patches? (ie can I take
>>> part of your other patches or just hold off)?
>>>
>>
>> This patch is kind of needed for few of them for parsing sake.
>> some of them dont depend on this e.g.
>>
>> [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak
>> [PATCH 09/17] libiconv: update from 1.9.2 ->  1.11.1
>> [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
>> [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
>> [PATCH 14/17] xorg-lib: inherit gettext class instead of adding
>> gettext to DEPENDS directly
>> [PATCH 15/17] attr: Fix compilation on uclibc
>> [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and
>> do_install_perf for uclibc
>
>
> Saul
>
> I have fixed this patch and pushed it to the pull tree. Can you pull fresh
> from the pull request tree and give it a shot ? Let me know how it goes
>
>>> Sau!
>>>
>>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>>> ---
>>>>   meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>>>>   1 files changed, 19 insertions(+), 12 deletions(-)
>>>>
>>>> diff --git a/meta/classes/gettext.bbclass b/meta/classes/gettext.bbclass
>>>> index a40e74f..a6f80f2 100644
>>>> --- a/meta/classes/gettext.bbclass
>>>> +++ b/meta/classes/gettext.bbclass
>>>> @@ -1,17 +1,24 @@
>>>>   def gettext_after_parse(d):
>>>> -    # Remove the NLS bits if USE_NLS is no.
>>>> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
>>>> -        cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>> -        cfg += " --disable-nls"
>>>> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
>>>> -        bb.data.setVar('DEPENDS',
>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
>>>> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>> -
>>>> +   # Remove the NLS bits if USE_NLS is no.
>>>> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
>>>> +       cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>> +       cfg += " --disable-nls"
>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>> +       depends =
>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$',
>>>> depends, d)
>>>> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross',
>>>> 'crosssdk'):
>>>> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove
>>>> dependencies
>>>> +   # added by this class through DEPENDS_GETTEXT
>>>> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d, True) +
>>>> ')$'
>>>> +       depends = oe_filter_out(gettext_deps, depends, d)
>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>>   python () {
>>>>       gettext_after_parse(d)
>>>>   }
>>>> -
>>>> -DEPENDS_GETTEXT = "gettext gettext-native"
>>>> -
>>>> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>>>>   EXTRA_OECONF += "--enable-nls"
>>>> +DEPENDS_GETTEXT ?= "virtual/gettext"
>>>> +DEPENDS_append = " ${DEPENDS_GETTEXT} "
>>>
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>>
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-27 22:51         ` Saul Wold
@ 2011-04-27 23:59           ` Khem Raj
  2011-04-28  0:53             ` Saul Wold
  0 siblings, 1 reply; 27+ messages in thread
From: Khem Raj @ 2011-04-27 23:59 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Wed, Apr 27, 2011 at 3:51 PM, Saul Wold <sgw@linux.intel.com> wrote:
> On 04/26/2011 03:17 PM, Khem Raj wrote:
>>
>> On Tue, Apr 26, 2011 at 1:02 PM, Khem Raj<raj.khem@gmail.com>  wrote:
>>>
>>> On Tue, Apr 26, 2011 at 12:48 PM, Saul Wold<sgw@linux.intel.com>  wrote:
>>>>
>>>> On 04/25/2011 11:54 AM, Khem Raj wrote:
>>>>>
>>>>> Ensure gettext and gettext-native are removed from DEPENDS when
>>>>> not using NLS
>>>>>
>>>>> Use append instead of += to get gettext dependecies processed
>>>>> correctly in all cases
>>>>>
>>>>> Dont remove gettext-native for native recipes as ENABLE_NLS is
>>>>> only for target and not for native recipes
>>>>>
>>>>> Replace using 1 for a boolean type with True
>>>>>
>>>>> Honor INHIBIT_DEFAULT_DEPS
>>>>>
>>>>> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is
>>>>> non
>>>>> null
>>>>>
>>>> Khem,
>>>>
>>>> This patch seems to be causing a circular dependency when you try to
>>>> build
>>>> meta-toolchain-sdk, not sure if you tested that target.
>>>>
>>>
>>> No I did not. Let me try it out.
>
> Khem,
>
> There still seems to be a problem with binutil-cross-canadian and
> gcc-cross-canadian not finding the gettext-native, it suggests to have these
> recipe inherit gettext, which binutils does via it's .inc file, but
> gcc-cross-canadian did not seem to do.  I tried to add inherit gettext to
> the cross-canadian.bbclass but that did not solve the issue.
>

Does it break the build ?
this may be a new issue unearthed. We can fix it with subsequent patch

> Currently the meta-toolchain gmae builds show this problem.
>

I will try to see whats going on here.

> Thanks
>        Sau!
>
>>>
>>>> % bitbake meta-toolchain-sdk -n
>>>> NOTE: Out of date cache found, rebuilding...
>>>> WARNING: No bb files matched BBFILE_PATTERN_yocto
>>>> '^/intel/poky/distro/meta-yocto/'
>>>> Parsing recipes: 100% |#########################################| Time:
>>>> 00:00:15
>>>> Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets,
>>>> 6
>>>> skipped, 0 masked, 0 errors.
>>>>
>>>> OE Build Configuration:
>>>> BB_VERSION        = "1.11.0"
>>>> METADATA_BRANCH   = "stage"
>>>> METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
>>>> TARGET_ARCH       = "i586"
>>>> TARGET_OS         = "linux"
>>>> MACHINE           = "qemux86"
>>>> DISTRO            = "poky-lsb"
>>>> DISTRO_VERSION    = "1.0+snapshot-20110426"
>>>> TARGET_FPU        = ""
>>>>
>>>> NOTE: Resolving any missing task queue dependencies
>>>> NOTE: Preparing runqueue
>>>> ERROR: Task
>>>>
>>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb
>>>> (do_package) has circular dependency on
>>>>
>>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb
>>>> (do_package)
>>>>
>>>> Is there dependencies on this patch with the other patches? (ie can I
>>>> take
>>>> part of your other patches or just hold off)?
>>>>
>>>
>>> This patch is kind of needed for few of them for parsing sake.
>>> some of them dont depend on this e.g.
>>>
>>> [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be
>>> weak
>>> [PATCH 09/17] libiconv: update from 1.9.2 ->  1.11.1
>>> [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
>>> [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
>>> [PATCH 14/17] xorg-lib: inherit gettext class instead of adding
>>> gettext to DEPENDS directly
>>> [PATCH 15/17] attr: Fix compilation on uclibc
>>> [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and
>>> do_install_perf for uclibc
>>
>>
>> Saul
>>
>> I have fixed this patch and pushed it to the pull tree. Can you pull fresh
>> from the pull request tree and give it a shot ? Let me know how it goes
>>
>>>> Sau!
>>>>
>>>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>>>> ---
>>>>>  meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>>>>>  1 files changed, 19 insertions(+), 12 deletions(-)
>>>>>
>>>>> diff --git a/meta/classes/gettext.bbclass
>>>>> b/meta/classes/gettext.bbclass
>>>>> index a40e74f..a6f80f2 100644
>>>>> --- a/meta/classes/gettext.bbclass
>>>>> +++ b/meta/classes/gettext.bbclass
>>>>> @@ -1,17 +1,24 @@
>>>>>  def gettext_after_parse(d):
>>>>> -    # Remove the NLS bits if USE_NLS is no.
>>>>> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
>>>>> -        cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>>> -        cfg += " --disable-nls"
>>>>> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
>>>>> -        bb.data.setVar('DEPENDS',
>>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
>>>>> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>>> -
>>>>> +   # Remove the NLS bits if USE_NLS is no.
>>>>> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
>>>>> +       cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>>> +       cfg += " --disable-nls"
>>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>>> +       depends =
>>>>>
>>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$',
>>>>> depends, d)
>>>>> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross',
>>>>> 'crosssdk'):
>>>>> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
>>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>>> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>>> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove
>>>>> dependencies
>>>>> +   # added by this class through DEPENDS_GETTEXT
>>>>> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
>>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>>> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d,
>>>>> True) +
>>>>> ')$'
>>>>> +       depends = oe_filter_out(gettext_deps, depends, d)
>>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>>>  python () {
>>>>>      gettext_after_parse(d)
>>>>>  }
>>>>> -
>>>>> -DEPENDS_GETTEXT = "gettext gettext-native"
>>>>> -
>>>>> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>>>>>  EXTRA_OECONF += "--enable-nls"
>>>>> +DEPENDS_GETTEXT ?= "virtual/gettext"
>>>>> +DEPENDS_append = " ${DEPENDS_GETTEXT} "
>>>>
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org
>>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>>>
>>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>



^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH 01/17] gettext.bbclass: Use _append instead of =+
  2011-04-27 23:59           ` Khem Raj
@ 2011-04-28  0:53             ` Saul Wold
  0 siblings, 0 replies; 27+ messages in thread
From: Saul Wold @ 2011-04-28  0:53 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 04/27/2011 04:59 PM, Khem Raj wrote:
> On Wed, Apr 27, 2011 at 3:51 PM, Saul Wold<sgw@linux.intel.com>  wrote:
>> On 04/26/2011 03:17 PM, Khem Raj wrote:
>>>
>>> On Tue, Apr 26, 2011 at 1:02 PM, Khem Raj<raj.khem@gmail.com>    wrote:
>>>>
>>>> On Tue, Apr 26, 2011 at 12:48 PM, Saul Wold<sgw@linux.intel.com>    wrote:
>>>>>
>>>>> On 04/25/2011 11:54 AM, Khem Raj wrote:
>>>>>>
>>>>>> Ensure gettext and gettext-native are removed from DEPENDS when
>>>>>> not using NLS
>>>>>>
>>>>>> Use append instead of += to get gettext dependecies processed
>>>>>> correctly in all cases
>>>>>>
>>>>>> Dont remove gettext-native for native recipes as ENABLE_NLS is
>>>>>> only for target and not for native recipes
>>>>>>
>>>>>> Replace using 1 for a boolean type with True
>>>>>>
>>>>>> Honor INHIBIT_DEFAULT_DEPS
>>>>>>
>>>>>> Remove the added dependencies for gettext if INHIBIT_DEFAULT_DEPS is
>>>>>> non
>>>>>> null
>>>>>>
>>>>> Khem,
>>>>>
>>>>> This patch seems to be causing a circular dependency when you try to
>>>>> build
>>>>> meta-toolchain-sdk, not sure if you tested that target.
>>>>>
>>>>
>>>> No I did not. Let me try it out.
>>
>> Khem,
>>
>> There still seems to be a problem with binutil-cross-canadian and
>> gcc-cross-canadian not finding the gettext-native, it suggests to have these
>> recipe inherit gettext, which binutils does via it's .inc file, but
>> gcc-cross-canadian did not seem to do.  I tried to add inherit gettext to
>> the cross-canadian.bbclass but that did not solve the issue.
>>
>
> Does it break the build ?
> this may be a new issue unearthed. We can fix it with subsequent patch
>
Not breaking the mainline build, but it is breaking the toolchain builds.

Sau!

>> Currently the meta-toolchain gmae builds show this problem.
>>
>
> I will try to see whats going on here.
>
>> Thanks
>>         Sau!
>>
>>>>
>>>>> % bitbake meta-toolchain-sdk -n
>>>>> NOTE: Out of date cache found, rebuilding...
>>>>> WARNING: No bb files matched BBFILE_PATTERN_yocto
>>>>> '^/intel/poky/distro/meta-yocto/'
>>>>> Parsing recipes: 100% |#########################################| Time:
>>>>> 00:00:15
>>>>> Parsing of 796 .bb files complete (0 cached, 796 parsed). 1008 targets,
>>>>> 6
>>>>> skipped, 0 masked, 0 errors.
>>>>>
>>>>> OE Build Configuration:
>>>>> BB_VERSION        = "1.11.0"
>>>>> METADATA_BRANCH   = "stage"
>>>>> METADATA_REVISION = "86f12953dbf57264cfcc06feac446f3edf0c6d8f"
>>>>> TARGET_ARCH       = "i586"
>>>>> TARGET_OS         = "linux"
>>>>> MACHINE           = "qemux86"
>>>>> DISTRO            = "poky-lsb"
>>>>> DISTRO_VERSION    = "1.0+snapshot-20110426"
>>>>> TARGET_FPU        = ""
>>>>>
>>>>> NOTE: Resolving any missing task queue dependencies
>>>>> NOTE: Preparing runqueue
>>>>> ERROR: Task
>>>>>
>>>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-core/gettext/gettext_0.17.bb
>>>>> (do_package) has circular dependency on
>>>>>
>>>>> virtual:nativesdk:/intel/poky/distro/meta/recipes-devtools/gcc/gcc-runtime_4.5.1.bb
>>>>> (do_package)
>>>>>
>>>>> Is there dependencies on this patch with the other patches? (ie can I
>>>>> take
>>>>> part of your other patches or just hold off)?
>>>>>
>>>>
>>>> This patch is kind of needed for few of them for parsing sake.
>>>> some of them dont depend on this e.g.
>>>>
>>>> [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be
>>>> weak
>>>> [PATCH 09/17] libiconv: update from 1.9.2 ->    1.11.1
>>>> [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc
>>>> [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc
>>>> [PATCH 14/17] xorg-lib: inherit gettext class instead of adding
>>>> gettext to DEPENDS directly
>>>> [PATCH 15/17] attr: Fix compilation on uclibc
>>>> [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and
>>>> do_install_perf for uclibc
>>>
>>>
>>> Saul
>>>
>>> I have fixed this patch and pushed it to the pull tree. Can you pull fresh
>>> from the pull request tree and give it a shot ? Let me know how it goes
>>>
>>>>> Sau!
>>>>>
>>>>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>>>>> ---
>>>>>>   meta/classes/gettext.bbclass |   31 +++++++++++++++++++------------
>>>>>>   1 files changed, 19 insertions(+), 12 deletions(-)
>>>>>>
>>>>>> diff --git a/meta/classes/gettext.bbclass
>>>>>> b/meta/classes/gettext.bbclass
>>>>>> index a40e74f..a6f80f2 100644
>>>>>> --- a/meta/classes/gettext.bbclass
>>>>>> +++ b/meta/classes/gettext.bbclass
>>>>>> @@ -1,17 +1,24 @@
>>>>>>   def gettext_after_parse(d):
>>>>>> -    # Remove the NLS bits if USE_NLS is no.
>>>>>> -    if bb.data.getVar('USE_NLS', d, 1) == 'no':
>>>>>> -        cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>>>> -        cfg += " --disable-nls"
>>>>>> -        depends = bb.data.getVar('DEPENDS', d, 1) or ""
>>>>>> -        bb.data.setVar('DEPENDS',
>>>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl)$', depends, d), d)
>>>>>> -        bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>>>> -
>>>>>> +   # Remove the NLS bits if USE_NLS is no.
>>>>>> +   if bb.data.getVar('USE_NLS', d, True) == 'no':
>>>>>> +       cfg = oe_filter_out('^--(dis|en)able-nls$',
>>>>>> bb.data.getVar('EXTRA_OECONF', d, 1) or "", d)
>>>>>> +       cfg += " --disable-nls"
>>>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>>>> +       depends =
>>>>>>
>>>>>> oe_filter_out('^(virtual/libiconv|virtual/libintl|virtual/gettext|gettext)$',
>>>>>> depends, d)
>>>>>> +       if not oe.utils.inherits(d, 'native', 'nativesdk', 'cross',
>>>>>> 'crosssdk'):
>>>>>> +           depends = oe_filter_out('^(gettext-native)$', depends, d)
>>>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>>>> +       bb.data.setVar('EXTRA_OECONF', cfg, d)
>>>>>> +   # check if INHIBIT_DEFAULT_DEPS is 1 then we forcibly remove
>>>>>> dependencies
>>>>>> +   # added by this class through DEPENDS_GETTEXT
>>>>>> +   if bb.data.getVar('INHIBIT_DEFAULT_DEPS', d, True):
>>>>>> +       depends = bb.data.getVar('DEPENDS', d, True) or ""
>>>>>> +       gettext_deps = '^(' + bb.data.getVar('DEPENDS_GETTEXT', d,
>>>>>> True) +
>>>>>> ')$'
>>>>>> +       depends = oe_filter_out(gettext_deps, depends, d)
>>>>>> +       bb.data.setVar('DEPENDS', depends, d)
>>>>>>   python () {
>>>>>>       gettext_after_parse(d)
>>>>>>   }
>>>>>> -
>>>>>> -DEPENDS_GETTEXT = "gettext gettext-native"
>>>>>> -
>>>>>> -DEPENDS =+ "${DEPENDS_GETTEXT}"
>>>>>>   EXTRA_OECONF += "--enable-nls"
>>>>>> +DEPENDS_GETTEXT ?= "virtual/gettext"
>>>>>> +DEPENDS_append = " ${DEPENDS_GETTEXT} "
>>>>>
>>>>> _______________________________________________
>>>>> Openembedded-core mailing list
>>>>> Openembedded-core@lists.openembedded.org
>>>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>>>>
>>>>
>>>
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2011-04-28  0:56 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-25 18:54 [PATCH 00/17] uclibc support in oe-core v2 Khem Raj
2011-04-25 18:54 ` [PATCH 01/17] gettext.bbclass: Use _append instead of =+ Khem Raj
2011-04-26 19:48   ` Saul Wold
2011-04-26 20:02     ` Khem Raj
2011-04-26 22:17       ` Khem Raj
2011-04-27 22:51         ` Saul Wold
2011-04-27 23:59           ` Khem Raj
2011-04-28  0:53             ` Saul Wold
2011-04-25 18:54 ` [PATCH 02/17] insane.bbclass: Move code to add function to tasks toward the end Khem Raj
2011-04-25 18:54 ` [PATCH 03/17] insane.bbclass: Checking for NLS too when checking gettext dependency Khem Raj
2011-04-25 18:54 ` [PATCH 04/17] autotools.bbclass: Copy gettext files only if --disable-nls is not set Khem Raj
2011-04-25 18:54 ` [PATCH 05/17] sstate.bbclass: Reduce SSTATE_MANIFESTS assignment to be weak Khem Raj
2011-04-25 18:54 ` [PATCH 06/17] gettext: Divide packaging into gettext-runtime and rest Khem Raj
2011-04-25 18:54 ` [PATCH 07/17] gettext_0.17.bb: PROVIDE virtual/gettext and RCONFLICT with proxy-libintl Khem Raj
2011-04-25 18:54 ` [PATCH 08/17] poky-default.inc: Weakly assign gettext to provide virtual/gettext Khem Raj
2011-04-25 18:55 ` [PATCH 09/17] libiconv: update from 1.9.2 -> 1.11.1 Khem Raj
2011-04-25 18:55 ` [PATCH 10/17] glib-2.0: Inherit gettext should provide right libintl so remove from DEPENDS Khem Raj
2011-04-25 18:55 ` [PATCH 11/17] util-linux.inc: remove virtual/libintl " Khem Raj
2011-04-25 18:55 ` [PATCH 12/17] elfutils_0.148.bb: Fix compilation issues on uclibc Khem Raj
2011-04-25 18:55 ` [PATCH 13/17] perl_5.12.2.bb: Undefine features not found in uclibc Khem Raj
2011-04-26 14:38   ` Saul Wold
2011-04-26 16:18   ` Saul Wold
2011-04-26 17:47     ` Khem Raj
2011-04-25 18:55 ` [PATCH 14/17] xorg-lib: inherit gettext class instead of adding gettext to DEPENDS directly Khem Raj
2011-04-25 18:55 ` [PATCH 15/17] attr: Fix compilation on uclibc Khem Raj
2011-04-25 18:55 ` [PATCH 16/17] linux-tools.inc: Dummify do_compile_perf and do_install_perf for uclibc Khem Raj
2011-04-25 18:55 ` [PATCH 17/17] alsa-utils_1.0.23.bb: Fix compilation for uclibc targets Khem Raj

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox