On 03/12/2013 03:52 AM, b28...@freescale.com wrote:
From: Ting Liu <b28...@freescale.com>

After using 'bitbake -c menuconfig busybox' to customize defconfig,
do_install fail to detect the changes. Grep configs in ${S}/.config
instead of defconfig.

With the recent changes from Richard to move towards using a separate ${B} directory for build vs ${S} directory for source (marked read-only in the future), this change should be updated to use ${B} instead of ${S} for checking the .config file.

The overall change is good, we just need to use the right directory for checking the build related changes to the .config file.


Thanks
    Sau!

Change-Id: I958c232eb9f8096e4113a1c3a2748f5f2eccdeb5
Signed-off-by: Ting Liu <b28...@freescale.com>
---
  meta/recipes-core/busybox/busybox.inc       |   22 +++++++++++-----------
  meta/recipes-core/busybox/busybox_1.20.2.bb |    3 ++-
  2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/meta/recipes-core/busybox/busybox.inc 
b/meta/recipes-core/busybox/busybox.inc
index fc6ca91..093bf2e 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -149,11 +149,11 @@ do_install () {

        install -d ${D}${sysconfdir}/init.d

-       if ! grep -q "CONFIG_FEATURE_INDIVIDUAL=y" ${WORKDIR}/defconfig; then
+       if ! grep -q "CONFIG_FEATURE_INDIVIDUAL=y" ${S}/.config; then
                # Install /bin/busybox, and the /bin/sh link so the postinst 
script
                # can run. Let update-alternatives handle the rest.
                install -d ${D}${base_bindir}
-               if grep -q "CONFIG_FEATURE_SUID=y" ${WORKDIR}/defconfig; then
+               if grep -q "CONFIG_FEATURE_SUID=y" ${S}/.config; then
                        install -m 4755 ${S}/busybox ${D}${base_bindir}
                else
                        install -m 0755 ${S}/busybox ${D}${base_bindir}
@@ -179,36 +179,36 @@ do_install () {
                fi
        fi

-       if grep -q "CONFIG_SYSLOGD=y" ${WORKDIR}/defconfig; then
+       if grep -q "CONFIG_SYSLOGD=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/syslog 
${D}${sysconfdir}/init.d/syslog.${BPN}
                install -m 644 ${WORKDIR}/syslog-startup.conf 
${D}${sysconfdir}/syslog-startup.conf.${BPN}
        fi
-       if grep "CONFIG_CROND=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_CROND=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/busybox-cron 
${D}${sysconfdir}/init.d/
        fi
-       if grep "CONFIG_HTTPD=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_HTTPD=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/busybox-httpd 
${D}${sysconfdir}/init.d/
                install -d ${D}/srv/www
        fi
-       if grep "CONFIG_UDHCPD=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_UDHCPD=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/busybox-udhcpd 
${D}${sysconfdir}/init.d/
        fi
-       if grep "CONFIG_HWCLOCK=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_HWCLOCK=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/hwclock.sh ${D}${sysconfdir}/init.d/
        fi
-       if grep "CONFIG_UDHCPC=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_UDHCPC=y" ${S}/.config; then
                install -d ${D}${sysconfdir}/udhcpc.d
                install -d ${D}${datadir}/udhcpc
                  install -m 0755 ${WORKDIR}/simple.script 
${D}${sysconfdir}/udhcpc.d/50default
                install -m 0755 ${WORKDIR}/default.script 
${D}${datadir}/udhcpc/default.script
        fi
-       if grep "CONFIG_INETD=y" ${WORKDIR}/defconfig; then
+       if grep "CONFIG_INETD=y" ${S}/.config; then
                install -m 0755 ${WORKDIR}/inetd 
${D}${sysconfdir}/init.d/inetd.${BPN}
                install -m 0644 ${WORKDIR}/inetd.conf ${D}${sysconfdir}/
        fi
-        if grep "CONFIG_MDEV=y" ${WORKDIR}/defconfig; then
+        if grep "CONFIG_MDEV=y" ${S}/.config; then
                 install -m 0755 ${WORKDIR}/mdev ${D}${sysconfdir}/init.d/mdev
-               if grep "CONFIG_FEATURE_MDEV_CONF=y" ${WORKDIR}/defconfig; then
+               if grep "CONFIG_FEATURE_MDEV_CONF=y" ${S}/.config; then
                         install -m 644 ${WORKDIR}/mdev.conf 
${D}${sysconfdir}/mdev.conf
                 fi
        fi
diff --git a/meta/recipes-core/busybox/busybox_1.20.2.bb 
b/meta/recipes-core/busybox/busybox_1.20.2.bb
index 783261e..255d56c 100644
--- a/meta/recipes-core/busybox/busybox_1.20.2.bb
+++ b/meta/recipes-core/busybox/busybox_1.20.2.bb
@@ -1,5 +1,5 @@
  require busybox.inc
-PR = "r5"
+PR = "r6"

  SRC_URI = 
"http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
             file://B921600.patch \
@@ -30,6 +30,7 @@ SRC_URI = 
"http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
             file://stat-usr-bin.patch \
             file://testsuite-du-du-k-works-fix-false-positive.patch"

+
  SRC_URI[tarball.md5sum] = "e025414bc6cd79579cc7a32a45d3ae1c"
  SRC_URI[tarball.sha256sum] = 
"eb13ff01dae5618ead2ef6f92ba879e9e0390f9583bd545d8789d27cf39b6882"



_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to