[yocto] [PATCH v2 2/6] chiefriver: add WEBTITLE & Compliance information
Tom Zanussi
tom.zanussi at intel.com
Wed Oct 24 17:14:58 PDT 2012
On Wed, 2012-10-24 at 13:25 -0700, nitin.a.kamble at intel.com wrote:
> From: Nitin A Kamble <nitin.a.kamble at 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.kamble at 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
>
More information about the yocto
mailing list