Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/8] Bug Fixes for M2 Rc2
@ 2011-07-07 23:31 Saul Wold
  2011-07-07 23:31 ` [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper Saul Wold
                   ` (9 more replies)
  0 siblings, 10 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

Richard,

This address the bash, xcb missing from libx11, locale issue and
the lack of space due to zypper eating space.

Sau!


The following changes since commit f1fc6d084b079dea21ff1a30b815496452042490:

  pulseaudio: add 0.9.23 (2011-07-07 13:44:36 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib sgw/fix
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/fix

Saul Wold (8):
  rootfs_rpm: Add 50Meg to image size for zypper
  image.bbclass: add LINGUAS_INSTALL to depends list
  gnome-doc-utils: Add RDEPENDS on bash
  quilt: Add RDEPENDS on bash
  console-tools: Add RDEPENDS on bash
  usbutils: Add RDEPENDS on bash
  eglibc: Add RDEPENDS on bash
  libx11: enable xcb support

 meta/classes/image.bbclass                         |    2 +-
 meta/classes/image_types.bbclass                   |    2 +-
 meta/classes/rootfs_rpm.bbclass                    |    5 +++++
 meta/recipes-bsp/usbutils/usbutils_0.91.bb         |    5 +++--
 .../console-tools/console-tools_0.3.2.bb           |    3 ++-
 meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
 meta/recipes-core/eglibc/eglibc.inc                |    2 +-
 meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
 meta/recipes-core/eglibc/eglibc_2.13.bb            |    2 +-
 meta/recipes-devtools/quilt/quilt.inc              |    2 ++
 meta/recipes-devtools/quilt/quilt_0.48.bb          |    2 +-
 meta/recipes-gnome/gnome/gnome-doc-utils.inc       |    2 ++
 meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb |    2 +-
 meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb     |   11 ++++++++---
 14 files changed, 30 insertions(+), 14 deletions(-)

-- 
1.7.3.4




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

* [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-07 23:38   ` Richard Purdie
  2011-07-07 23:31 ` [PATCH 2/8] image.bbclass: add LINGUAS_INSTALL to depends list Saul Wold
                   ` (8 subsequent siblings)
  9 siblings, 1 reply; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1171]

Add /var space for zypper due to its space usage for db maintence

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/classes/image_types.bbclass |    2 +-
 meta/classes/rootfs_rpm.bbclass  |    5 +++++
 2 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index 8c86227..505342d 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -13,7 +13,7 @@ def get_imagecmds(d):
 
 runimagecmd () {
 	# Image generation code for image type ${type}
-	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE}; OFMT = "%.0f" ; print (size > ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
+	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE} + ${ZYPPER_VAR_DB_SPACE}; OFMT = "%.0f" ; print (size > ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
 	${cmd}
 	cd ${DEPLOY_DIR_IMAGE}/
 	rm -f ${DEPLOY_DIR_IMAGE}/${IMAGE_LINK_NAME}.${type}
diff --git a/meta/classes/rootfs_rpm.bbclass b/meta/classes/rootfs_rpm.bbclass
index 3a11858..67c547f 100644
--- a/meta/classes/rootfs_rpm.bbclass
+++ b/meta/classes/rootfs_rpm.bbclass
@@ -3,6 +3,10 @@
 #
 
 ROOTFS_PKGMANAGE = "rpm zypper"
+
+# Add 50Meg of extra space for zypper database space
+ZYPPER_VAR_DB_SPACE = "51200"
+
 # Postinstalls on device are handled within this class at present
 ROOTFS_PKGMANAGE_BOOTSTRAP = ""
 
@@ -101,6 +105,7 @@ EOF
 	export IPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
 	export OPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
 
+
 	${ROOTFS_POSTINSTALL_COMMAND}
 
 	mkdir -p ${IMAGE_ROOTFS}/etc/rpm-postinsts/
-- 
1.7.3.4




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

* [PATCH 2/8] image.bbclass: add LINGUAS_INSTALL to depends list
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
  2011-07-07 23:31 ` [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-07 23:31 ` [PATCH 3/8] gnome-doc-utils: Add RDEPENDS on bash Saul Wold
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1215]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/classes/image.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index 1768a0b..edfb2d6 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -5,7 +5,7 @@ inherit imagetest-${IMAGETEST}
 
 LICENSE = "MIT"
 PACKAGES = ""
-RDEPENDS += "${IMAGE_INSTALL}"
+RDEPENDS += "${IMAGE_INSTALL} ${LINGUAS_INSTALL}"
 
 INHIBIT_DEFAULT_DEPS = "1"
 
-- 
1.7.3.4




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

* [PATCH 3/8] gnome-doc-utils: Add RDEPENDS on bash
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
  2011-07-07 23:31 ` [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper Saul Wold
  2011-07-07 23:31 ` [PATCH 2/8] image.bbclass: add LINGUAS_INSTALL to depends list Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-07 23:31 ` [PATCH 4/8] quilt: " Saul Wold
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1214]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-gnome/gnome/gnome-doc-utils.inc       |    2 ++
 meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb |    2 +-
 2 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-gnome/gnome/gnome-doc-utils.inc b/meta/recipes-gnome/gnome/gnome-doc-utils.inc
index 32fcf7a..7f1ab83 100644
--- a/meta/recipes-gnome/gnome/gnome-doc-utils.inc
+++ b/meta/recipes-gnome/gnome/gnome-doc-utils.inc
@@ -15,4 +15,6 @@ do_install_append() {
 
 FILES_${PN} += "${datadir}/xml*"
 
+RDEPENDS_${PN} = "bash"
+
 BBCLASSEXTEND = "native"
diff --git a/meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb b/meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb
index d85697f..0019f45 100644
--- a/meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb
+++ b/meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb
@@ -1,7 +1,7 @@
 require gnome-doc-utils.inc
 LIC_FILES_CHKSUM = "file://COPYING.GPL;md5=eb723b61539feef013de476e68b5c50a \
 		    file://COPYING.LGPL;md5=a6f89e2100d9b6cdffcea4f398e37343"
-PR = "r1"
+PR = "r2"
 
 SRC_URI += "file://xsltproc_nonet.patch"
 
-- 
1.7.3.4




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

* [PATCH 4/8] quilt: Add RDEPENDS on bash
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (2 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 3/8] gnome-doc-utils: Add RDEPENDS on bash Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-07 23:31 ` [PATCH 5/8] console-tools: " Saul Wold
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1214]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-devtools/quilt/quilt.inc     |    2 ++
 meta/recipes-devtools/quilt/quilt_0.48.bb |    2 +-
 2 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
index 4542511..3a66093 100644
--- a/meta/recipes-devtools/quilt/quilt.inc
+++ b/meta/recipes-devtools/quilt/quilt.inc
@@ -17,6 +17,8 @@ FILES_guards = "${bindir}/guards"
 FILES_${PN}-doc = "${mandir}/man1/quilt.1 ${docdir}/${P}"
 FILES_guards-doc = "${mandir}/man1/guards.1"
 
+RDEPENDS_${PN} = "bash"
+
 # quilt ignores DESTDIR
 do_install () {
 	oe_runmake 'BUILD_ROOT=${D}' install
diff --git a/meta/recipes-devtools/quilt/quilt_0.48.bb b/meta/recipes-devtools/quilt/quilt_0.48.bb
index 124e2cd..1600c62 100644
--- a/meta/recipes-devtools/quilt/quilt_0.48.bb
+++ b/meta/recipes-devtools/quilt/quilt_0.48.bb
@@ -2,7 +2,7 @@ require quilt.inc
 LICENSE="GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
 RDEPENDS_${PN} += "patch diffstat bzip2 util-linux"
-PR = "r1"
+PR = "r2"
 SRC_URI += "file://aclocal.patch \
             file://gnu_patch_test_fix_target.patch \
            "
-- 
1.7.3.4




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

* [PATCH 5/8] console-tools: Add RDEPENDS on bash
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (3 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 4/8] quilt: " Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-07 23:31 ` [PATCH 6/8] usbutils: " Saul Wold
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1214]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 .../console-tools/console-tools_0.3.2.bb           |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-core/console-tools/console-tools_0.3.2.bb b/meta/recipes-core/console-tools/console-tools_0.3.2.bb
index c740119..5b09105 100644
--- a/meta/recipes-core/console-tools/console-tools_0.3.2.bb
+++ b/meta/recipes-core/console-tools/console-tools_0.3.2.bb
@@ -3,7 +3,7 @@ LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING.kbd;md5=9b2d91511d3d80d4d20ac6e6b0137fe9"
 SUMMARY = "Allows you to set-up and manipulate the Linux console."
 DESCRIPTION = "Provides tools that enable the set-up and manipulation of the linux console and console-font files."
-PR = "r3"
+PR = "r4"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/lct/console-tools-${PV}.tar.gz \
            file://codepage.patch;patch=1 \
@@ -54,3 +54,4 @@ pkg_prerm_${PN} () {
 	update-alternatives --remove fgconsole fgconsole.${PN}
 }
 
+RDEPENDS_${PN} = "bash"
-- 
1.7.3.4




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

* [PATCH 6/8] usbutils: Add RDEPENDS on bash
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (4 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 5/8] console-tools: " Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-08  7:28   ` Koen Kooi
  2011-07-07 23:31 ` [PATCH 7/8] eglibc: " Saul Wold
                   ` (3 subsequent siblings)
  9 siblings, 1 reply; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1214]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-bsp/usbutils/usbutils_0.91.bb |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-bsp/usbutils/usbutils_0.91.bb b/meta/recipes-bsp/usbutils/usbutils_0.91.bb
index 5d605c5..c5420d0 100644
--- a/meta/recipes-bsp/usbutils/usbutils_0.91.bb
+++ b/meta/recipes-bsp/usbutils/usbutils_0.91.bb
@@ -7,8 +7,7 @@ LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
 
 DEPENDS = "libusb zlib"
-RDEPENDS_${PN} = "${PN}-ids"
-PR = "r1"
+PR = "r2"
 
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/usb/usbutils/usbutils-${PV}.tar.gz"
 
@@ -26,3 +25,5 @@ do_install_append() {
 PACKAGES += "${PN}-ids"
 FILES_${PN} += "${datadir}/pkgconfig"
 FILES_${PN}-ids = "${datadir}/usb*"
+
+RDEPENDS_${PN} = "${PN}-ids bash"
-- 
1.7.3.4




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

* [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (5 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 6/8] usbutils: " Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-08  0:59   ` Khem Raj
  2011-07-08  6:53   ` Frans Meulenbroeks
  2011-07-07 23:31 ` [PATCH 8/8] libx11: enable xcb support Saul Wold
                   ` (2 subsequent siblings)
  9 siblings, 2 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1214]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-core/eglibc/eglibc-package.inc |    2 +-
 meta/recipes-core/eglibc/eglibc.inc         |    2 +-
 meta/recipes-core/eglibc/eglibc_2.12.bb     |    2 +-
 meta/recipes-core/eglibc/eglibc_2.13.bb     |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/eglibc/eglibc-package.inc b/meta/recipes-core/eglibc/eglibc-package.inc
index 1c6626c..f0fac76 100644
--- a/meta/recipes-core/eglibc/eglibc-package.inc
+++ b/meta/recipes-core/eglibc/eglibc-package.inc
@@ -69,7 +69,7 @@ FILES_eglibc-utils = "${bindir}/* ${sbindir}/*"
 FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
 FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
 RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
-EDEPENDS_eglibc-utils = "libsotruss"
+RDEPENDS_eglibc-utils += "libsotruss bash"
 FILES_eglibc-pcprofile = "${base_libdir}/libpcprofile.so"
 FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db*"
 RPROVIDES_eglibc-dev += "libc-dev"
diff --git a/meta/recipes-core/eglibc/eglibc.inc b/meta/recipes-core/eglibc/eglibc.inc
index 74afb9d..058d58e 100644
--- a/meta/recipes-core/eglibc/eglibc.inc
+++ b/meta/recipes-core/eglibc/eglibc.inc
@@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend = " \
 # nptl needs unwind support in gcc, which can't be built without glibc.
 DEPENDS = "virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
 #this leads to circular deps, so lets not add it yet
-#RDEPENDS_ldd += " bash"
+RDEPENDS_ldd += " bash"
 # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
 #RDEPENDS_${PN} += "${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
 PROVIDES = "virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bb b/meta/recipes-core/eglibc/eglibc_2.12.bb
index 85d58fa..fd7b485 100644
--- a/meta/recipes-core/eglibc/eglibc_2.12.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
@@ -1,7 +1,7 @@
 require eglibc.inc
 
 DEPENDS += "gperf-native"
-PR = "r18"
+PR = "r19"
 
 SRCREV = "14158"
 
diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
index 7986131..be65787 100644
--- a/meta/recipes-core/eglibc/eglibc_2.13.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
@@ -4,7 +4,7 @@ SRCREV = "14157"
 
 DEPENDS += "gperf-native"
 FILESPATHPKG =. "eglibc-svn:"
-PR = "r5"
+PR = "r6"
 PR_append = "+svnr${SRCPV}"
 
 EGLIBC_BRANCH="eglibc-2_13"
-- 
1.7.3.4




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

* [PATCH 8/8] libx11: enable xcb support
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (6 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 7/8] eglibc: " Saul Wold
@ 2011-07-07 23:31 ` Saul Wold
  2011-07-08 13:57   ` Phil Blundell
  2011-07-18 22:55   ` Martin Jansa
  2011-07-08  6:49 ` [PATCH 0/8] Bug Fixes for M2 Rc2 Frans Meulenbroeks
  2011-07-08 15:41 ` Richard Purdie
  9 siblings, 2 replies; 25+ messages in thread
From: Saul Wold @ 2011-07-07 23:31 UTC (permalink / raw)
  To: openembedded-core

[YOCTO #1196]

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb b/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
index 3e046a1..04d4a97 100644
--- a/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
+++ b/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
@@ -5,7 +5,7 @@ LICENSE = "MIT & MIT-style & BSD"
 LIC_FILES_CHKSUM = "file://COPYING;md5=bf75bfe4d05068311b5e6862d4b5f2c5"
 
 PE = "1"
-PR = "r1"
+PR = "r3"
 
 SRC_URI += "file://x11_disable_makekeys.patch \
             file://nodolt.patch \
@@ -16,8 +16,13 @@ SRC_URI[md5sum] = "f65c9c7ecbfb64c19dbd7927160d63fd"
 SRC_URI[sha256sum] = "88d7238ce5f7cd123450567de7a3b56a43556e4ccc45df38b8324147c889a844"
 
 DEPENDS += "bigreqsproto xproto xextproto xtrans libxau xcmiscproto \
-            libxdmcp xf86bigfontproto kbproto inputproto xproto-native"
+            libxdmcp xf86bigfontproto kbproto inputproto xproto-native libxcb"
 
-EXTRA_OECONF += "--without-xcb"
+DEPENDS_virtclass-native += "bigreqsproto-native xproto-native xextproto-native xtrans-native libxau-native xcmiscproto-native \
+            libxdmcp-native xf86bigfontproto-native kbproto-native inputproto-native xproto-native"
+
+XCB = "--with-xcb"
+XCB_virtclass-native = "--without-xcb"
+EXTRA_OECONF += "${XCB}"
 
 BBCLASSEXTEND = "native nativesdk"
-- 
1.7.3.4




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

* Re: [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper
  2011-07-07 23:31 ` [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper Saul Wold
@ 2011-07-07 23:38   ` Richard Purdie
  2011-07-08  5:42     ` Saul Wold
  0 siblings, 1 reply; 25+ messages in thread
From: Richard Purdie @ 2011-07-07 23:38 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2011-07-07 at 16:31 -0700, Saul Wold wrote:
> [YOCTO #1171]
> 
> Add /var space for zypper due to its space usage for db maintence

This hasn't been tested for opkg/deb and is missing something like:

ZYPPER_VAR_DB_SPACE ??= 0

I'd suggest we just change this to append " + 51200" to
IMAGE_ROOTFS_EXTRA_SPACE. I'd also like this only to happen when we're
actually installing zypper so the minimal images are unaffected...

Cheers,

Richard





> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta/classes/image_types.bbclass |    2 +-
>  meta/classes/rootfs_rpm.bbclass  |    5 +++++
>  2 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
> index 8c86227..505342d 100644
> --- a/meta/classes/image_types.bbclass
> +++ b/meta/classes/image_types.bbclass
> @@ -13,7 +13,7 @@ def get_imagecmds(d):
>  
>  runimagecmd () {
>  	# Image generation code for image type ${type}
> -	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE}; OFMT = "%.0f" ; print (size > ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
> +	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE} + ${ZYPPER_VAR_DB_SPACE}; OFMT = "%.0f" ; print (size > ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
>  	${cmd}
>  	cd ${DEPLOY_DIR_IMAGE}/
>  	rm -f ${DEPLOY_DIR_IMAGE}/${IMAGE_LINK_NAME}.${type}
> diff --git a/meta/classes/rootfs_rpm.bbclass b/meta/classes/rootfs_rpm.bbclass
> index 3a11858..67c547f 100644
> --- a/meta/classes/rootfs_rpm.bbclass
> +++ b/meta/classes/rootfs_rpm.bbclass
> @@ -3,6 +3,10 @@
>  #
>  
>  ROOTFS_PKGMANAGE = "rpm zypper"
> +
> +# Add 50Meg of extra space for zypper database space
> +ZYPPER_VAR_DB_SPACE = "51200"
> +
>  # Postinstalls on device are handled within this class at present
>  ROOTFS_PKGMANAGE_BOOTSTRAP = ""
>  
> @@ -101,6 +105,7 @@ EOF
>  	export IPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
>  	export OPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
>  
> +
>  	${ROOTFS_POSTINSTALL_COMMAND}
>  
>  	mkdir -p ${IMAGE_ROOTFS}/etc/rpm-postinsts/





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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-07 23:31 ` [PATCH 7/8] eglibc: " Saul Wold
@ 2011-07-08  0:59   ` Khem Raj
  2011-07-08  1:15     ` Saul Wold
  2011-07-08  6:53   ` Frans Meulenbroeks
  1 sibling, 1 reply; 25+ messages in thread
From: Khem Raj @ 2011-07-08  0:59 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
  Cc: openembedded-core@lists.openembedded.org



On Jul 7, 2011, at 4:31 PM, Saul Wold <sgw@linux.intel.com> wrote:

> [YOCTO #1214]
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
> meta/recipes-core/eglibc/eglibc-package.inc |    2 +-
> meta/recipes-core/eglibc/eglibc.inc         |    2 +-
> meta/recipes-core/eglibc/eglibc_2.12.bb     |    2 +-
> meta/recipes-core/eglibc/eglibc_2.13.bb     |    2 +-
> 4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-core/eglibc/eglibc-package.inc b/meta/recipes-core/eglibc/eglibc-package.inc
> index 1c6626c..f0fac76 100644
> --- a/meta/recipes-core/eglibc/eglibc-package.inc
> +++ b/meta/recipes-core/eglibc/eglibc-package.inc
> @@ -69,7 +69,7 @@ FILES_eglibc-utils = "${bindir}/* ${sbindir}/*"
> FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
> FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
> RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
> -EDEPENDS_eglibc-utils = "libsotruss"
> +RDEPENDS_eglibc-utils += "libsotruss bash"

While typo fix is ok why do we need dependency on bash ?
If it is for ldd then there is already a fix to free it from bashism
We should use that

> FILES_eglibc-pcprofile = "${base_libdir}/libpcprofile.so"
> FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db*"
> RPROVIDES_eglibc-dev += "libc-dev"
> diff --git a/meta/recipes-core/eglibc/eglibc.inc b/meta/recipes-core/eglibc/eglibc.inc
> index 74afb9d..058d58e 100644
> --- a/meta/recipes-core/eglibc/eglibc.inc
> +++ b/meta/recipes-core/eglibc/eglibc.inc
> @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend = " \
> # nptl needs unwind support in gcc, which can't be built without glibc.
> DEPENDS = "virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
> #this leads to circular deps, so lets not add it yet
> -#RDEPENDS_ldd += " bash"
> +RDEPENDS_ldd += " bash"
> # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
> #RDEPENDS_${PN} += "${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
> PROVIDES = "virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
> diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bb b/meta/recipes-core/eglibc/eglibc_2.12.bb
> index 85d58fa..fd7b485 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
> @@ -1,7 +1,7 @@
> require eglibc.inc
> 
> DEPENDS += "gperf-native"
> -PR = "r18"
> +PR = "r19"
> 
> SRCREV = "14158"
> 
> diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
> index 7986131..be65787 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
> @@ -4,7 +4,7 @@ SRCREV = "14157"
> 
> DEPENDS += "gperf-native"
> FILESPATHPKG =. "eglibc-svn:"
> -PR = "r5"
> +PR = "r6"
> PR_append = "+svnr${SRCPV}"
> 
> EGLIBC_BRANCH="eglibc-2_13"
> -- 
> 1.7.3.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-08  0:59   ` Khem Raj
@ 2011-07-08  1:15     ` Saul Wold
  2011-07-08  5:27       ` Khem Raj
  0 siblings, 1 reply; 25+ messages in thread
From: Saul Wold @ 2011-07-08  1:15 UTC (permalink / raw)
  To: openembedded-core

On 07/07/2011 05:59 PM, Khem Raj wrote:
>
>
> On Jul 7, 2011, at 4:31 PM, Saul Wold<sgw@linux.intel.com>  wrote:
>
>> [YOCTO #1214]
>>
>> Signed-off-by: Saul Wold<sgw@linux.intel.com>
>> ---
>> meta/recipes-core/eglibc/eglibc-package.inc |    2 +-
>> meta/recipes-core/eglibc/eglibc.inc         |    2 +-
>> meta/recipes-core/eglibc/eglibc_2.12.bb     |    2 +-
>> meta/recipes-core/eglibc/eglibc_2.13.bb     |    2 +-
>> 4 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/recipes-core/eglibc/eglibc-package.inc b/meta/recipes-core/eglibc/eglibc-package.inc
>> index 1c6626c..f0fac76 100644
>> --- a/meta/recipes-core/eglibc/eglibc-package.inc
>> +++ b/meta/recipes-core/eglibc/eglibc-package.inc
>> @@ -69,7 +69,7 @@ FILES_eglibc-utils = "${bindir}/* ${sbindir}/*"
>> FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
>> FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
>> RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
>> -EDEPENDS_eglibc-utils = "libsotruss"
>> +RDEPENDS_eglibc-utils += "libsotruss bash"
>
> While typo fix is ok why do we need dependency on bash ?
> If it is for ldd then there is already a fix to free it from bashism
> We should use that
>
Actually it's for xtrace and tzselect which are part of eglibc-utils and 
are scripts that use /bin/bash.

Where's the fix for ldd?  I don't think its part of eglibc currently.

Sau!

>> FILES_eglibc-pcprofile = "${base_libdir}/libpcprofile.so"
>> FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db*"
>> RPROVIDES_eglibc-dev += "libc-dev"
>> diff --git a/meta/recipes-core/eglibc/eglibc.inc b/meta/recipes-core/eglibc/eglibc.inc
>> index 74afb9d..058d58e 100644
>> --- a/meta/recipes-core/eglibc/eglibc.inc
>> +++ b/meta/recipes-core/eglibc/eglibc.inc
>> @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend = " \
>> # nptl needs unwind support in gcc, which can't be built without glibc.
>> DEPENDS = "virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
>> #this leads to circular deps, so lets not add it yet
>> -#RDEPENDS_ldd += " bash"
>> +RDEPENDS_ldd += " bash"
>> # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
>> #RDEPENDS_${PN} += "${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
>> PROVIDES = "virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
>> diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bb b/meta/recipes-core/eglibc/eglibc_2.12.bb
>> index 85d58fa..fd7b485 100644
>> --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
>> +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
>> @@ -1,7 +1,7 @@
>> require eglibc.inc
>>
>> DEPENDS += "gperf-native"
>> -PR = "r18"
>> +PR = "r19"
>>
>> SRCREV = "14158"
>>
>> diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
>> index 7986131..be65787 100644
>> --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
>> +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
>> @@ -4,7 +4,7 @@ SRCREV = "14157"
>>
>> DEPENDS += "gperf-native"
>> FILESPATHPKG =. "eglibc-svn:"
>> -PR = "r5"
>> +PR = "r6"
>> PR_append = "+svnr${SRCPV}"
>>
>> EGLIBC_BRANCH="eglibc-2_13"
>> --
>> 1.7.3.4
>>
>>
>> _______________________________________________
>> 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] 25+ messages in thread

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-08  1:15     ` Saul Wold
@ 2011-07-08  5:27       ` Khem Raj
  2011-07-12 13:38         ` Phil Blundell
  0 siblings, 1 reply; 25+ messages in thread
From: Khem Raj @ 2011-07-08  5:27 UTC (permalink / raw)
  To: openembedded-core

On 07/07/2011 06:15 PM, Saul Wold wrote:
> On 07/07/2011 05:59 PM, Khem Raj wrote:
>>
>>
>> On Jul 7, 2011, at 4:31 PM, Saul Wold<sgw@linux.intel.com> wrote:
>>
>>> [YOCTO #1214]
>>>
>>> Signed-off-by: Saul Wold<sgw@linux.intel.com>
>>> ---
>>> meta/recipes-core/eglibc/eglibc-package.inc | 2 +-
>>> meta/recipes-core/eglibc/eglibc.inc | 2 +-
>>> meta/recipes-core/eglibc/eglibc_2.12.bb | 2 +-
>>> meta/recipes-core/eglibc/eglibc_2.13.bb | 2 +-
>>> 4 files changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/meta/recipes-core/eglibc/eglibc-package.inc
>>> b/meta/recipes-core/eglibc/eglibc-package.inc
>>> index 1c6626c..f0fac76 100644
>>> --- a/meta/recipes-core/eglibc/eglibc-package.inc
>>> +++ b/meta/recipes-core/eglibc/eglibc-package.inc
>>> @@ -69,7 +69,7 @@ FILES_eglibc-utils = "${bindir}/* ${sbindir}/*"
>>> FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
>>> FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
>>> RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
>>> -EDEPENDS_eglibc-utils = "libsotruss"
>>> +RDEPENDS_eglibc-utils += "libsotruss bash"
>>
>> While typo fix is ok why do we need dependency on bash ?
>> If it is for ldd then there is already a fix to free it from bashism
>> We should use that
>>
> Actually it's for xtrace and tzselect which are part of eglibc-utils and
> are scripts that use /bin/bash.

hmm I see, see if it can be unbashed if not then the dep is ok.

>
> Where's the fix for ldd? I don't think its part of eglibc currently.

0a831e2cd1c27d30cd76ddada9a44a703ab60f41 in oe.dev

>
> Sau!
>
>>> FILES_eglibc-pcprofile = "${base_libdir}/libpcprofile.so"
>>> FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db*"
>>> RPROVIDES_eglibc-dev += "libc-dev"
>>> diff --git a/meta/recipes-core/eglibc/eglibc.inc
>>> b/meta/recipes-core/eglibc/eglibc.inc
>>> index 74afb9d..058d58e 100644
>>> --- a/meta/recipes-core/eglibc/eglibc.inc
>>> +++ b/meta/recipes-core/eglibc/eglibc.inc
>>> @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend = " \
>>> # nptl needs unwind support in gcc, which can't be built without glibc.
>>> DEPENDS = "virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
>>> #this leads to circular deps, so lets not add it yet
>>> -#RDEPENDS_ldd += " bash"
>>> +RDEPENDS_ldd += " bash"
>>> # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect
>>> this
>>> #RDEPENDS_${PN} += "${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
>>> PROVIDES = "virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
>>> diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bb
>>> b/meta/recipes-core/eglibc/eglibc_2.12.bb
>>> index 85d58fa..fd7b485 100644
>>> --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
>>> +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
>>> @@ -1,7 +1,7 @@
>>> require eglibc.inc
>>>
>>> DEPENDS += "gperf-native"
>>> -PR = "r18"
>>> +PR = "r19"
>>>
>>> SRCREV = "14158"
>>>
>>> diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb
>>> b/meta/recipes-core/eglibc/eglibc_2.13.bb
>>> index 7986131..be65787 100644
>>> --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
>>> +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
>>> @@ -4,7 +4,7 @@ SRCREV = "14157"
>>>
>>> DEPENDS += "gperf-native"
>>> FILESPATHPKG =. "eglibc-svn:"
>>> -PR = "r5"
>>> +PR = "r6"
>>> PR_append = "+svnr${SRCPV}"
>>>
>>> EGLIBC_BRANCH="eglibc-2_13"
>>> --
>>> 1.7.3.4
>>>
>>>
>>> _______________________________________________
>>> 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] 25+ messages in thread

* Re: [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper
  2011-07-07 23:38   ` Richard Purdie
@ 2011-07-08  5:42     ` Saul Wold
  2011-07-08 14:52       ` Richard Purdie
  0 siblings, 1 reply; 25+ messages in thread
From: Saul Wold @ 2011-07-08  5:42 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 07/07/2011 04:38 PM, Richard Purdie wrote:
> On Thu, 2011-07-07 at 16:31 -0700, Saul Wold wrote:
>> [YOCTO #1171]
>>
>> Add /var space for zypper due to its space usage for db maintence
>
> This hasn't been tested for opkg/deb and is missing something like:
>
> ZYPPER_VAR_DB_SPACE ??= 0
>
> I'd suggest we just change this to append " + 51200" to
> IMAGE_ROOTFS_EXTRA_SPACE. I'd also like this only to happen when we're
> actually installing zypper so the minimal images are unaffected...
>
I can do the + 51200, but I have not figured out how to do the check for 
zypper, maybe it's something straight forward, but I am not finding a 
way to do it easily.

Sau!

> Cheers,
>
> Richard
>
>
>
>
>
>> Signed-off-by: Saul Wold<sgw@linux.intel.com>
>> ---
>>   meta/classes/image_types.bbclass |    2 +-
>>   meta/classes/rootfs_rpm.bbclass  |    5 +++++
>>   2 files changed, 6 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
>> index 8c86227..505342d 100644
>> --- a/meta/classes/image_types.bbclass
>> +++ b/meta/classes/image_types.bbclass
>> @@ -13,7 +13,7 @@ def get_imagecmds(d):
>>
>>   runimagecmd () {
>>   	# Image generation code for image type ${type}
>> -	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE}; OFMT = "%.0f" ; print (size>  ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
>> +	ROOTFS_SIZE=`du -ks ${IMAGE_ROOTFS}|awk '{size = $1 * ${IMAGE_OVERHEAD_FACTOR} + ${IMAGE_ROOTFS_EXTRA_SPACE} + ${ZYPPER_VAR_DB_SPACE}; OFMT = "%.0f" ; print (size>  ${IMAGE_ROOTFS_SIZE} ? size : ${IMAGE_ROOTFS_SIZE}) }'`
>>   	${cmd}
>>   	cd ${DEPLOY_DIR_IMAGE}/
>>   	rm -f ${DEPLOY_DIR_IMAGE}/${IMAGE_LINK_NAME}.${type}
>> diff --git a/meta/classes/rootfs_rpm.bbclass b/meta/classes/rootfs_rpm.bbclass
>> index 3a11858..67c547f 100644
>> --- a/meta/classes/rootfs_rpm.bbclass
>> +++ b/meta/classes/rootfs_rpm.bbclass
>> @@ -3,6 +3,10 @@
>>   #
>>
>>   ROOTFS_PKGMANAGE = "rpm zypper"
>> +
>> +# Add 50Meg of extra space for zypper database space
>> +ZYPPER_VAR_DB_SPACE = "51200"
>> +
>>   # Postinstalls on device are handled within this class at present
>>   ROOTFS_PKGMANAGE_BOOTSTRAP = ""
>>
>> @@ -101,6 +105,7 @@ EOF
>>   	export IPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
>>   	export OPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
>>
>> +
>>   	${ROOTFS_POSTINSTALL_COMMAND}
>>
>>   	mkdir -p ${IMAGE_ROOTFS}/etc/rpm-postinsts/
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>



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

* Re: [PATCH 0/8] Bug Fixes for M2 Rc2
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (7 preceding siblings ...)
  2011-07-07 23:31 ` [PATCH 8/8] libx11: enable xcb support Saul Wold
@ 2011-07-08  6:49 ` Frans Meulenbroeks
  2011-07-08 15:41 ` Richard Purdie
  9 siblings, 0 replies; 25+ messages in thread
From: Frans Meulenbroeks @ 2011-07-08  6:49 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 2497 bytes --]

2011/7/8 Saul Wold <sgw@linux.intel.com>

> Richard,
>
> This address the bash, xcb missing from libx11, locale issue and
> the lack of space due to zypper eating space.
>

I don't think all the dependencies on bash are needed.

E.g:  eglibc: Add RDEPENDS on bash
eglibc itself does not need bash to be present at runtime (or maybe only in
some eglibc configurations).
I am currently happily running an eglibc system with busybox sh without any
problem at all.
Footprintwise for embedded systems that is desirable and as such it is quite
a common setup.

Frans.



>
> Sau!
>
>
> The following changes since commit
> f1fc6d084b079dea21ff1a30b815496452042490:
>
>  pulseaudio: add 0.9.23 (2011-07-07 13:44:36 +0100)
>
> are available in the git repository at:
>  git://git.openembedded.org/openembedded-core-contrib sgw/fix
>
> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/fix
>
> Saul Wold (8):
>  rootfs_rpm: Add 50Meg to image size for zypper
>  image.bbclass: add LINGUAS_INSTALL to depends list
>  gnome-doc-utils: Add RDEPENDS on bash
>  quilt: Add RDEPENDS on bash
>  console-tools: Add RDEPENDS on bash
>  usbutils: Add RDEPENDS on bash
>  eglibc: Add RDEPENDS on bash
>  libx11: enable xcb support
>
>  meta/classes/image.bbclass                         |    2 +-
>  meta/classes/image_types.bbclass                   |    2 +-
>  meta/classes/rootfs_rpm.bbclass                    |    5 +++++
>  meta/recipes-bsp/usbutils/usbutils_0.91.bb         |    5 +++--
>  .../console-tools/console-tools_0.3.2.bb           |    3 ++-
>  meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
>  meta/recipes-core/eglibc/eglibc.inc                |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.13.bb            |    2 +-
>  meta/recipes-devtools/quilt/quilt.inc              |    2 ++
>  meta/recipes-devtools/quilt/quilt_0.48.bb          |    2 +-
>  meta/recipes-gnome/gnome/gnome-doc-utils.inc       |    2 ++
>  meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb |    2 +-
>  meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb     |   11 ++++++++---
>  14 files changed, 30 insertions(+), 14 deletions(-)
>
> --
> 1.7.3.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 3815 bytes --]

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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-07 23:31 ` [PATCH 7/8] eglibc: " Saul Wold
  2011-07-08  0:59   ` Khem Raj
@ 2011-07-08  6:53   ` Frans Meulenbroeks
  2011-07-08  7:01     ` Koen Kooi
  1 sibling, 1 reply; 25+ messages in thread
From: Frans Meulenbroeks @ 2011-07-08  6:53 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 3133 bytes --]

2011/7/8 Saul Wold <sgw@linux.intel.com>

> [YOCTO #1214]
>
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta/recipes-core/eglibc/eglibc-package.inc |    2 +-
>  meta/recipes-core/eglibc/eglibc.inc         |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.12.bb     |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.13.bb     |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/eglibc/eglibc-package.inc
> b/meta/recipes-core/eglibc/eglibc-package.inc
> index 1c6626c..f0fac76 100644
> --- a/meta/recipes-core/eglibc/eglibc-package.inc
> +++ b/meta/recipes-core/eglibc/eglibc-package.inc
> @@ -69,7 +69,7 @@ FILES_eglibc-utils = "${bindir}/* ${sbindir}/*"
>  FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
>  FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
>  RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
> -EDEPENDS_eglibc-utils = "libsotruss"
> +RDEPENDS_eglibc-utils += "libsotruss bash"
>  FILES_eglibc-pcprofile = "${base_libdir}/libpcprofile.so"
>  FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db*"
>  RPROVIDES_eglibc-dev += "libc-dev"
> diff --git a/meta/recipes-core/eglibc/eglibc.inc
> b/meta/recipes-core/eglibc/eglibc.inc
> index 74afb9d..058d58e 100644
> --- a/meta/recipes-core/eglibc/eglibc.inc
> +++ b/meta/recipes-core/eglibc/eglibc.inc
> @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend = " \
>  # nptl needs unwind support in gcc, which can't be built without glibc.
>  DEPENDS = "virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
>  #this leads to circular deps, so lets not add it yet
> -#RDEPENDS_ldd += " bash"
> +RDEPENDS_ldd += " bash"
>

Ah ok, now see that this is only for ldd.
In that case the commit message is misleading. (and probably also not
conformant with the commit message policy).

Frans


>  # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
>  #RDEPENDS_${PN} += "${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
>  PROVIDES = "virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
> diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bbb/meta/recipes-core/eglibc/
> eglibc_2.12.bb
> index 85d58fa..fd7b485 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
> @@ -1,7 +1,7 @@
>  require eglibc.inc
>
>  DEPENDS += "gperf-native"
> -PR = "r18"
> +PR = "r19"
>
>  SRCREV = "14158"
>
> diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bbb/meta/recipes-core/eglibc/
> eglibc_2.13.bb
> index 7986131..be65787 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
> @@ -4,7 +4,7 @@ SRCREV = "14157"
>
>  DEPENDS += "gperf-native"
>  FILESPATHPKG =. "eglibc-svn:"
> -PR = "r5"
> +PR = "r6"
>  PR_append = "+svnr${SRCPV}"
>
>  EGLIBC_BRANCH="eglibc-2_13"
> --
> 1.7.3.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 4741 bytes --]

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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-08  6:53   ` Frans Meulenbroeks
@ 2011-07-08  7:01     ` Koen Kooi
  2011-07-08  7:23       ` Frans Meulenbroeks
  0 siblings, 1 reply; 25+ messages in thread
From: Koen Kooi @ 2011-07-08  7:01 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
  Cc: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 3483 bytes --]



Op 8 jul. 2011 om 07:53 heeft Frans Meulenbroeks <fransmeulenbroeks@gmail.com> het volgende geschreven:

> 
> 
> 2011/7/8 Saul Wold <sgw@linux.intel.com>
> [YOCTO #1214]
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta/recipes-core/eglibc/eglibc-package.inc |    2 +-
>  meta/recipes-core/eglibc/eglibc.inc         |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.12.bb     |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.13.bb     |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-core/eglibc/eglibc-package.inc b/meta/recipes-core/eglibc/eglibc-package.inc
> index 1c6626c..f0fac76 100644
> --- a/meta/recipes-core/eglibc/eglibc-package.inc
> +++ b/meta/recipes-core/eglibc/eglibc-package.inc
> @@ -69,7 +69,7 @@ FILES_eglibc-utils ="${bindir}/* ${sbindir}/*"
>  FILES_${PN}-dbg +="${libexecdir}/*/.debug ${libdir}/audit/.debug"
>  FILES_catchsegv${PKGSUFFIX} ="${bindir}/catchsegv"
>  RDEPENDS_catchsegv${PKGSUFFIX} ="libsegfault"
> -EDEPENDS_eglibc-utils ="libsotruss"
> +RDEPENDS_eglibc-utils +="libsotruss bash"
>  FILES_eglibc-pcprofile ="${base_libdir}/libpcprofile.so"
>  FILES_eglibc-thread-db${PKGSUFFIX} ="${base_libdir}/libthread_db*"
>  RPROVIDES_eglibc-dev +="libc-dev"
> diff --git a/meta/recipes-core/eglibc/eglibc.inc b/meta/recipes-core/eglibc/eglibc.inc
> index 74afb9d..058d58e 100644
> --- a/meta/recipes-core/eglibc/eglibc.inc
> +++ b/meta/recipes-core/eglibc/eglibc.inc
> @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend =" \
>  # nptl needs unwind support in gcc, which can't be built without glibc.
>  DEPENDS ="virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers"
>  #this leads to circular deps, so lets not add it yet
> -#RDEPENDS_ldd +=" bash"
> +RDEPENDS_ldd +=" bash"
> 
> Ah ok, now see that this is only for ldd.
> In that case the commit message is misleading. (and probably also not conformant with the commit message policy).
> 

there's an ldd-unbash patch in . dev




> Frans
>  
>  # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
>  #RDEPENDS_${PN} +="${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}"
>  PROVIDES ="virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc"
> diff --git a/meta/recipes-core/eglibc/eglibc_2.12.bb b/meta/recipes-core/eglibc/eglibc_2.12.bb
> index 85d58fa..fd7b485 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
> @@ -1,7 +1,7 @@
>  require eglibc.inc
> 
>  DEPENDS +="gperf-native"
> -PR ="r18"
> +PR ="r19"
> 
>  SRCREV ="14158"
> 
> diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
> index 7986131..be65787 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
> @@ -4,7 +4,7 @@ SRCREV ="14157"
> 
>  DEPENDS +="gperf-native"
>  FILESPATHPKG =. "eglibc-svn:"
> -PR ="r5"
> +PR ="r6"
>  PR_append ="+svnr${SRCPV}"
> 
>  EGLIBC_BRANCH="eglibc-2_13"
> --
> 1.7.3.4
> 
> 
> _______________________________________________
> 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

[-- Attachment #2: Type: text/html, Size: 5794 bytes --]

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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-08  7:01     ` Koen Kooi
@ 2011-07-08  7:23       ` Frans Meulenbroeks
  0 siblings, 0 replies; 25+ messages in thread
From: Frans Meulenbroeks @ 2011-07-08  7:23 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 518 bytes --]

2011/7/8 Koen Kooi <koen@dominion.thruhere.net>

>
>
> Op 8 jul. 2011 om 07:53 heeft Frans Meulenbroeks <
> fransmeulenbroeks@gmail.com> het volgende geschreven:
>
>
>> -#RDEPENDS_ldd +=" bash"
>> +RDEPENDS_ldd +=" bash"
>>
>
> Ah ok, now see that this is only for ldd.
> In that case the commit message is misleading. (and probably also not
> conformant with the commit message policy).
>
>
> there's an ldd-unbash patch in . dev
>
>
> Seems to me that that is a better alternative then

Frans

[-- Attachment #2: Type: text/html, Size: 1277 bytes --]

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

* Re: [PATCH 6/8] usbutils: Add RDEPENDS on bash
  2011-07-07 23:31 ` [PATCH 6/8] usbutils: " Saul Wold
@ 2011-07-08  7:28   ` Koen Kooi
  2011-07-08 15:42     ` Richard Purdie
  0 siblings, 1 reply; 25+ messages in thread
From: Koen Kooi @ 2011-07-08  7:28 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
  Cc: openembedded-core@lists.openembedded.org

iirc only the ids package depends on bash, and only to check for wget. Fixing the update script would be better

Op 8 jul. 2011 om 00:31 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven:

> [YOCTO #1214]
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
> meta/recipes-bsp/usbutils/usbutils_0.91.bb |    5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-bsp/usbutils/usbutils_0.91.bb b/meta/recipes-bsp/usbutils/usbutils_0.91.bb
> index 5d605c5..c5420d0 100644
> --- a/meta/recipes-bsp/usbutils/usbutils_0.91.bb
> +++ b/meta/recipes-bsp/usbutils/usbutils_0.91.bb
> @@ -7,8 +7,7 @@ LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
> 
> DEPENDS = "libusb zlib"
> -RDEPENDS_${PN} = "${PN}-ids"
> -PR = "r1"
> +PR = "r2"
> 
> SRC_URI = "${KERNELORG_MIRROR}/linux/utils/usb/usbutils/usbutils-${PV}.tar.gz"
> 
> @@ -26,3 +25,5 @@ do_install_append() {
> PACKAGES += "${PN}-ids"
> FILES_${PN} += "${datadir}/pkgconfig"
> FILES_${PN}-ids = "${datadir}/usb*"
> +
> +RDEPENDS_${PN} = "${PN}-ids bash"
> -- 
> 1.7.3.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



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

* Re: [PATCH 8/8] libx11: enable xcb support
  2011-07-07 23:31 ` [PATCH 8/8] libx11: enable xcb support Saul Wold
@ 2011-07-08 13:57   ` Phil Blundell
  2011-07-18 22:55   ` Martin Jansa
  1 sibling, 0 replies; 25+ messages in thread
From: Phil Blundell @ 2011-07-08 13:57 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2011-07-07 at 16:31 -0700, Saul Wold wrote:
> [YOCTO #1196]
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>

We seem to be seeing quite a lot of patches recently whose long-form
comment consists of a Yocto bug number, S-O-B line and nothing else.
That might be fine for the most notorious bugs, where even non-Yocto
people would immediately recognise the number, but in general it's a bit
tedious to keep having to go to the Yocto bugzilla to find out what the
patch is trying to do.  Would it be possible to get a slightly more
descriptive commit message for future changes?

See also the fine guidelines from our helpful TSC:

http://wiki.openembedded.org/index.php/Commit_Patch_Message_Guidelines

thanks

p.





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

* Re: [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper
  2011-07-08  5:42     ` Saul Wold
@ 2011-07-08 14:52       ` Richard Purdie
  0 siblings, 0 replies; 25+ messages in thread
From: Richard Purdie @ 2011-07-08 14:52 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2011-07-07 at 22:42 -0700, Saul Wold wrote:
> On 07/07/2011 04:38 PM, Richard Purdie wrote:
> > On Thu, 2011-07-07 at 16:31 -0700, Saul Wold wrote:
> >> [YOCTO #1171]
> >>
> >> Add /var space for zypper due to its space usage for db maintence
> >
> > This hasn't been tested for opkg/deb and is missing something like:
> >
> > ZYPPER_VAR_DB_SPACE ??= 0
> >
> > I'd suggest we just change this to append " + 51200" to
> > IMAGE_ROOTFS_EXTRA_SPACE. I'd also like this only to happen when we're
> > actually installing zypper so the minimal images are unaffected...
> >
> I can do the + 51200, but I have not figured out how to do the check for 
> zypper, maybe it's something straight forward, but I am not finding a 
> way to do it easily.

How about something like:

IMAGE_ROOTFS_EXTRA_SPACE += "${@base_contains("PACKAGE_INSTALL", "zypper", " + 51200", "" ,d)}"

?

Not perfect but should basically work for what we need until we find a
bettr fix for zypper itself.

Cheers,

Richard




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

* Re: [PATCH 0/8] Bug Fixes for M2 Rc2
  2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
                   ` (8 preceding siblings ...)
  2011-07-08  6:49 ` [PATCH 0/8] Bug Fixes for M2 Rc2 Frans Meulenbroeks
@ 2011-07-08 15:41 ` Richard Purdie
  9 siblings, 0 replies; 25+ messages in thread
From: Richard Purdie @ 2011-07-08 15:41 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2011-07-07 at 16:31 -0700, Saul Wold wrote:
> Richard,
> 
> This address the bash, xcb missing from libx11, locale issue and
> the lack of space due to zypper eating space.
> 
> Sau!
> 
> 
> The following changes since commit f1fc6d084b079dea21ff1a30b815496452042490:
> 
>   pulseaudio: add 0.9.23 (2011-07-07 13:44:36 +0100)
> 
> are available in the git repository at:
>   git://git.openembedded.org/openembedded-core-contrib sgw/fix
>   http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/fix
> 
> Saul Wold (8):
>  
>   image.bbclass: add LINGUAS_INSTALL to depends list
>   gnome-doc-utils: Add RDEPENDS on bash
>   quilt: Add RDEPENDS on bash
>   console-tools: Add RDEPENDS on bash
>   usbutils: Add RDEPENDS on bash

I've taken these (rewording the image.bbclass commit).

>   rootfs_rpm: Add 50Meg to image size for zypper

This needs tweaking as per the discussion

>   eglibc: Add RDEPENDS on bash

We should likely take OE's ldd patch here and then this can go in.

>   libx11: enable xcb support

and this needs a revised commit message.

Cheers,

Richard




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

* Re: [PATCH 6/8] usbutils: Add RDEPENDS on bash
  2011-07-08  7:28   ` Koen Kooi
@ 2011-07-08 15:42     ` Richard Purdie
  0 siblings, 0 replies; 25+ messages in thread
From: Richard Purdie @ 2011-07-08 15:42 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Fri, 2011-07-08 at 08:28 +0100, Koen Kooi wrote:
> iirc only the ids package depends on bash, and only to check for wget. Fixing the update script would be better

Agreed and I'll take a follow up patch on this one.

The fix in itself is correct as the script is wanting bash currently.

Cheers,

Richard





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

* Re: [PATCH 7/8] eglibc: Add RDEPENDS on bash
  2011-07-08  5:27       ` Khem Raj
@ 2011-07-12 13:38         ` Phil Blundell
  0 siblings, 0 replies; 25+ messages in thread
From: Phil Blundell @ 2011-07-12 13:38 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2011-07-07 at 22:27 -0700, Khem Raj wrote:
> On 07/07/2011 06:15 PM, Saul Wold wrote:
> > On 07/07/2011 05:59 PM, Khem Raj wrote:
> >> While typo fix is ok why do we need dependency on bash ?
> >> If it is for ldd then there is already a fix to free it from bashism
> >> We should use that
> >
> > Where's the fix for ldd? I don't think its part of eglibc currently.
> 
> 0a831e2cd1c27d30cd76ddada9a44a703ab60f41 in oe.dev

As we discussed on IRC the other day, that patch is not really a
complete fix.  All it does is change the #!/bin/bash to #!/bin/sh
without addressing the parts of the script that actually use
bash-specific syntax.  In particular there are quite a lot of places
that use the $"string" gettext shorthand.

p.





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

* Re: [PATCH 8/8] libx11: enable xcb support
  2011-07-07 23:31 ` [PATCH 8/8] libx11: enable xcb support Saul Wold
  2011-07-08 13:57   ` Phil Blundell
@ 2011-07-18 22:55   ` Martin Jansa
  1 sibling, 0 replies; 25+ messages in thread
From: Martin Jansa @ 2011-07-18 22:55 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 1937 bytes --]

On Thu, Jul 07, 2011 at 04:31:40PM -0700, Saul Wold wrote:
> [YOCTO #1196]
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb |   11 ++++++++---
>  1 files changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb b/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
> index 3e046a1..04d4a97 100644
> --- a/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
> +++ b/meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb
> @@ -5,7 +5,7 @@ LICENSE = "MIT & MIT-style & BSD"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=bf75bfe4d05068311b5e6862d4b5f2c5"
>  
>  PE = "1"
> -PR = "r1"
> +PR = "r3"
>  
>  SRC_URI += "file://x11_disable_makekeys.patch \
>              file://nodolt.patch \
> @@ -16,8 +16,13 @@ SRC_URI[md5sum] = "f65c9c7ecbfb64c19dbd7927160d63fd"
>  SRC_URI[sha256sum] = "88d7238ce5f7cd123450567de7a3b56a43556e4ccc45df38b8324147c889a844"
>  
>  DEPENDS += "bigreqsproto xproto xextproto xtrans libxau xcmiscproto \
> -            libxdmcp xf86bigfontproto kbproto inputproto xproto-native"
> +            libxdmcp xf86bigfontproto kbproto inputproto xproto-native libxcb"
>  
> -EXTRA_OECONF += "--without-xcb"
> +DEPENDS_virtclass-native += "bigreqsproto-native xproto-native xextproto-native xtrans-native libxau-native xcmiscproto-native \
> +            libxdmcp-native xf86bigfontproto-native kbproto-native inputproto-native xproto-native"
> +
> +XCB = "--with-xcb"
> +XCB_virtclass-native = "--without-xcb"
> +EXTRA_OECONF += "${XCB}"
>  
>  BBCLASSEXTEND = "native nativesdk"

FWIW:

few months back I've replied on similar change that --without-xcb is not
available in newer libx11 and that it would be better to upgrade to
newer libx11 or keep xcb enabled for -native too and add 2
BBCLASSEXTENDS to needed xcb recipes..

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

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

end of thread, other threads:[~2011-07-18 23:00 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-07 23:31 [PATCH 0/8] Bug Fixes for M2 Rc2 Saul Wold
2011-07-07 23:31 ` [PATCH 1/8] rootfs_rpm: Add 50Meg to image size for zypper Saul Wold
2011-07-07 23:38   ` Richard Purdie
2011-07-08  5:42     ` Saul Wold
2011-07-08 14:52       ` Richard Purdie
2011-07-07 23:31 ` [PATCH 2/8] image.bbclass: add LINGUAS_INSTALL to depends list Saul Wold
2011-07-07 23:31 ` [PATCH 3/8] gnome-doc-utils: Add RDEPENDS on bash Saul Wold
2011-07-07 23:31 ` [PATCH 4/8] quilt: " Saul Wold
2011-07-07 23:31 ` [PATCH 5/8] console-tools: " Saul Wold
2011-07-07 23:31 ` [PATCH 6/8] usbutils: " Saul Wold
2011-07-08  7:28   ` Koen Kooi
2011-07-08 15:42     ` Richard Purdie
2011-07-07 23:31 ` [PATCH 7/8] eglibc: " Saul Wold
2011-07-08  0:59   ` Khem Raj
2011-07-08  1:15     ` Saul Wold
2011-07-08  5:27       ` Khem Raj
2011-07-12 13:38         ` Phil Blundell
2011-07-08  6:53   ` Frans Meulenbroeks
2011-07-08  7:01     ` Koen Kooi
2011-07-08  7:23       ` Frans Meulenbroeks
2011-07-07 23:31 ` [PATCH 8/8] libx11: enable xcb support Saul Wold
2011-07-08 13:57   ` Phil Blundell
2011-07-18 22:55   ` Martin Jansa
2011-07-08  6:49 ` [PATCH 0/8] Bug Fixes for M2 Rc2 Frans Meulenbroeks
2011-07-08 15:41 ` Richard Purdie

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