[PATCH 0/2] A couple yocto-bsp fixes, v2

[PATCH 0/2] A couple yocto-bsp fixes, v2

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

Fixes for a couple yocto-bsp bugs.

v2: fixed up the yocto-bsp help text as suggested by Darren Hart.

The following changes since commit c47a869308cb0cccde0b1aee9a001352c99a8395:
  Richard Purdie (1):
        netbase: Correctly set FILESEXTRAPATHS to include the version

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib.git tzanussi/yocto-bsp-fixes-2219-2330
  http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=tzanussi/yocto-bsp-fixes-2219-2330

Tom Zanussi (2):
  yocto-kernel: use BUILDDIR to find bblayers.conf
  yocto-bsp: clarify help with reference to meta-intel

 scripts/lib/bsp/help.py   |   46 ++++++++++++++++++++++++++++++++++++++++++--
 scripts/lib/bsp/kernel.py |    7 +++++-
 2 files changed, 49 insertions(+), 4 deletions(-)

[PATCH 1/2] yocto-kernel: use BUILDDIR to find bblayers.conf

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The current code assumes that builddir == srcdir/build, which it
obviously isn't sometimes.  Use BUILDDIR to get the actual builddir
being used.

Fixes [YOCTO #2219].

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 scripts/lib/bsp/kernel.py |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/scripts/lib/bsp/kernel.py b/scripts/lib/bsp/kernel.py
index 360851b..8b3aa72 100644
--- a/scripts/lib/bsp/kernel.py
+++ b/scripts/lib/bsp/kernel.py
@@ -37,7 +37,12 @@ def find_bblayers(scripts_path):
     """
     Find and return a sanitized list of the layers found in BBLAYERS.
     """
-    bblayers_conf = os.path.join(scripts_path, "../build/conf/bblayers.conf")
+    try:
+        builddir = os.environ["BUILDDIR"]
+    except KeyError:
+        print "BUILDDIR not found, exiting. (Did you forget to source oe-init-build-env?)"
+        sys.exit(1)
+    bblayers_conf = os.path.join(builddir, "conf/bblayers.conf")
 
     layers = []
 
-- 
1.7.0.4

[PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The current yocto-bsp help assumes knowledge that the meta-intel layer
needs to be cloned before it's put into the BBLAYERS.  Avoid the
guesswork and state the details explicitly in the help.

Also, the shorter 'usage' string doesn't mention it at all; it would
help to at minimum mention it and refer the user to the detailed help.

Fixes [YOCTO #2330].

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 scripts/lib/bsp/help.py |   46 +++++++++++++++++++++++++++++++++++++++++++---
 1 files changed, 43 insertions(+), 3 deletions(-)

diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
index f78b09b..2355657 100644
--- a/scripts/lib/bsp/help.py
+++ b/scripts/lib/bsp/help.py
@@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
  'properties' that will be used to fill out the BSP-specific portions
  of the BSP.  The possible values for the 'karch' paramter can be
  listed via 'yocto-bsp list karch'.
+
+ NOTE: Once created, you should add your new layer to your
+ bblayers.conf file in order for it to be subsquently seen and
+ modified by the yocto-kernel tool.
+
+ See 'yocto bsp help create' for more detailed instructions.
+
+ NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
+ presence of the of the meta-intel layer. Ensure the meta-intel layer
+ is present and added to bblayers.conf.
+
+ See 'yocto bsp help create' for more detailed instructions.
 """
 
 yocto_bsp_create_help = """
@@ -159,9 +171,37 @@ DESCRIPTION
     bblayers.conf file in order for it to be subsquently seen and
     modified by the yocto-kernel tool.
 
-    NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
-    presence of the of the meta-intel layer, so you should also have a
-    meta-intel layer present and added to your bblayers.conf as well.
+    For example, assuming your poky repo is at /path/to/poky, your new
+    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
+    is /path/to/build:
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-mybsp \\
+      "
+
+    NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes
+    the presence of the of the meta-intel layer. Ensure the meta-intel
+    layer is present and added to bblayers.conf.
+
+    For example, assuming your poky repo is at /path/to/poky, your new
+    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
+    is /path/to/build:
+
+    $ cd /path/to/poky
+    $ git clone git://git.yoctoproject.org/meta-intel.git
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-intel \\
+      /path/to/poky/meta-mybsp \\
+      "
 """
 
 yocto_bsp_list_usage = """
-- 
1.7.0.4

Re: [PATCH 1/2] yocto-kernel: use BUILDDIR to find bblayers.conf

[Darren Hart]

On 04/27/2012 05:25 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi <tom.zanussi@intel.com>
> 
> The current code assumes that builddir == srcdir/build, which it
> obviously isn't sometimes.  Use BUILDDIR to get the actual builddir
> being used.
> 
> Fixes [YOCTO #2219].
> 
> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>

Acked-by: Darren Hart <dvhart@linux.intel.com>

> ---
>  scripts/lib/bsp/kernel.py |    7 ++++++-
>  1 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/scripts/lib/bsp/kernel.py b/scripts/lib/bsp/kernel.py
> index 360851b..8b3aa72 100644
> --- a/scripts/lib/bsp/kernel.py
> +++ b/scripts/lib/bsp/kernel.py
> @@ -37,7 +37,12 @@ def find_bblayers(scripts_path):
>      """
>      Find and return a sanitized list of the layers found in BBLAYERS.
>      """
> -    bblayers_conf = os.path.join(scripts_path, "../build/conf/bblayers.conf")
> +    try:
> +        builddir = os.environ["BUILDDIR"]
> +    except KeyError:
> +        print "BUILDDIR not found, exiting. (Did you forget to source oe-init-build-env?)"
> +        sys.exit(1)
> +    bblayers_conf = os.path.join(builddir, "conf/bblayers.conf")
>  
>      layers = []
>  

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Darren Hart]

On 04/27/2012 05:25 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi <tom.zanussi@intel.com>
> 
> The current yocto-bsp help assumes knowledge that the meta-intel layer
> needs to be cloned before it's put into the BBLAYERS.  Avoid the
> guesswork and state the details explicitly in the help.
> 
> Also, the shorter 'usage' string doesn't mention it at all; it would
> help to at minimum mention it and refer the user to the detailed help.
> 
> Fixes [YOCTO #2330].
> 
> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
> ---
>  scripts/lib/bsp/help.py |   46 +++++++++++++++++++++++++++++++++++++++++++---
>  1 files changed, 43 insertions(+), 3 deletions(-)
> 
> diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
> index f78b09b..2355657 100644
> --- a/scripts/lib/bsp/help.py
> +++ b/scripts/lib/bsp/help.py
> @@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
>   'properties' that will be used to fill out the BSP-specific portions
>   of the BSP.  The possible values for the 'karch' paramter can be
>   listed via 'yocto-bsp list karch'.
> +
> + NOTE: Once created, you should add your new layer to your
> + bblayers.conf file in order for it to be subsquently seen and

Gah, I didn't catch this the first time. Typo in "subsequently" above.
Maybe just fix in the branch? No need to resend.

> + modified by the yocto-kernel tool.
> +
> + See 'yocto bsp help create' for more detailed instructions.
> +
> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> + presence of the of the meta-intel layer. Ensure the meta-intel layer
> + is present and added to bblayers.conf.
> +
> + See 'yocto bsp help create' for more detailed instructions.
>  """
>  
>  yocto_bsp_create_help = """
> @@ -159,9 +171,37 @@ DESCRIPTION
>      bblayers.conf file in order for it to be subsquently seen and

Here too.

Otherwise looks good,

Acked-by: Darren Hart <dvhart@linux.intel.com>

>      modified by the yocto-kernel tool.
>  
> -    NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> -    presence of the of the meta-intel layer, so you should also have a
> -    meta-intel layer present and added to your bblayers.conf as well.
> +    For example, assuming your poky repo is at /path/to/poky, your new
> +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> +    is /path/to/build:
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-mybsp \\
> +      "
> +
> +    NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes
> +    the presence of the of the meta-intel layer. Ensure the meta-intel
> +    layer is present and added to bblayers.conf.
> +
> +    For example, assuming your poky repo is at /path/to/poky, your new
> +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> +    is /path/to/build:
> +
> +    $ cd /path/to/poky
> +    $ git clone git://git.yoctoproject.org/meta-intel.git
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-intel \\
> +      /path/to/poky/meta-mybsp \\
> +      "
>  """
>  
>  yocto_bsp_list_usage = """

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Tom Zanussi]

On Fri, 2012-04-27 at 22:44 -0700, Darren Hart wrote:
> 
> On 04/27/2012 05:25 PM, tom.zanussi@intel.com wrote:
> > From: Tom Zanussi <tom.zanussi@intel.com>
> > 
> > The current yocto-bsp help assumes knowledge that the meta-intel layer
> > needs to be cloned before it's put into the BBLAYERS.  Avoid the
> > guesswork and state the details explicitly in the help.
> > 
> > Also, the shorter 'usage' string doesn't mention it at all; it would
> > help to at minimum mention it and refer the user to the detailed help.
> > 
> > Fixes [YOCTO #2330].
> > 
> > Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
> > ---
> >  scripts/lib/bsp/help.py |   46 +++++++++++++++++++++++++++++++++++++++++++---
> >  1 files changed, 43 insertions(+), 3 deletions(-)
> > 
> > diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
> > index f78b09b..2355657 100644
> > --- a/scripts/lib/bsp/help.py
> > +++ b/scripts/lib/bsp/help.py
> > @@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
> >   'properties' that will be used to fill out the BSP-specific portions
> >   of the BSP.  The possible values for the 'karch' paramter can be
> >   listed via 'yocto-bsp list karch'.
> > +
> > + NOTE: Once created, you should add your new layer to your
> > + bblayers.conf file in order for it to be subsquently seen and
> 
> Gah, I didn't catch this the first time. Typo in "subsequently" above.
> Maybe just fix in the branch? No need to resend.
> 

Good catch again, thanks.  Updated in the branch...

Tom

> > + modified by the yocto-kernel tool.
> > +
> > + See 'yocto bsp help create' for more detailed instructions.
> > +
> > + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> > + presence of the of the meta-intel layer. Ensure the meta-intel layer
> > + is present and added to bblayers.conf.
> > +
> > + See 'yocto bsp help create' for more detailed instructions.
> >  """
> >  
> >  yocto_bsp_create_help = """
> > @@ -159,9 +171,37 @@ DESCRIPTION
> >      bblayers.conf file in order for it to be subsquently seen and
> 
> Here too.
> 
> Otherwise looks good,
> 
> Acked-by: Darren Hart <dvhart@linux.intel.com>
> 
> >      modified by the yocto-kernel tool.
> >  
> > -    NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> > -    presence of the of the meta-intel layer, so you should also have a
> > -    meta-intel layer present and added to your bblayers.conf as well.
> > +    For example, assuming your poky repo is at /path/to/poky, your new
> > +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> > +    is /path/to/build:
> > +
> > +    $ gedit /path/to/build/conf/bblayers.conf
> > +
> > +    BBLAYERS ?= " \\
> > +      /path/to/poky/meta \\
> > +      /path/to/poky/meta-yocto \\
> > +      /path/to/poky/meta-mybsp \\
> > +      "
> > +
> > +    NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes
> > +    the presence of the of the meta-intel layer. Ensure the meta-intel
> > +    layer is present and added to bblayers.conf.
> > +
> > +    For example, assuming your poky repo is at /path/to/poky, your new
> > +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> > +    is /path/to/build:
> > +
> > +    $ cd /path/to/poky
> > +    $ git clone git://git.yoctoproject.org/meta-intel.git
> > +
> > +    $ gedit /path/to/build/conf/bblayers.conf
> > +
> > +    BBLAYERS ?= " \\
> > +      /path/to/poky/meta \\
> > +      /path/to/poky/meta-yocto \\
> > +      /path/to/poky/meta-intel \\
> > +      /path/to/poky/meta-mybsp \\
> > +      "
> >  """
> >  
> >  yocto_bsp_list_usage = """
> 

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Darren Hart]

On 04/30/2012 12:12 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi <tom.zanussi@intel.com>
> 
> The current yocto-bsp help assumes knowledge that the meta-intel layer
> needs to be cloned before it's put into the BBLAYERS.  Avoid the
> guesswork and state the details explicitly in the help.
> 
> Also, the shorter 'usage' string doesn't mention it at all; it would
> help to at minimum mention it and refer the user to the detailed help.
> 
> Fixes [YOCTO #2330].
> 
> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>

Acked-by: Darren Hart <dvhart@linux.intel.com>

> ---
>  scripts/lib/bsp/help.py |   48 +++++++++++++++++++++++++++++++++++++++++++---
>  1 files changed, 44 insertions(+), 4 deletions(-)
> 
> diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
> index f78b09b..2d16d2e 100644
> --- a/scripts/lib/bsp/help.py
> +++ b/scripts/lib/bsp/help.py
> @@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
>   'properties' that will be used to fill out the BSP-specific portions
>   of the BSP.  The possible values for the 'karch' paramter can be
>   listed via 'yocto-bsp list karch'.
> +
> + NOTE: Once created, you should add your new layer to your
> + bblayers.conf file in order for it to be subsequently seen and
> + modified by the yocto-kernel tool.
> +
> + See 'yocto bsp help create' for more detailed instructions.
> +
> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> + presence of the meta-intel layer. Ensure the meta-intel layer is
> + present and added to bblayers.conf.
> +
> + See 'yocto bsp help create' for more detailed instructions.
>  """
>  
>  yocto_bsp_create_help = """
> @@ -156,12 +168,40 @@ DESCRIPTION
>      current directory, and is useful for debugging.
>  
>      NOTE: Once created, you should add your new layer to your
> -    bblayers.conf file in order for it to be subsquently seen and
> +    bblayers.conf file in order for it to be subsequently seen and
>      modified by the yocto-kernel tool.
>  
> -    NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> -    presence of the of the meta-intel layer, so you should also have a
> -    meta-intel layer present and added to your bblayers.conf as well.
> +    For example, assuming your poky repo is at /path/to/poky, your new
> +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> +    is /path/to/build:
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-mybsp \\
> +      "
> +
> +    NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes
> +    the presence of the meta-intel layer. Ensure the meta-intel layer
> +    is present and added to bblayers.conf.
> +
> +    For example, assuming your poky repo is at /path/to/poky, your new
> +    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
> +    is /path/to/build:
> +
> +    $ cd /path/to/poky
> +    $ git clone git://git.yoctoproject.org/meta-intel.git
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-intel \\
> +      /path/to/poky/meta-mybsp \\
> +      "
>  """
>  
>  yocto_bsp_list_usage = """

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Tom Zanussi]

On Mon, 2012-04-30 at 13:31 -0400, William Mills wrote:
> nit of nit
> 
> On 04/27/2012 08:01 PM, Darren Hart wrote:
> > On 04/27/2012 12:00 PM, tom.zanussi@intel.com wrote:
> >> From: Tom Zanussi<tom.zanussi@intel.com>
> >>
> >> + NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> >> + presence of the of the meta-intel layer, so you should also have a
> >> + meta-intel layer present and added to your bblayers.conf as well.
> > Consider:
> >
> > + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> > + presence of the of the meta-intel layer. Ensure the meta-intel layer
> > + is present and added to bblayers.conf.
> >
> 
> Unless I am really mis-reading this I think "presence of the of the" 
> should just be "presence of the"

Doh, you're correct, good catch.  Third version of the most perused help
text in history submitted...

Thanks,

Tom

[PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The current yocto-bsp help assumes knowledge that the meta-intel layer
needs to be cloned before it's put into the BBLAYERS.  Avoid the
guesswork and state the details explicitly in the help.

Also, the shorter 'usage' string doesn't mention it at all; it would
help to at minimum mention it and refer the user to the detailed help.

Fixes [YOCTO #2330].

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 scripts/lib/bsp/help.py |   48 +++++++++++++++++++++++++++++++++++++++++++---
 1 files changed, 44 insertions(+), 4 deletions(-)

diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
index f78b09b..2d16d2e 100644
--- a/scripts/lib/bsp/help.py
+++ b/scripts/lib/bsp/help.py
@@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
  'properties' that will be used to fill out the BSP-specific portions
  of the BSP.  The possible values for the 'karch' paramter can be
  listed via 'yocto-bsp list karch'.
+
+ NOTE: Once created, you should add your new layer to your
+ bblayers.conf file in order for it to be subsequently seen and
+ modified by the yocto-kernel tool.
+
+ See 'yocto bsp help create' for more detailed instructions.
+
+ NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
+ presence of the meta-intel layer. Ensure the meta-intel layer is
+ present and added to bblayers.conf.
+
+ See 'yocto bsp help create' for more detailed instructions.
 """
 
 yocto_bsp_create_help = """
@@ -156,12 +168,40 @@ DESCRIPTION
     current directory, and is useful for debugging.
 
     NOTE: Once created, you should add your new layer to your
-    bblayers.conf file in order for it to be subsquently seen and
+    bblayers.conf file in order for it to be subsequently seen and
     modified by the yocto-kernel tool.
 
-    NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
-    presence of the of the meta-intel layer, so you should also have a
-    meta-intel layer present and added to your bblayers.conf as well.
+    For example, assuming your poky repo is at /path/to/poky, your new
+    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
+    is /path/to/build:
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-mybsp \\
+      "
+
+    NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes
+    the presence of the meta-intel layer. Ensure the meta-intel layer
+    is present and added to bblayers.conf.
+
+    For example, assuming your poky repo is at /path/to/poky, your new
+    BSP layer is at /path/to/poky/meta-mybsp, and your build directory
+    is /path/to/build:
+
+    $ cd /path/to/poky
+    $ git clone git://git.yoctoproject.org/meta-intel.git
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-intel \\
+      /path/to/poky/meta-mybsp \\
+      "
 """
 
 yocto_bsp_list_usage = """
-- 
1.7.0.4

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Autif Khan]

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

>>> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
>>> + presence of the of the meta-intel layer. Ensure the meta-intel layer
>>> + is present and added to bblayers.conf.
>>>
>>
>> Unless I am really mis-reading this I think "presence of the of the"
>> should just be "presence of the"
>
> Oh wow. :-)
>

There is an illusion like this (attached jpeg)

[-- Attachment #2: illusion.jpg --]
[-- Type: image/jpeg, Size: 19297 bytes --]

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Darren Hart]

On 04/30/2012 10:31 AM, William Mills wrote:
> nit of nit
> 
> On 04/27/2012 08:01 PM, Darren Hart wrote:
>> On 04/27/2012 12:00 PM, tom.zanussi@intel.com wrote:
>>> From: Tom Zanussi<tom.zanussi@intel.com>
>>>
>>> + NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
>>> + presence of the of the meta-intel layer, so you should also have a
>>> + meta-intel layer present and added to your bblayers.conf as well.
>> Consider:
>>
>> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
>> + presence of the of the meta-intel layer. Ensure the meta-intel layer
>> + is present and added to bblayers.conf.
>>
> 
> Unless I am really mis-reading this I think "presence of the of the" 
> should just be "presence of the"

Oh wow. :-)

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[William Mills]

nit of nit

On 04/27/2012 08:01 PM, Darren Hart wrote:
> On 04/27/2012 12:00 PM, tom.zanussi@intel.com wrote:
>> From: Tom Zanussi<tom.zanussi@intel.com>
>>
>> + NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
>> + presence of the of the meta-intel layer, so you should also have a
>> + meta-intel layer present and added to your bblayers.conf as well.
> Consider:
>
> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> + presence of the of the meta-intel layer. Ensure the meta-intel layer
> + is present and added to bblayers.conf.
>

Unless I am really mis-reading this I think "presence of the of the" 
should just be "presence of the"

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Tom Zanussi]

On Fri, 2012-04-27 at 17:01 -0700, Darren Hart wrote:
> 
> On 04/27/2012 12:00 PM, tom.zanussi@intel.com wrote:
> > From: Tom Zanussi <tom.zanussi@intel.com>
> > 
> > The current yocto-bsp help assumes knowledge that the meta-intel layer
> > needs to be cloned before it's put into the BBLAYERS.  Avoid the
> > guesswork and state the details explicitly in the help.
> > 
> > Also, the shorter 'usage' string doesn't mention it at all; it would
> > help to at minimum mention it and refer the user to the detailed help.
> > 
> > Fixes [YOCTO #2330].
> > 
> > Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
> 
> Some minor nits below:

Thanks for catching those, patches resubmitted...

Tom


> 
> > ---
> >  scripts/lib/bsp/help.py |   40 ++++++++++++++++++++++++++++++++++++++++
> >  1 files changed, 40 insertions(+), 0 deletions(-)
> > 
> > diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
> > index f78b09b..f849833 100644
> > --- a/scripts/lib/bsp/help.py
> > +++ b/scripts/lib/bsp/help.py
> > @@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
> >   'properties' that will be used to fill out the BSP-specific portions
> >   of the BSP.  The possible values for the 'karch' paramter can be
> >   listed via 'yocto-bsp list karch'.
> > +
> > + NOTE: Once created, you should add your new layer to your
> > + bblayers.conf file in order for it to be subsquently seen and
> > + modified by the yocto-kernel tool.
> > +
> > + See 'yocto bsp help create' for more detailed instructions.
> > +
> > + NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> > + presence of the of the meta-intel layer, so you should also have a
> > + meta-intel layer present and added to your bblayers.conf as well.
> 
> Consider:
> 
> + NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
> + presence of the of the meta-intel layer. Ensure the meta-intel layer
> + is present and added to bblayers.conf.
> 
> 
> > +
> > + See 'yocto bsp help create' for more detailed instructions.
> >  """
> >  
> >  yocto_bsp_create_help = """
> > @@ -159,9 +171,37 @@ DESCRIPTION
> >      bblayers.conf file in order for it to be subsquently seen and
> >      modified by the yocto-kernel tool.
> >  
> > +    For example, assuming your poky repo is at /path/to/poky, your
> > +    your new BSP layer is at /path/to/poky/meta-mybsp, and your build
> 
> 
> The word "your" appears twice in a row.
> 
> 
> > +    directory is /path/to/build:
> > +
> > +    $ gedit /path/to/build/conf/bblayers.conf
> > +
> > +    BBLAYERS ?= " \\
> > +      /path/to/poky/meta \\
> > +      /path/to/poky/meta-yocto \\
> > +      /path/to/poky/meta-mybsp \\
> > +      "
> > +
> >      NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> >      presence of the of the meta-intel layer, so you should also have a
> >      meta-intel layer present and added to your bblayers.conf as well.
> 
> OK, this is just context, but same comment as above.
> 
> > +
> > +    For example, assuming your poky repo is at /path/to/poky, your
> > +    your new BSP layer is at /path/to/poky/meta-mybsp, and your build
> 
> Same double "your" here as well.
> 
> > +    directory is /path/to/build:
> > +
> > +    $ cd /path/to/poky
> > +    $ git clone git://git.yoctoproject.org/meta-intel.git
> > +
> > +    $ gedit /path/to/build/conf/bblayers.conf
> > +
> > +    BBLAYERS ?= " \\
> > +      /path/to/poky/meta \\
> > +      /path/to/poky/meta-yocto \\
> > +      /path/to/poky/meta-intel \\
> > +      /path/to/poky/meta-mybsp \\
> > +      "
> >  """
> >  
> >  yocto_bsp_list_usage = """
> 

Re: [PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[Darren Hart]

On 04/27/2012 12:00 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi <tom.zanussi@intel.com>
> 
> The current yocto-bsp help assumes knowledge that the meta-intel layer
> needs to be cloned before it's put into the BBLAYERS.  Avoid the
> guesswork and state the details explicitly in the help.
> 
> Also, the shorter 'usage' string doesn't mention it at all; it would
> help to at minimum mention it and refer the user to the detailed help.
> 
> Fixes [YOCTO #2330].
> 
> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>

Some minor nits below:

> ---
>  scripts/lib/bsp/help.py |   40 ++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 40 insertions(+), 0 deletions(-)
> 
> diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
> index f78b09b..f849833 100644
> --- a/scripts/lib/bsp/help.py
> +++ b/scripts/lib/bsp/help.py
> @@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
>   'properties' that will be used to fill out the BSP-specific portions
>   of the BSP.  The possible values for the 'karch' paramter can be
>   listed via 'yocto-bsp list karch'.
> +
> + NOTE: Once created, you should add your new layer to your
> + bblayers.conf file in order for it to be subsquently seen and
> + modified by the yocto-kernel tool.
> +
> + See 'yocto bsp help create' for more detailed instructions.
> +
> + NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
> + presence of the of the meta-intel layer, so you should also have a
> + meta-intel layer present and added to your bblayers.conf as well.

Consider:

+ NOTE: For x86- and x86_64-based BSPs, the generated BSP assumes the
+ presence of the of the meta-intel layer. Ensure the meta-intel layer
+ is present and added to bblayers.conf.


> +
> + See 'yocto bsp help create' for more detailed instructions.
>  """
>  
>  yocto_bsp_create_help = """
> @@ -159,9 +171,37 @@ DESCRIPTION
>      bblayers.conf file in order for it to be subsquently seen and
>      modified by the yocto-kernel tool.
>  
> +    For example, assuming your poky repo is at /path/to/poky, your
> +    your new BSP layer is at /path/to/poky/meta-mybsp, and your build


The word "your" appears twice in a row.


> +    directory is /path/to/build:
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-mybsp \\
> +      "
> +
>      NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
>      presence of the of the meta-intel layer, so you should also have a
>      meta-intel layer present and added to your bblayers.conf as well.

OK, this is just context, but same comment as above.

> +
> +    For example, assuming your poky repo is at /path/to/poky, your
> +    your new BSP layer is at /path/to/poky/meta-mybsp, and your build

Same double "your" here as well.

> +    directory is /path/to/build:
> +
> +    $ cd /path/to/poky
> +    $ git clone git://git.yoctoproject.org/meta-intel.git
> +
> +    $ gedit /path/to/build/conf/bblayers.conf
> +
> +    BBLAYERS ?= " \\
> +      /path/to/poky/meta \\
> +      /path/to/poky/meta-yocto \\
> +      /path/to/poky/meta-intel \\
> +      /path/to/poky/meta-mybsp \\
> +      "
>  """
>  
>  yocto_bsp_list_usage = """

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

[PATCH 2/2] yocto-bsp: clarify help with reference to meta-intel

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The current yocto-bsp help assumes knowledge that the meta-intel layer
needs to be cloned before it's put into the BBLAYERS.  Avoid the
guesswork and state the details explicitly in the help.

Also, the shorter 'usage' string doesn't mention it at all; it would
help to at minimum mention it and refer the user to the detailed help.

Fixes [YOCTO #2330].

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 scripts/lib/bsp/help.py |   40 ++++++++++++++++++++++++++++++++++++++++
 1 files changed, 40 insertions(+), 0 deletions(-)

diff --git a/scripts/lib/bsp/help.py b/scripts/lib/bsp/help.py
index f78b09b..f849833 100644
--- a/scripts/lib/bsp/help.py
+++ b/scripts/lib/bsp/help.py
@@ -115,6 +115,18 @@ yocto_bsp_create_usage = """
  'properties' that will be used to fill out the BSP-specific portions
  of the BSP.  The possible values for the 'karch' paramter can be
  listed via 'yocto-bsp list karch'.
+
+ NOTE: Once created, you should add your new layer to your
+ bblayers.conf file in order for it to be subsquently seen and
+ modified by the yocto-kernel tool.
+
+ See 'yocto bsp help create' for more detailed instructions.
+
+ NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
+ presence of the of the meta-intel layer, so you should also have a
+ meta-intel layer present and added to your bblayers.conf as well.
+
+ See 'yocto bsp help create' for more detailed instructions.
 """
 
 yocto_bsp_create_help = """
@@ -159,9 +171,37 @@ DESCRIPTION
     bblayers.conf file in order for it to be subsquently seen and
     modified by the yocto-kernel tool.
 
+    For example, assuming your poky repo is at /path/to/poky, your
+    your new BSP layer is at /path/to/poky/meta-mybsp, and your build
+    directory is /path/to/build:
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-mybsp \\
+      "
+
     NOTE for x86- and x86_64-based BSPs: The generated BSP assumes the
     presence of the of the meta-intel layer, so you should also have a
     meta-intel layer present and added to your bblayers.conf as well.
+
+    For example, assuming your poky repo is at /path/to/poky, your
+    your new BSP layer is at /path/to/poky/meta-mybsp, and your build
+    directory is /path/to/build:
+
+    $ cd /path/to/poky
+    $ git clone git://git.yoctoproject.org/meta-intel.git
+
+    $ gedit /path/to/build/conf/bblayers.conf
+
+    BBLAYERS ?= " \\
+      /path/to/poky/meta \\
+      /path/to/poky/meta-yocto \\
+      /path/to/poky/meta-intel \\
+      /path/to/poky/meta-mybsp \\
+      "
 """
 
 yocto_bsp_list_usage = """
-- 
1.7.0.4