[OE-core] [PATCH V2] selftest/buildoptions: pexpect not a builtin module

Istrate, Daniel AlexandruX daniel.alexandrux.istrate at intel.com
Mon Sep 21 10:49:45 UTC 2015


Hi Ross,

I sent the patch: http://lists.openembedded.org/pipermail/openembedded-core/2015-September/110738.html

--Daniel

From: Burton, Ross [mailto:ross.burton at intel.com]
Sent: Monday, September 21, 2015 13:20
To: Istrate, Daniel AlexandruX
Cc: Benjamin Esquivel; openembedded-core at lists.openembedded.org; paul.eggleton at linux.intel.com
Subject: Re: [OE-core] [PATCH V2] selftest/buildoptions: pexpect not a builtin module


On 21 September 2015 at 09:37, Istrate, Daniel AlexandruX <daniel.alexandrux.istrate at intel.com<mailto:daniel.alexandrux.istrate at intel.com>> wrote:
Just want to point out that 'pexpect' is an unused import in buildoptions.py. Unused imports are also unittest and logging.
Instead of try-catch an import that is not used I think it's best to get rid of them.

Totally agreed.  Can you send a quick patch?

Ross
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150921/37afdb6f/attachment-0002.html>


More information about the Openembedded-core mailing list