Done.
V2 will be sent out soon.

Thanks,
Chen Qi

On 05/31/2018 10:57 PM, Christopher Larson wrote:
While you're at it, I'd suggest fixing this to stop using the horrible indexing a list with a boolean hack and use a proper inline conditional. 'foo' if bar else 'baz'.

On Thu, May 31, 2018 at 1:43 AM Chen Qi <qi.c...@windriver.com <mailto:qi.c...@windriver.com>> wrote:

    A list of tools are added to HOSTTOOLS depending on if we inherit
    testimage or not. Unfortunately, if we use TEST_IMAGE variable to
    automate the test, these tools are not added to HOSTTOOLS.

    Modify the condition to also check TEST_IMAGE to fix the above
    problem.

    Signed-off-by: Chen Qi <qi.c...@windriver.com
    <mailto:qi.c...@windriver.com>>
    ---
     meta/conf/bitbake.conf | 2 +-
     1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
    index 6ecf78b..7b7472c 100644
    --- a/meta/conf/bitbake.conf
    +++ b/meta/conf/bitbake.conf
    @@ -487,7 +487,7 @@ HOSTTOOLS += " \
     "

     # Tools needed to run testimage runtime image testing
    -HOSTTOOLS += "${@['', 'ip ping ps scp ssh
    stty'][bb.data.inherits_class('testimage', d)]}"
    +HOSTTOOLS += "${@['', 'ip ping ps scp ssh
    stty'][bb.data.inherits_class('testimage', d) or
    d.getVar('TEST_IMAGE') == '1']}"

     # Link to these if present
     HOSTTOOLS_NONFATAL += "aws ccache gcc-ar gpg ld.bfd ld.gold nc
    sftp socat ssh sudo"
-- 1.9.1

-- _______________________________________________
    Openembedded-core mailing list
    Openembedded-core@lists.openembedded.org
    <mailto:Openembedded-core@lists.openembedded.org>
    http://lists.openembedded.org/mailman/listinfo/openembedded-core



--
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics


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

Reply via email to