[oe] wpa-supplicant-0.7.inc patch

Martin Bures needemesleepe at gmail.com
Mon May 16 14:02:22 UTC 2011


I have updated the patch and simplified it a bit.  I do not want to switch
the recipe to use defconfig instead of .config because defconfig is modified
by the recipe.

The change simplifies down to just changing the directory back to ${S}.
Then .config can be found again.  The line above the first test changes the
pwd and this is the problem.

As I already submitted the patch via email, I will continue for this one and
use git's submission mechanism in the future.

Best regards,
martin.

>From f3f91d271e2f8664b72ed77fc523a733094f701a Mon Sep 17 00:00:00 2001
From: Martin Bures <needemesleepe at gmail.com>
Date: Mon, 16 May 2011 09:47:51 -0400
Subject: [PATCH] wpa-supplicant: fixed .inc recipe to correctly deploy dbus
configuration
 files when dbus is configured.

The current recipe changes the directory, changing the scope such that
the recipe cannot resolve .config.  When tests are made whether .config
contains DBUS configurations, it always fails.  This causes the
placement of DBUS configuration files to always fail.

Signed-off-by: Martin Bures <needemesleepe at gmail.com>
---
 recipes/wpa-supplicant/wpa-supplicant-0.7.inc |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/recipes/wpa-supplicant/wpa-supplicant-0.7.inc
b/recipes/wpa-supplicant/wpa-supplicant-0.7.inc
index 7649038..abf01c2 100644
--- a/recipes/wpa-supplicant/wpa-supplicant-0.7.inc
+++ b/recipes/wpa-supplicant/wpa-supplicant-0.7.inc
@@ -63,6 +63,7 @@ do_install () {
     cd ${D}${sysconfdir}/network/ && \
     ln -sf ../if-pre-up.d/wpa-supplicant if-post-down.d/wpa-supplicant

+    cd ${S}
     if grep -q ^CONFIG_CTRL_IFACE_DBUS=y .config || grep -q
^CONFIG_CTRL_IFACE_DBUS_NEW=y .config; then
         install -d ${D}/${sysconfdir}/dbus-1/system.d
         install -m 644 ${S}/dbus/dbus-wpa_supplicant.conf
${D}/${sysconfdir}/dbus-1/system.d
-- 
1.7.0.4


On Thu, May 12, 2011 at 6:13 PM, Paul Menzel <
paulepanter at users.sourceforge.net> wrote:

> Dear Martin,
>
>
> Am Donnerstag, den 12.05.2011, 17:54 -0400 schrieb Martin Bures:
>
> > I will take a look at the policy and try to update the patch tomorrow.
>
> thanks.
>
> > As for your questions/comments about .config.  That was how the recipe
> was
> > when I found it - I was just trying to make minimal modifications from
> what
> > was there to get it working.  I agree that it could be changed to use
> > defconfig.  As with the patch submission guidelines, I will take a look
> and
> > see if this change as-suggested works correctly tomorrow and modify and
> > resubmit the patch accordingly.
>
> Please also take a look at commit aa35ba5c [1] and maybe even CC the
> author and committer.
>
>        commit aa35ba5cf7e1bf30fb2edb1408c0b599ec89290a
>        Author: Steve Sakoman <steve at sakoman.com>
>        Date:   Tue Apr 19 08:40:52 2011 -0700
>
>            wpa-supplicant: Enable CONFIG_CTRL_IFACE_DBUS and
> CONFIG_CTRL_IFACE_DBUS_NEW
>
>            Signed-off-by: Steve Sakoman <steve at sakoman.com>
>            Signed-off-by: Koen Kooi <koen at openembedded.org>
>
> I do not know what build configuration you use. Please remember to write
> down what version you are using and what distribution and machine.
>
>
> Thanks,
>
> Paul
>
>
> [1]
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=aa35ba5cf7e1bf30fb2edb1408c0b599ec89290a
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>



More information about the Openembedded-devel mailing list