[oe] [PATCH 1/2] matrix-gui: updating SRC rev to pick up new platforms and matrix code enhancements

[Denys Dmytriyenko]

On Tue, Dec 07, 2010 at 01:13:50PM -0600, Jeff Lance wrote:
> * Added Matrix Enhancements:
>   * Added new platforms am180x, am181x and am3517
>   * add control menubar at the top of matrix-gui (HTML change not code change)
>   * add descriptions to matrix applications (source code change)
>   * make matrix scalable with respect to number of applications.
>     * allow addtional main menu pages and allow additional sub-menu pages (source code change)
> 
> NOTE: code changes are backwards compatible with existing platform support files.

Jeff, Chase,

The only comment I have is that in the first patch you add new .bb files with 
latest SRCREV and for some reason bump INC_PR in the .inc files, altough there 
are no changes there. Then in the second patch you update the .inc files and 
don't bump the INC_PR. It would make more sense to increment INC_PR as part of 
the second patch and not the first.

-- 
Denys


> Signed-off-by: Jeff Lance <j-lance1 at ti.com>
> Signed-off-by: Chase Maupin <chase.maupin at ti.com>
> ---
>  recipes/ti/matrix-gui-common.inc    |    2 +-
>  recipes/ti/matrix-gui-common_1.4.bb |    8 ++++++++
>  recipes/ti/matrix-gui-e.inc         |    2 +-
>  recipes/ti/matrix-gui-e_1.3.bb      |    5 +++++
>  recipes/ti/matrix-gui.inc           |    2 +-
>  recipes/ti/matrix-gui_1.3.bb        |    4 ++++
>  6 files changed, 20 insertions(+), 3 deletions(-)
>  create mode 100644 recipes/ti/matrix-gui-common_1.4.bb
>  create mode 100644 recipes/ti/matrix-gui-e_1.3.bb
>  create mode 100644 recipes/ti/matrix-gui_1.3.bb
> 
> diff --git a/recipes/ti/matrix-gui-common.inc b/recipes/ti/matrix-gui-common.inc
> index b5a368e..2d2cea3 100644
> --- a/recipes/ti/matrix-gui-common.inc
> +++ b/recipes/ti/matrix-gui-common.inc
> @@ -4,7 +4,7 @@ LICENSE = "BSD"
>  SECTION = "multimedia"
>  PRIORITY = "optional"
>  
> -INC_PR = "r5"
> +INC_PR = "r6"
>  
>  COMPATIBLE_MACHINE = "(dm365|omapl138|omap3|ti816x)"
>  
> diff --git a/recipes/ti/matrix-gui-common_1.4.bb b/recipes/ti/matrix-gui-common_1.4.bb
> new file mode 100644
> index 0000000..c578a47
> --- /dev/null
> +++ b/recipes/ti/matrix-gui-common_1.4.bb
> @@ -0,0 +1,8 @@
> +require matrix-gui-common.inc
> +
> +# Use the mem_util application from am-sysinfo instead of a
> +# prebuilt version in the repository.
> +RRECOMMENDS_${PN} = "am-sysinfo"
> +
> +SRCREV = "192"
> +PR = "${INC_PR}.0"
> diff --git a/recipes/ti/matrix-gui-e.inc b/recipes/ti/matrix-gui-e.inc
> index 86f08b1..c3a9341 100644
> --- a/recipes/ti/matrix-gui-e.inc
> +++ b/recipes/ti/matrix-gui-e.inc
> @@ -4,7 +4,7 @@ LICENSE = "BSD"
>  SECTION = "multimedia"
>  PRIORITY = "optional"
>  
> -INC_PR = "r20"
> +INC_PR = "r21"
>  
>  COMPATIBLE_MACHINE = "(dm365|omapl138|omap3|ti816x)"
>  
> diff --git a/recipes/ti/matrix-gui-e_1.3.bb b/recipes/ti/matrix-gui-e_1.3.bb
> new file mode 100644
> index 0000000..dd1b16b
> --- /dev/null
> +++ b/recipes/ti/matrix-gui-e_1.3.bb
> @@ -0,0 +1,5 @@
> +require matrix-gui-e.inc
> +
> +SRCREV = "192"
> +PR = "${INC_PR}.0"
> +
> diff --git a/recipes/ti/matrix-gui.inc b/recipes/ti/matrix-gui.inc
> index a2a2721..b7678b2 100644
> --- a/recipes/ti/matrix-gui.inc
> +++ b/recipes/ti/matrix-gui.inc
> @@ -4,7 +4,7 @@ LICENSE = "BSD"
>  SECTION = "multimedia"
>  PRIORITY = "optional"
>  
> -INC_PR = "r18"
> +INC_PR = "r19"
>  
>  COMPATIBLE_MACHINE = "(dm365|omapl138|omap3|ti816x)"
>  
> diff --git a/recipes/ti/matrix-gui_1.3.bb b/recipes/ti/matrix-gui_1.3.bb
> new file mode 100644
> index 0000000..53db8af
> --- /dev/null
> +++ b/recipes/ti/matrix-gui_1.3.bb
> @@ -0,0 +1,4 @@
> +require matrix-gui.inc
> +
> +SRCREV = "192"
> +PR = "${INC_PR}.0"
> -- 
> 1.6.3.3
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel