Re: [PATCH v2] wic: look for wks files in <layer>/wic

[Ed Bartosh <ed.bartosh@linux.intel.com>]

On Mon, Dec 19, 2016 at 03:31:29PM -0800, Khem Raj wrote:
> On Mon, Dec 19, 2016 at 4:41 AM, Ed Bartosh <ed.bartosh@linux.intel.com> wrote:
> > Currently wic looks for wks files in
> > <layer dir>/scripts/lib/wic/canned-wks/ directories.
> > This path is too nested and doesn't look consistent with the
> > naming scheme of layer directories.
> >
> > Added <layer>/wic directory to the list of paths
> > to look for wks files.
> 
> Does it also affect documentation ?
> 

Yes, in a way. There is only indirect mentioning of
scripts/lib/wic/canned-wks/ in the documentation. I'm going to add new
path there or even directly mention both paths in dev manual.

> >
> > Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> > ---
> >  scripts/lib/wic/engine.py | 7 +++++--
> >  1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py
> > index 5b10463..2adef2f 100644
> > --- a/scripts/lib/wic/engine.py
> > +++ b/scripts/lib/wic/engine.py
> > @@ -52,6 +52,7 @@ def verify_build_env():
> >
> >  CANNED_IMAGE_DIR = "lib/wic/canned-wks" # relative to scripts
> >  SCRIPTS_CANNED_IMAGE_DIR = "scripts/" + CANNED_IMAGE_DIR
> > +WIC_DIR = "wic"
> >
> >  def build_canned_image_list(path):
> >      layers_path = misc.get_bitbake_var("BBLAYERS")
> > @@ -59,8 +60,10 @@ def build_canned_image_list(path):
> >
> >      if layers_path is not None:
> >          for layer_path in layers_path.split():
> > -            cpath = os.path.join(layer_path, SCRIPTS_CANNED_IMAGE_DIR)
> > -            canned_wks_layer_dirs.append(cpath)
> > +            for wks_path in (WIC_DIR, SCRIPTS_CANNED_IMAGE_DIR):
> > +                cpath = os.path.join(layer_path, wks_path)
> > +                if os.path.isdir(cpath):
> > +                    canned_wks_layer_dirs.append(cpath)
> >
> >      cpath = os.path.join(path, CANNED_IMAGE_DIR)
> >      canned_wks_layer_dirs.append(cpath)
> > --
> > 2.1.4
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core

-- 
--
Regards,
Ed