On 08/08/2017 11:30 PM, Richard Purdie wrote:
On Mon, 2017-07-31 at 02:50 -0700, Robert Yang wrote:The "if qemu_use_kvm" is not needed. Signed-off-by: Robert Yang <liezhi.y...@windriver.com> --- meta/lib/oeqa/targetcontrol.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py index 3255e3a5c63..11e6c820e85 100644 --- a/meta/lib/oeqa/targetcontrol.py +++ b/meta/lib/oeqa/targetcontrol.py @@ -132,9 +132,8 @@ class QemuTarget(BaseTarget): dump_host_cmds = d.getVar("testimage_dump_host") dump_dir = d.getVar("TESTIMAGE_DUMP_DIR") qemu_use_kvm = d.getVar("QEMU_USE_KVM") - if qemu_use_kvm and \ - (qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \ - d.getVar("MACHINE") in qemu_use_kvm.split()): + if qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \ + d.getVar("MACHINE") in qemu_use_kvm.split(): use_kvm = True else: use_kvm = FalseIt is needed since qemu_use_kvm could be None and None.split() will cause an error.
Thanks, I removed it from the repo: git://git.openembedded.org/openembedded-core-contrib rbt/ptest http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/ptest // Robert
Cheers, Richard
-- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
