Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Scott Garman <scott.a.garman@intel.com>
To: "Eric Bénard" <eric@eukrea.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: useradd changes to dbus, base-passwd
Date: Sun, 06 Nov 2011 15:16:42 -0800	[thread overview]
Message-ID: <4EB7155A.1050808@intel.com> (raw)
In-Reply-To: <4EB6EE8E.4000401@eukrea.com>

On 11/06/2011 12:31 PM, Eric Bénard wrote:
> Hi Scott,
>
> Le 26/10/2011 18:52, Scott Garman a écrit :
>> Grepping through our classes, it looks like I can detect the nativesdk
>> within
>> useradd.bbclass by checking for bb.data.inherits_class('nativesdk', d).
>>
>> I've filed a bug for this and will develop a patch to ensure useradd is
>> becomes a no-op for nativesdk builds:
>>
>> http://bugzilla.pokylinux.org/show_bug.cgi?id=1702
>>
> here is a patch tentative which allowed me to compile meta-toolchain-qte.
> Is that the right way to solve the problem (in which case I'll submit
> the patch with comment & sob)?

Hi Eric,

This looks good, thank you! Two questions though:

When I was testing my own variation of this fix out, I found that 
testing bb.data.inherits_class('nativesdk', d) did not work as I 
expected. It returned false for a recipe I was testing with that used 
BBCLASSEXTEND = "native nativesdk". So instead I was checking if PN 
ended with -nativesdk. Are you certain the inherits_class test works?

My second question is why the autotools_do_install is needed for the 
nativesdk case?

Thanks,

Scott

-- 
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center



  reply	other threads:[~2011-11-06 23:22 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-26  1:34 useradd changes to dbus, base-passwd James Limbouris
2011-10-26  3:37 ` James Limbouris
2011-10-26 15:23   ` Mark Hatle
2011-10-26 16:52     ` Scott Garman
2011-11-06 20:31       ` Eric Bénard
2011-11-06 23:16         ` Scott Garman [this message]
2011-11-07  8:36           ` Eric Bénard
2011-11-07 18:24             ` Scott Garman
2011-11-07 21:11               ` [PATCH 1/2] useradd.bbclass: handle nativesdk case Eric Bénard
2011-11-07 21:11                 ` [PATCH 2/2] dbus: fix install for virtclass-nativesdk Eric Bénard
2011-11-07 21:32                   ` Scott Garman
2011-11-07 21:32                 ` [PATCH 1/2] useradd.bbclass: handle nativesdk case Scott Garman
2011-11-07 23:47                 ` Scott Garman
2011-11-08  8:41                   ` Eric Bénard
2011-11-08  8:41                     ` [PATCH 2/2] dbus: fix install for virtclass-nativesdk Eric Bénard
2011-11-08 14:19                       ` Richard Purdie
2011-11-08 14:11                     ` [PATCH 1/2] useradd.bbclass: handle nativesdk case Richard Purdie
2011-11-08 15:46                       ` [PATCH] " Eric Bénard
2011-11-08 16:30                         ` Richard Purdie
2011-11-08  8:45                   ` [PATCH 1/2] " Eric Bénard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EB7155A.1050808@intel.com \
    --to=scott.a.garman@intel.com \
    --cc=eric@eukrea.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox