Re: [PATCH V2] selftest/buildoptions: pexpect not a builtin module

[Benjamin Esquivel <benjamin.esquivel@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 975 bytes --]

Great! thanks Daniel.
On Mon, 2015-09-21 at 10:49 +0000, Istrate, Daniel AlexandruX wrote:
> Hi Ross,
>  
> I sent the patch: http://lists.openembedded.org/pipermail/openembedde
> d-core/2015-September/110738.html
>  
> --Daniel
>  
> From: Burton, Ross [mailto:ross.burton@intel.com] 
> Sent: Monday, September 21, 2015 13:20
> To: Istrate, Daniel AlexandruX
> Cc: Benjamin Esquivel; openembedded-core@lists.openembedded.org; 
> paul.eggleton@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@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 

[-- Attachment #2: Type: text/html, Size: 4391 bytes --]