> -----Original Message-----
> From: Saul Wold [mailto:s...@linux.intel.com]
> Sent: Thursday, March 21, 2013 1:05 AM
> To: Liu Ting-B28495
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [oe-core][PATCH 1/2] busybox: detects customized
> configs when do_install
>
> 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.
>
>
Patch v2 sent out. Please review.
> 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
