[OE-core] [PATCH 0/3] Enhancement on devtool edit-recipe

[Paul Eggleton]

I've looked at these already, in case that's a concern.

Acked-by: Paul Eggleton <paul.eggleton at linux.intel.com>

Cheers,
Paul

On Tuesday, 11 July 2017 2:50:44 AM BST Chang, Rebecca Swee Fun wrote:
> Ping in case maintainer has missed this series.
> Please let me know if there are any concern to merge this enhancement on
> devtool edit-recipe and I would rework it asap for 2.4M2.
> Thank you very much.
> 
> Rebecca
> 
> > -----Original Message-----
> > From: Chang, Rebecca Swee Fun
> > Sent: Wednesday, June 28, 2017 9:59 AM
> > To: OpenEmbedded Core Mailing List <openembedded-
> > core at lists.openembedded.org>
> > Cc: Chang, Rebecca Swee Fun <rebecca.swee.fun.chang at intel.com>
> > Subject: [PATCH 0/3] Enhancement on devtool edit-recipe
> > 
> > Hi,
> > 
> > The patches in this series are to enhance devtool edit-recipe as a request from
> > Bugzilla #11434.
> > 
> > This first enhancement I did was to ensure devtool edit-recipe not to print out
> > ugly tracebacks that is not informative by catching the exception.
> > 
> > However, this has introduced new tracebacks where python complaints about
> > NameError: name 'logger' is not defined in run_editor definition. We could pass
> > in logger as a parameter from run_editor function call. This was fixed in the
> > second patch in this series.
> > 
> > Lastly, I have enable additional subcommand 'find-recipe' for devtool. This will
> > help users to find the path to the recipes in workspace and it can be extended
> > to find recipe outside of workspace using '-a/--any-recipe' option.
> > 
> > Regards,
> > Rebecca
> > 
> > Chang Rebecca Swee Fun (3):
> >   scriptutils: exit politely when no text editor available
> >   scriptutils: pass in logger as parameter
> >   devtool: find-recipe: enable new subcommand for devtool
> > 
> >  scripts/lib/devtool/utilcmds.py     | 18 ++++++++++++++----
> >  scripts/lib/recipetool/newappend.py |  2 +-
> >  scripts/lib/scriptutils.py          |  6 +++---
> >  3 files changed, 18 insertions(+), 8 deletions(-)
> > 
> > --
> > 2.7.4
> 
> 


-- 

Paul Eggleton
Intel Open Source Technology Centre