On Wed, 2012-10-24 at 13:25 -0700, nitin.a.kam...@intel.com wrote: > From: Nitin A Kamble <nitin.a.kam...@intel.com> > > The WEBTITLE will be used to publish the BSP on the Yocto Project Website. > And adding the Yocto Project Compliance information for the 1.3 release. > Also specifying all the layers used from meta-intel repository. > > Signed-off-by: Nitin A Kamble <nitin.a.kam...@intel.com> > --- > meta-chiefriver/README | 8 +++++++- > meta-chiefriver/conf/machine/chiefriver.conf | 2 ++ > 2 files changed, 9 insertions(+), 1 deletions(-) > > diff --git a/meta-chiefriver/README b/meta-chiefriver/README > index 7c47b02..249a389 100644 > --- a/meta-chiefriver/README > +++ b/meta-chiefriver/README > @@ -7,6 +7,12 @@ plus the Panther Point PCH. This BSP assumes that the Ivy > Bridge > integrated graphics are being used. > > > +Compliance > +========== > +This BSP is compliant with Yocto Project as per requirements listed here: > +http://www.yoctoproject.org/yocto-project-compatible-registration > +
To be consistent with the rest of the README, there should be a blank line between the ===== and the 'This BSP... ' text. Also, for readability, it would be nice to have the URL also separated by a blank line from the preceding text, and indented a couple spaces like similar lines in the README. Also, I think the compliance text reads better as: This BSP is compliant with the Yocto Project as per the requirements listed here: > + > Dependencies > ============ > > @@ -20,7 +26,7 @@ This layer depends on: > branch: master > > URI: git://git.yoctoproject.org/meta-intel > - layers: intel > + layers: meta-intel, meta-chiefriver If you look at the meta-intel/conf/layer.conf, you see the layer actually is 'intel'. Also, this is the README for the chiefriver layer, so it can't depend on itself i.e. meta-chiefriver or 'chiefriver' shouldn't be listed in layers > branch: master > > > diff --git a/meta-chiefriver/conf/machine/chiefriver.conf > b/meta-chiefriver/conf/machine/chiefriver.conf > index b8b8754..5005ce0 100644 > --- a/meta-chiefriver/conf/machine/chiefriver.conf > +++ b/meta-chiefriver/conf/machine/chiefriver.conf > @@ -1,6 +1,8 @@ > #@TYPE: Machine > #@NAME: chiefriver > > +#@WEBTITLE: Intel 3rd Generations Core Platforms: Core i3, i5, i7 (Ivy > Bridge) > + Since this is text that will appear on the website, we need to be a little picky about grammar: it should read '3rd Generation' rather than '3rd Generations' These comments apply to the other README files as well, so I won't repeat them there, other than to fix any text that might appear on the website for those as well... Tom > #@DESCRIPTION: Machine configuration for Chief River systems > # i.e. Ivy Bridge + Panther Point > _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto
