All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
@ 2014-03-03 20:01 Sam Nelson
  2014-03-14 12:43 ` Maupin, Chase
  2014-03-17 19:58 ` Maupin, Chase
  0 siblings, 2 replies; 10+ messages in thread
From: Sam Nelson @ 2014-03-03 20:01 UTC (permalink / raw)
  To: meta-arago

- Recipes bring images and menus needed for keystone2 demos

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
---
 .../matrix/matrix-gui-apps-ks2-git.inc             |   13 +++++
 .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19 ++++++++
 .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51 ++++++++++++++++++++
 .../matrix/matrix-gui-demos-demos_2.0.bb           |   18 +++++++
 .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38 +++++++++++++++
 .../matrix/matrix-gui-utility-demos_2.0.bb         |   18 +++++++
 6 files changed, 157 insertions(+)
 create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-git.inc
 create mode 100644 meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-images_2.0.bb
 create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2.inc
 create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-gui-demos-demos_2.0.bb
 create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-ks2_2.0.bb
 create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-gui-utility-demos_2.0.bb

diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-git.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-git.inc
new file mode 100755
index 0000000..4a7b21d
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-git.inc
@@ -0,0 +1,13 @@
+LICENSE = "CC-BY-SA"
+#By default all Matrix v2 applications use the same license.
+LICENSE_CHECKSUM = "LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
+
+BRANCH ?= "master"
+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
+SRC_URI = "git://git.ti.com/keystone-demos/matrix-apps.git;protocol=git;branch=${BRANCH}"
+INC_PR = "r0"
+
+# Pull in the base package for installing matrix applications
+require matrix-gui-apps-ks2.inc
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-images_2.0.bb
new file mode 100644
index 0000000..cf69d81
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-images_2.0.bb
@@ -0,0 +1,19 @@
+DESCRIPTION = "Images package for Matrix GUI v2 Applications for Keystone II"
+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-demos/matrix-apps.git"
+LICENSE = "CC-BY-SA"
+
+require matrix-gui-apps-ks2-git.inc
+require matrix-gui-paths.inc
+
+PR = "${INC_PR}.0"
+
+inherit allarch
+
+S = "${WORKDIR}/git/images"
+
+do_install(){
+    install -d ${D}${MATRIX_APP_DIR}
+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
+}
+
+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2.inc
new file mode 100755
index 0000000..fb3c866
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2.inc
@@ -0,0 +1,51 @@
+require matrix-gui-paths.inc
+
+MATRIX_FILES_DIR ?= "${S}"
+
+# Append a generic function to the do_install step that will look in the
+# MATRIX_FILES_DIR for .desktop files and then:
+#   1.  Install all non-script files into the MATRIX_APP_DIR
+#   2.  Install all script files into ${bindir}
+do_install_append(){
+    cd ${MATRIX_FILES_DIR}
+    applist=`find . -name "*.desktop"`
+
+    for app in $applist
+    do
+        appdir=`dirname $app`
+        install -d ${D}${MATRIX_APP_DIR}/$appdir
+
+        # Get the matrix metadata
+        matrix_desktop=`find $appdir -type f -name  "*.desktop"`
+        for m in $matrix_desktop
+        do
+            install -m 0644 ${MATRIX_FILES_DIR}/$m ${D}${MATRIX_APP_DIR}/$appdir/
+        done
+
+        matrix_html=`find $appdir -type f -name  "*.html"`
+        for m in $matrix_html
+        do
+            install -m 0644 ${MATRIX_FILES_DIR}/$m ${D}${MATRIX_APP_DIR}/$appdir/
+        done
+
+        # Get the script files as executables
+        scripts=`find $appdir -type f -name "*.sh"`
+        for s in $scripts
+        do
+            install -m 0755 ${MATRIX_FILES_DIR}/$s ${D}${MATRIX_APP_DIR}/$appdir/
+        done
+        
+        # Get the cgi files as executables
+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
+
+        for c in $matrix_cgi
+        do
+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
+            install -m 0755 ${MATRIX_FILES_DIR}/$c ${D}${MATRIX_APP_DIR}/$c
+        done    
+    done
+
+    # Go back to the directory we started from in case there are
+    # other appends.
+    cd -
+}
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-demos_2.0.bb
new file mode 100755
index 0000000..eb89cca
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-demos_2.0.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = "Demo description for Matrix v2"
+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-demos/matrix-apps.git"
+LICENSE = "CC-BY-SA"
+
+require matrix-gui-apps-ks2-git.inc
+
+PR = "${INC_PR}.0"
+
+inherit allarch
+
+S = "${WORKDIR}/git/demo_apps"
+
+PACKAGES += "${PN}-demos"
+
+# Make sure crypto submenu and app images has been installed. Also make sure openssl is available
+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-submenus-ks2-demos"
+
+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-ks2_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-ks2_2.0.bb
new file mode 100755
index 0000000..2936ad1
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-ks2_2.0.bb
@@ -0,0 +1,38 @@
+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone II"
+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-demos/matrix-apps.git"
+LICENSE = "CC-BY-SA"
+
+require matrix-gui-apps-ks2-git.inc
+require matrix-gui-paths.inc
+
+# This package does not use a subdirectory as ${S} so we need to
+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-git.inc file
+LIC_FILES_CHKSUM = "file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
+
+PR = "${INC_PR}.0"
+
+# These packages make submenus in matrix and are not architecture specific
+inherit allarch
+
+S = "${WORKDIR}/git"
+
+# List of submenus to build packages for
+SUBMENUS = "utilities_submenu demos_submenu"
+
+do_install(){
+    install -d ${D}${MATRIX_APP_DIR}
+
+    for x in ${SUBMENUS}
+    do
+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
+    done
+}
+
+PACKAGES += "${PN}-utility ${PN}-demos"
+
+# Make sure app images has been installed
+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
+
+# Add the files for each submenu package
+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-demos_2.0.bb
new file mode 100755
index 0000000..8f769e9
--- /dev/null
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-demos_2.0.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-demos/matrix-apps.git"
+LICENSE = "CC-BY-SA"
+
+require matrix-gui-apps-ks2-git.inc
+
+PR = "${INC_PR}.0"
+
+inherit allarch
+
+S = "${WORKDIR}/git/utility_apps"
+
+PACKAGES += "${PN}-utility"
+
+# Make sure crypto submenu and app images has been installed. Also make sure openssl is available
+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-submenus-ks2-utility"
+
+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-03 20:01 [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms Sam Nelson
@ 2014-03-14 12:43 ` Maupin, Chase
  2014-03-17 18:48   ` Denys Dmytriyenko
  2014-03-17 19:58 ` Maupin, Chase
  1 sibling, 1 reply; 10+ messages in thread
From: Maupin, Chase @ 2014-03-14 12:43 UTC (permalink / raw)
  To: Nelson, Sam, meta-arago

>-----Original Message-----
>From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>bounces@arago-project.org] On Behalf Of Nelson, Sam
>Sent: Monday, March 03, 2014 2:02 PM
>To: meta-arago@arago-project.org
>Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
>demos for keystone 2 platforms
>
>- Recipes bring images and menus needed for keystone2 demos

NAK

These should be pushed to the shared matrix repositories.  There is no reason to duplicate all these recipes and include files.  The purpose of matrix being modular is to enable you to share and not fork.  If you add these to the shared repo then you can create simple recipes to package you individual demos and install those for ks2 devices.  Please refer to how this is done for other devices.

As a side note, if you were going to do tis you should break this up into separate patches for the different recipes, not one mega patch adding everything.  Also, you have recipes like matrix-gui-demos-demos which doesn't seem right.

>
>Signed-off-by: Sam Nelson <sam.nelson@ti.com>
>---
> .../matrix/matrix-gui-apps-ks2-git.inc             |   13 +++++
> .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
>++++++++
> .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
>++++++++++++++++++++
> .../matrix/matrix-gui-demos-demos_2.0.bb           |   18 +++++++
> .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
>+++++++++++++++
> .../matrix/matrix-gui-utility-demos_2.0.bb         |   18 +++++++
> 6 files changed, 157 insertions(+)
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-git.inc
> create mode 100644 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-images_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2.inc
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-demos-demos_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-submenus-ks2_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-utility-demos_2.0.bb
>
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2-git.inc b/meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-git.inc
>new file mode 100755
>index 0000000..4a7b21d
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
>git.inc
>@@ -0,0 +1,13 @@
>+LICENSE = "CC-BY-SA"
>+#By default all Matrix v2 applications use the same license.
>+LICENSE_CHECKSUM = "LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
>+
>+BRANCH ?= "master"
>+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
>+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
>+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
>apps.git;protocol=git;branch=${BRANCH}"
>+INC_PR = "r0"
>+
>+# Pull in the base package for installing matrix applications
>+require matrix-gui-apps-ks2.inc
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-apps-ks2-images_2.0.bb
>new file mode 100644
>index 0000000..cf69d81
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
>images_2.0.bb
>@@ -0,0 +1,19 @@
>+DESCRIPTION = "Images package for Matrix GUI v2 Applications for
>Keystone II"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+require matrix-gui-paths.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/images"
>+
>+do_install(){
>+    install -d ${D}${MATRIX_APP_DIR}
>+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
>+}
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2.inc
>new file mode 100755
>index 0000000..fb3c866
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
>ks2.inc
>@@ -0,0 +1,51 @@
>+require matrix-gui-paths.inc
>+
>+MATRIX_FILES_DIR ?= "${S}"
>+
>+# Append a generic function to the do_install step that will look
>in the
>+# MATRIX_FILES_DIR for .desktop files and then:
>+#   1.  Install all non-script files into the MATRIX_APP_DIR
>+#   2.  Install all script files into ${bindir}
>+do_install_append(){
>+    cd ${MATRIX_FILES_DIR}
>+    applist=`find . -name "*.desktop"`
>+
>+    for app in $applist
>+    do
>+        appdir=`dirname $app`
>+        install -d ${D}${MATRIX_APP_DIR}/$appdir
>+
>+        # Get the matrix metadata
>+        matrix_desktop=`find $appdir -type f -name  "*.desktop"`
>+        for m in $matrix_desktop
>+        do
>+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done
>+
>+        matrix_html=`find $appdir -type f -name  "*.html"`
>+        for m in $matrix_html
>+        do
>+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done
>+
>+        # Get the script files as executables
>+        scripts=`find $appdir -type f -name "*.sh"`
>+        for s in $scripts
>+        do
>+            install -m 0755 ${MATRIX_FILES_DIR}/$s
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done
>+
>+        # Get the cgi files as executables
>+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
>+
>+        for c in $matrix_cgi
>+        do
>+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
>+            install -m 0755 ${MATRIX_FILES_DIR}/$c
>${D}${MATRIX_APP_DIR}/$c
>+        done
>+    done
>+
>+    # Go back to the directory we started from in case there are
>+    # other appends.
>+    cd -
>+}
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>demos-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-
>gui-demos-demos_2.0.bb
>new file mode 100755
>index 0000000..eb89cca
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
>demos_2.0.bb
>@@ -0,0 +1,18 @@
>+DESCRIPTION = "Demo description for Matrix v2"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/demo_apps"
>+
>+PACKAGES += "${PN}-demos"
>+
>+# Make sure crypto submenu and app images has been installed.
>Also make sure openssl is available
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>submenus-ks2-demos"
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-submenus-ks2_2.0.bb
>new file mode 100755
>index 0000000..2936ad1
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-
>ks2_2.0.bb
>@@ -0,0 +1,38 @@
>+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
>II"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+require matrix-gui-paths.inc
>+
>+# This package does not use a subdirectory as ${S} so we need to
>+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
>git.inc file
>+LIC_FILES_CHKSUM =
>"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>+
>+PR = "${INC_PR}.0"
>+
>+# These packages make submenus in matrix and are not architecture
>specific
>+inherit allarch
>+
>+S = "${WORKDIR}/git"
>+
>+# List of submenus to build packages for
>+SUBMENUS = "utilities_submenu demos_submenu"
>+
>+do_install(){
>+    install -d ${D}${MATRIX_APP_DIR}
>+
>+    for x in ${SUBMENUS}
>+    do
>+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
>+    done
>+}
>+
>+PACKAGES += "${PN}-utility ${PN}-demos"
>+
>+# Make sure app images has been installed
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
>+
>+# Add the files for each submenu package
>+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
>+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>utility-demos_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-utility-demos_2.0.bb
>new file mode 100755
>index 0000000..8f769e9
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
>demos_2.0.bb
>@@ -0,0 +1,18 @@
>+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/utility_apps"
>+
>+PACKAGES += "${PN}-utility"
>+
>+# Make sure crypto submenu and app images has been installed.
>Also make sure openssl is available
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>submenus-ks2-utility"
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>--
>1.7.9.5
>
>_______________________________________________
>meta-arago mailing list
>meta-arago@arago-project.org
>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-14 12:43 ` Maupin, Chase
@ 2014-03-17 18:48   ` Denys Dmytriyenko
  2014-03-17 19:46     ` Maupin, Chase
  0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2014-03-17 18:48 UTC (permalink / raw)
  To: Maupin, Chase; +Cc: meta-arago

On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> >-----Original Message-----
> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> >Sent: Monday, March 03, 2014 2:02 PM
> >To: meta-arago@arago-project.org
> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> >demos for keystone 2 platforms
> >
> >- Recipes bring images and menus needed for keystone2 demos
> 
> NAK
> 
> These should be pushed to the shared matrix repositories.  There is no 
> reason to duplicate all these recipes and include files.  The purpose of 
> matrix being modular is to enable you to share and not fork.  If you add 
> these to the shared repo then you can create simple recipes to package you 
> individual demos and install those for ks2 devices.  Please refer to how 
> this is done for other devices.

I don't get it. Specifically due to matrix being modular, it should be 
possible to have additional demos in separate repositories - the way it's 
done here. And the corresponding recipes that pull from those additional 
repositories and package them up for matrix to use. I don't see this as being 
a fork, I see this as being an out-of-tree extenstion. Getting those 
extensions in to the main tree has some benefits, but reduces control...


> As a side note, if you were going to do tis you should break this up into 
> separate patches for the different recipes, not one mega patch adding 
> everything.  Also, you have recipes like matrix-gui-demos-demos which 
> doesn't seem right.
> 
> >
> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> >---
> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13 +++++
> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> >++++++++
> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> >++++++++++++++++++++
> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18 +++++++
> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> >+++++++++++++++
> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18 +++++++
> > 6 files changed, 157 insertions(+)
> > create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-apps-ks2-git.inc
> > create mode 100644 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-apps-ks2-images_2.0.bb
> > create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-apps-ks2.inc
> > create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-demos-demos_2.0.bb
> > create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-submenus-ks2_2.0.bb
> > create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
> >gui-utility-demos_2.0.bb
> >
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >apps-ks2-git.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> >gui-apps-ks2-git.inc
> >new file mode 100755
> >index 0000000..4a7b21d
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
> >git.inc
> >@@ -0,0 +1,13 @@
> >+LICENSE = "CC-BY-SA"
> >+#By default all Matrix v2 applications use the same license.
> >+LICENSE_CHECKSUM = "LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> >+
> >+BRANCH ?= "master"
> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> >apps.git;protocol=git;branch=${BRANCH}"
> >+INC_PR = "r0"
> >+
> >+# Pull in the base package for installing matrix applications
> >+require matrix-gui-apps-ks2.inc
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> >new file mode 100644
> >index 0000000..cf69d81
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
> >images_2.0.bb
> >@@ -0,0 +1,19 @@
> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications for
> >Keystone II"
> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >demos/matrix-apps.git"
> >+LICENSE = "CC-BY-SA"
> >+
> >+require matrix-gui-apps-ks2-git.inc
> >+require matrix-gui-paths.inc
> >+
> >+PR = "${INC_PR}.0"
> >+
> >+inherit allarch
> >+
> >+S = "${WORKDIR}/git/images"
> >+
> >+do_install(){
> >+    install -d ${D}${MATRIX_APP_DIR}
> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> >+}
> >+
> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >apps-ks2.inc
> >new file mode 100755
> >index 0000000..fb3c866
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> >ks2.inc
> >@@ -0,0 +1,51 @@
> >+require matrix-gui-paths.inc
> >+
> >+MATRIX_FILES_DIR ?= "${S}"
> >+
> >+# Append a generic function to the do_install step that will look
> >in the
> >+# MATRIX_FILES_DIR for .desktop files and then:
> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> >+#   2.  Install all script files into ${bindir}
> >+do_install_append(){
> >+    cd ${MATRIX_FILES_DIR}
> >+    applist=`find . -name "*.desktop"`
> >+
> >+    for app in $applist
> >+    do
> >+        appdir=`dirname $app`
> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> >+
> >+        # Get the matrix metadata
> >+        matrix_desktop=`find $appdir -type f -name  "*.desktop"`
> >+        for m in $matrix_desktop
> >+        do
> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> >${D}${MATRIX_APP_DIR}/$appdir/
> >+        done
> >+
> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> >+        for m in $matrix_html
> >+        do
> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> >${D}${MATRIX_APP_DIR}/$appdir/
> >+        done
> >+
> >+        # Get the script files as executables
> >+        scripts=`find $appdir -type f -name "*.sh"`
> >+        for s in $scripts
> >+        do
> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> >${D}${MATRIX_APP_DIR}/$appdir/
> >+        done
> >+
> >+        # Get the cgi files as executables
> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> >+
> >+        for c in $matrix_cgi
> >+        do
> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> >${D}${MATRIX_APP_DIR}/$c
> >+        done
> >+    done
> >+
> >+    # Go back to the directory we started from in case there are
> >+    # other appends.
> >+    cd -
> >+}
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >demos-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-
> >gui-demos-demos_2.0.bb
> >new file mode 100755
> >index 0000000..eb89cca
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> >demos_2.0.bb
> >@@ -0,0 +1,18 @@
> >+DESCRIPTION = "Demo description for Matrix v2"
> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >demos/matrix-apps.git"
> >+LICENSE = "CC-BY-SA"
> >+
> >+require matrix-gui-apps-ks2-git.inc
> >+
> >+PR = "${INC_PR}.0"
> >+
> >+inherit allarch
> >+
> >+S = "${WORKDIR}/git/demo_apps"
> >+
> >+PACKAGES += "${PN}-demos"
> >+
> >+# Make sure crypto submenu and app images has been installed.
> >Also make sure openssl is available
> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> >submenus-ks2-demos"
> >+
> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> >new file mode 100755
> >index 0000000..2936ad1
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-
> >ks2_2.0.bb
> >@@ -0,0 +1,38 @@
> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> >II"
> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >demos/matrix-apps.git"
> >+LICENSE = "CC-BY-SA"
> >+
> >+require matrix-gui-apps-ks2-git.inc
> >+require matrix-gui-paths.inc
> >+
> >+# This package does not use a subdirectory as ${S} so we need to
> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> >git.inc file
> >+LIC_FILES_CHKSUM =
> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> >+
> >+PR = "${INC_PR}.0"
> >+
> >+# These packages make submenus in matrix and are not architecture
> >specific
> >+inherit allarch
> >+
> >+S = "${WORKDIR}/git"
> >+
> >+# List of submenus to build packages for
> >+SUBMENUS = "utilities_submenu demos_submenu"
> >+
> >+do_install(){
> >+    install -d ${D}${MATRIX_APP_DIR}
> >+
> >+    for x in ${SUBMENUS}
> >+    do
> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> >+    done
> >+}
> >+
> >+PACKAGES += "${PN}-utility ${PN}-demos"
> >+
> >+# Make sure app images has been installed
> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> >+
> >+# Add the files for each submenu package
> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> >core/matrix/matrix-gui-utility-demos_2.0.bb
> >new file mode 100755
> >index 0000000..8f769e9
> >--- /dev/null
> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> >demos_2.0.bb
> >@@ -0,0 +1,18 @@
> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >demos/matrix-apps.git"
> >+LICENSE = "CC-BY-SA"
> >+
> >+require matrix-gui-apps-ks2-git.inc
> >+
> >+PR = "${INC_PR}.0"
> >+
> >+inherit allarch
> >+
> >+S = "${WORKDIR}/git/utility_apps"
> >+
> >+PACKAGES += "${PN}-utility"
> >+
> >+# Make sure crypto submenu and app images has been installed.
> >Also make sure openssl is available
> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> >submenus-ks2-utility"
> >+
> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >--
> >1.7.9.5
> >
> >_______________________________________________
> >meta-arago mailing list
> >meta-arago@arago-project.org
> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-17 18:48   ` Denys Dmytriyenko
@ 2014-03-17 19:46     ` Maupin, Chase
  2014-03-17 19:57       ` Denys Dmytriyenko
  0 siblings, 1 reply; 10+ messages in thread
From: Maupin, Chase @ 2014-03-17 19:46 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago

>-----Original Message-----
>From: Dmytriyenko, Denys
>Sent: Monday, March 17, 2014 1:49 PM
>To: Maupin, Chase
>Cc: Nelson, Sam; meta-arago@arago-project.org
>Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
>demos for keystone 2 platforms
>
>On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
>> >-----Original Message-----
>> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>> >bounces@arago-project.org] On Behalf Of Nelson, Sam
>> >Sent: Monday, March 03, 2014 2:02 PM
>> >To: meta-arago@arago-project.org
>> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
>> >demos for keystone 2 platforms
>> >
>> >- Recipes bring images and menus needed for keystone2 demos
>>
>> NAK
>>
>> These should be pushed to the shared matrix repositories.  There
>is no
>> reason to duplicate all these recipes and include files.  The
>purpose of
>> matrix being modular is to enable you to share and not fork.  If
>you add
>> these to the shared repo then you can create simple recipes to
>package you
>> individual demos and install those for ks2 devices.  Please
>refer to how
>> this is done for other devices.
>
>I don't get it. Specifically due to matrix being modular, it
>should be
>possible to have additional demos in separate repositories - the
>way it's
>done here. And the corresponding recipes that pull from those
>additional
>repositories and package them up for matrix to use. I don't see
>this as being
>a fork, I see this as being an out-of-tree extenstion. Getting
>those
>extensions in to the main tree has some benefits, but reduces
>control...

It also tends to reduce re-use.  I think it is first worth looking into adding to the shared repo first.

Also, even if you want an external repo for additions it seems strange that the .inc files needed to be copied as well.  For example:

Matrix-gui-apps-ks2-git.inc is almost a direct copy of matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a mimimum why not inherit matrix-gui-apps-git.inc and change those few settings for your demos, images, submenu, etc recipes to inherit.

For matrix-gui-apps-ks2.inc there are also a few changes that could either be common or look wrong and don't match the comments above the install function.  I'll send some additional feedback on that section.  The main changes I see are:
	1. changing the .sh scripts to be installed into the ${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is contrary to all other matrix apps.
	2. there seem to be .cgi files added and they are being installed into the root of the ${MATRIX_APP_DIR} even though the directory ${MATRIX_APP_DIR}/$appdir/cgi-bin was created.

These changed could be rolled into the generic .inc instead of forking.

>
>
>> As a side note, if you were going to do tis you should break
>this up into
>> separate patches for the different recipes, not one mega patch
>adding
>> everything.  Also, you have recipes like matrix-gui-demos-demos
>which
>> doesn't seem right.
>>
>> >
>> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
>> >---
>> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
>+++++
>> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
>> >++++++++
>> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
>> >++++++++++++++++++++
>> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
>+++++++
>> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
>> >+++++++++++++++
>> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
>+++++++
>> > 6 files changed, 157 insertions(+)
>> > create mode 100755 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-apps-ks2-git.inc
>> > create mode 100644 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-apps-ks2-images_2.0.bb
>> > create mode 100755 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-apps-ks2.inc
>> > create mode 100755 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-demos-demos_2.0.bb
>> > create mode 100755 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-submenus-ks2_2.0.bb
>> > create mode 100755 meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-utility-demos_2.0.bb
>> >
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >apps-ks2-git.inc b/meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-apps-ks2-git.inc
>> >new file mode 100755
>> >index 0000000..4a7b21d
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
>ks2-
>> >git.inc
>> >@@ -0,0 +1,13 @@
>> >+LICENSE = "CC-BY-SA"
>> >+#By default all Matrix v2 applications use the same license.
>> >+LICENSE_CHECKSUM =
>"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
>> >+
>> >+BRANCH ?= "master"
>> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
>> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
>> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
>> >apps.git;protocol=git;branch=${BRANCH}"
>> >+INC_PR = "r0"
>> >+
>> >+# Pull in the base package for installing matrix applications
>> >+require matrix-gui-apps-ks2.inc
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
>> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
>> >new file mode 100644
>> >index 0000000..cf69d81
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
>ks2-
>> >images_2.0.bb
>> >@@ -0,0 +1,19 @@
>> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
>for
>> >Keystone II"
>> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>> >demos/matrix-apps.git"
>> >+LICENSE = "CC-BY-SA"
>> >+
>> >+require matrix-gui-apps-ks2-git.inc
>> >+require matrix-gui-paths.inc
>> >+
>> >+PR = "${INC_PR}.0"
>> >+
>> >+inherit allarch
>> >+
>> >+S = "${WORKDIR}/git/images"
>> >+
>> >+do_install(){
>> >+    install -d ${D}${MATRIX_APP_DIR}
>> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
>> >+}
>> >+
>> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
>gui-
>> >apps-ks2.inc
>> >new file mode 100755
>> >index 0000000..fb3c866
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
>> >ks2.inc
>> >@@ -0,0 +1,51 @@
>> >+require matrix-gui-paths.inc
>> >+
>> >+MATRIX_FILES_DIR ?= "${S}"
>> >+
>> >+# Append a generic function to the do_install step that will
>look
>> >in the
>> >+# MATRIX_FILES_DIR for .desktop files and then:
>> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
>> >+#   2.  Install all script files into ${bindir}
>> >+do_install_append(){
>> >+    cd ${MATRIX_FILES_DIR}
>> >+    applist=`find . -name "*.desktop"`
>> >+
>> >+    for app in $applist
>> >+    do
>> >+        appdir=`dirname $app`
>> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
>> >+
>> >+        # Get the matrix metadata
>> >+        matrix_desktop=`find $appdir -type f -name
>"*.desktop"`
>> >+        for m in $matrix_desktop
>> >+        do
>> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>> >${D}${MATRIX_APP_DIR}/$appdir/
>> >+        done
>> >+
>> >+        matrix_html=`find $appdir -type f -name  "*.html"`
>> >+        for m in $matrix_html
>> >+        do
>> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>> >${D}${MATRIX_APP_DIR}/$appdir/
>> >+        done
>> >+
>> >+        # Get the script files as executables
>> >+        scripts=`find $appdir -type f -name "*.sh"`
>> >+        for s in $scripts
>> >+        do
>> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
>> >${D}${MATRIX_APP_DIR}/$appdir/
>> >+        done
>> >+
>> >+        # Get the cgi files as executables
>> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
>> >+
>> >+        for c in $matrix_cgi
>> >+        do
>> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
>> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
>> >${D}${MATRIX_APP_DIR}/$c
>> >+        done
>> >+    done
>> >+
>> >+    # Go back to the directory we started from in case there
>are
>> >+    # other appends.
>> >+    cd -
>> >+}
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-
>> >gui-demos-demos_2.0.bb
>> >new file mode 100755
>> >index 0000000..eb89cca
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
>> >demos_2.0.bb
>> >@@ -0,0 +1,18 @@
>> >+DESCRIPTION = "Demo description for Matrix v2"
>> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>> >demos/matrix-apps.git"
>> >+LICENSE = "CC-BY-SA"
>> >+
>> >+require matrix-gui-apps-ks2-git.inc
>> >+
>> >+PR = "${INC_PR}.0"
>> >+
>> >+inherit allarch
>> >+
>> >+S = "${WORKDIR}/git/demo_apps"
>> >+
>> >+PACKAGES += "${PN}-demos"
>> >+
>> >+# Make sure crypto submenu and app images has been installed.
>> >Also make sure openssl is available
>> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>> >submenus-ks2-demos"
>> >+
>> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
>> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
>> >new file mode 100755
>> >index 0000000..2936ad1
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
>submenus-
>> >ks2_2.0.bb
>> >@@ -0,0 +1,38 @@
>> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
>> >II"
>> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>> >demos/matrix-apps.git"
>> >+LICENSE = "CC-BY-SA"
>> >+
>> >+require matrix-gui-apps-ks2-git.inc
>> >+require matrix-gui-paths.inc
>> >+
>> >+# This package does not use a subdirectory as ${S} so we need
>to
>> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
>> >git.inc file
>> >+LIC_FILES_CHKSUM =
>> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>> >+
>> >+PR = "${INC_PR}.0"
>> >+
>> >+# These packages make submenus in matrix and are not
>architecture
>> >specific
>> >+inherit allarch
>> >+
>> >+S = "${WORKDIR}/git"
>> >+
>> >+# List of submenus to build packages for
>> >+SUBMENUS = "utilities_submenu demos_submenu"
>> >+
>> >+do_install(){
>> >+    install -d ${D}${MATRIX_APP_DIR}
>> >+
>> >+    for x in ${SUBMENUS}
>> >+    do
>> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
>> >+    done
>> >+}
>> >+
>> >+PACKAGES += "${PN}-utility ${PN}-demos"
>> >+
>> >+# Make sure app images has been installed
>> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
>> >+
>> >+# Add the files for each submenu package
>> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
>> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
>> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
>> >core/matrix/matrix-gui-utility-demos_2.0.bb
>> >new file mode 100755
>> >index 0000000..8f769e9
>> >--- /dev/null
>> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
>> >demos_2.0.bb
>> >@@ -0,0 +1,18 @@
>> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
>> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>> >demos/matrix-apps.git"
>> >+LICENSE = "CC-BY-SA"
>> >+
>> >+require matrix-gui-apps-ks2-git.inc
>> >+
>> >+PR = "${INC_PR}.0"
>> >+
>> >+inherit allarch
>> >+
>> >+S = "${WORKDIR}/git/utility_apps"
>> >+
>> >+PACKAGES += "${PN}-utility"
>> >+
>> >+# Make sure crypto submenu and app images has been installed.
>> >Also make sure openssl is available
>> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>> >submenus-ks2-utility"
>> >+
>> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>> >--
>> >1.7.9.5
>> >
>> >_______________________________________________
>> >meta-arago mailing list
>> >meta-arago@arago-project.org
>> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-17 19:46     ` Maupin, Chase
@ 2014-03-17 19:57       ` Denys Dmytriyenko
  2014-03-17 20:59         ` Cooper Jr., Franklin
  0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2014-03-17 19:57 UTC (permalink / raw)
  To: Maupin, Chase; +Cc: meta-arago

On Mon, Mar 17, 2014 at 03:46:17PM -0400, Maupin, Chase wrote:
> >-----Original Message-----
> >From: Dmytriyenko, Denys
> >Sent: Monday, March 17, 2014 1:49 PM
> >To: Maupin, Chase
> >Cc: Nelson, Sam; meta-arago@arago-project.org
> >Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> >demos for keystone 2 platforms
> >
> >On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> >> >-----Original Message-----
> >> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> >> >Sent: Monday, March 03, 2014 2:02 PM
> >> >To: meta-arago@arago-project.org
> >> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> >> >demos for keystone 2 platforms
> >> >
> >> >- Recipes bring images and menus needed for keystone2 demos
> >>
> >> NAK
> >>
> >> These should be pushed to the shared matrix repositories.  There
> >is no
> >> reason to duplicate all these recipes and include files.  The
> >purpose of
> >> matrix being modular is to enable you to share and not fork.  If
> >you add
> >> these to the shared repo then you can create simple recipes to
> >package you
> >> individual demos and install those for ks2 devices.  Please
> >refer to how
> >> this is done for other devices.
> >
> >I don't get it. Specifically due to matrix being modular, it
> >should be
> >possible to have additional demos in separate repositories - the
> >way it's
> >done here. And the corresponding recipes that pull from those
> >additional
> >repositories and package them up for matrix to use. I don't see
> >this as being
> >a fork, I see this as being an out-of-tree extenstion. Getting
> >those
> >extensions in to the main tree has some benefits, but reduces
> >control...
> 
> It also tends to reduce re-use.  I think it is first worth looking into 
> adding to the shared repo first.

Then we should outline the process to get 3rd-party changes to the matrix 
repositories (patches, pull requests). Is Franklin the current owner?


> Also, even if you want an external repo for additions it seems strange that 
> the .inc files needed to be copied as well.  For example:

Indeed, I was not commenting on the recipes themselves, but on the method to 
add additional functionality to the matrix. Recipes do require more work.


> Matrix-gui-apps-ks2-git.inc is almost a direct copy of 
> matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a mimimum 
> why not inherit matrix-gui-apps-git.inc and change those few settings for 
> your demos, images, submenu, etc recipes to inherit.
> 
> For matrix-gui-apps-ks2.inc there are also a few changes that could either 
> be common or look wrong and don't match the comments above the install 
> function.  I'll send some additional feedback on that section.  The main 
> changes I see are:
> 	1. changing the .sh scripts to be installed into the 
> 	${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is 
> 	contrary to all other matrix apps.
> 	2. there seem to be .cgi files added and they are being installed into 
> 	the root of the ${MATRIX_APP_DIR} even though the directory 
> 	${MATRIX_APP_DIR}/$appdir/cgi-bin was created.
> 
> These changed could be rolled into the generic .inc instead of forking.
> 
> >
> >
> >> As a side note, if you were going to do tis you should break
> >this up into
> >> separate patches for the different recipes, not one mega patch
> >adding
> >> everything.  Also, you have recipes like matrix-gui-demos-demos
> >which
> >> doesn't seem right.
> >>
> >> >
> >> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> >> >---
> >> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
> >+++++
> >> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> >> >++++++++
> >> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> >> >++++++++++++++++++++
> >> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
> >+++++++
> >> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> >> >+++++++++++++++
> >> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
> >+++++++
> >> > 6 files changed, 157 insertions(+)
> >> > create mode 100755 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-apps-ks2-git.inc
> >> > create mode 100644 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-apps-ks2-images_2.0.bb
> >> > create mode 100755 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-apps-ks2.inc
> >> > create mode 100755 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-demos-demos_2.0.bb
> >> > create mode 100755 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-submenus-ks2_2.0.bb
> >> > create mode 100755 meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-utility-demos_2.0.bb
> >> >
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >apps-ks2-git.inc b/meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-apps-ks2-git.inc
> >> >new file mode 100755
> >> >index 0000000..4a7b21d
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> >ks2-
> >> >git.inc
> >> >@@ -0,0 +1,13 @@
> >> >+LICENSE = "CC-BY-SA"
> >> >+#By default all Matrix v2 applications use the same license.
> >> >+LICENSE_CHECKSUM =
> >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> >> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> >> >+
> >> >+BRANCH ?= "master"
> >> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> >> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> >> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> >> >apps.git;protocol=git;branch=${BRANCH}"
> >> >+INC_PR = "r0"
> >> >+
> >> >+# Pull in the base package for installing matrix applications
> >> >+require matrix-gui-apps-ks2.inc
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> >> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> >> >new file mode 100644
> >> >index 0000000..cf69d81
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> >ks2-
> >> >images_2.0.bb
> >> >@@ -0,0 +1,19 @@
> >> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
> >for
> >> >Keystone II"
> >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >> >demos/matrix-apps.git"
> >> >+LICENSE = "CC-BY-SA"
> >> >+
> >> >+require matrix-gui-apps-ks2-git.inc
> >> >+require matrix-gui-paths.inc
> >> >+
> >> >+PR = "${INC_PR}.0"
> >> >+
> >> >+inherit allarch
> >> >+
> >> >+S = "${WORKDIR}/git/images"
> >> >+
> >> >+do_install(){
> >> >+    install -d ${D}${MATRIX_APP_DIR}
> >> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> >> >+}
> >> >+
> >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> >gui-
> >> >apps-ks2.inc
> >> >new file mode 100755
> >> >index 0000000..fb3c866
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> >> >ks2.inc
> >> >@@ -0,0 +1,51 @@
> >> >+require matrix-gui-paths.inc
> >> >+
> >> >+MATRIX_FILES_DIR ?= "${S}"
> >> >+
> >> >+# Append a generic function to the do_install step that will
> >look
> >> >in the
> >> >+# MATRIX_FILES_DIR for .desktop files and then:
> >> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> >> >+#   2.  Install all script files into ${bindir}
> >> >+do_install_append(){
> >> >+    cd ${MATRIX_FILES_DIR}
> >> >+    applist=`find . -name "*.desktop"`
> >> >+
> >> >+    for app in $applist
> >> >+    do
> >> >+        appdir=`dirname $app`
> >> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> >> >+
> >> >+        # Get the matrix metadata
> >> >+        matrix_desktop=`find $appdir -type f -name
> >"*.desktop"`
> >> >+        for m in $matrix_desktop
> >> >+        do
> >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> >> >${D}${MATRIX_APP_DIR}/$appdir/
> >> >+        done
> >> >+
> >> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> >> >+        for m in $matrix_html
> >> >+        do
> >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> >> >${D}${MATRIX_APP_DIR}/$appdir/
> >> >+        done
> >> >+
> >> >+        # Get the script files as executables
> >> >+        scripts=`find $appdir -type f -name "*.sh"`
> >> >+        for s in $scripts
> >> >+        do
> >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> >> >${D}${MATRIX_APP_DIR}/$appdir/
> >> >+        done
> >> >+
> >> >+        # Get the cgi files as executables
> >> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> >> >+
> >> >+        for c in $matrix_cgi
> >> >+        do
> >> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> >> >${D}${MATRIX_APP_DIR}/$c
> >> >+        done
> >> >+    done
> >> >+
> >> >+    # Go back to the directory we started from in case there
> >are
> >> >+    # other appends.
> >> >+    cd -
> >> >+}
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
> >core/matrix/matrix-
> >> >gui-demos-demos_2.0.bb
> >> >new file mode 100755
> >> >index 0000000..eb89cca
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> >> >demos_2.0.bb
> >> >@@ -0,0 +1,18 @@
> >> >+DESCRIPTION = "Demo description for Matrix v2"
> >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >> >demos/matrix-apps.git"
> >> >+LICENSE = "CC-BY-SA"
> >> >+
> >> >+require matrix-gui-apps-ks2-git.inc
> >> >+
> >> >+PR = "${INC_PR}.0"
> >> >+
> >> >+inherit allarch
> >> >+
> >> >+S = "${WORKDIR}/git/demo_apps"
> >> >+
> >> >+PACKAGES += "${PN}-demos"
> >> >+
> >> >+# Make sure crypto submenu and app images has been installed.
> >> >Also make sure openssl is available
> >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> >> >submenus-ks2-demos"
> >> >+
> >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> >> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> >> >new file mode 100755
> >> >index 0000000..2936ad1
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >submenus-
> >> >ks2_2.0.bb
> >> >@@ -0,0 +1,38 @@
> >> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> >> >II"
> >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >> >demos/matrix-apps.git"
> >> >+LICENSE = "CC-BY-SA"
> >> >+
> >> >+require matrix-gui-apps-ks2-git.inc
> >> >+require matrix-gui-paths.inc
> >> >+
> >> >+# This package does not use a subdirectory as ${S} so we need
> >to
> >> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> >> >git.inc file
> >> >+LIC_FILES_CHKSUM =
> >> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> >> >+
> >> >+PR = "${INC_PR}.0"
> >> >+
> >> >+# These packages make submenus in matrix and are not
> >architecture
> >> >specific
> >> >+inherit allarch
> >> >+
> >> >+S = "${WORKDIR}/git"
> >> >+
> >> >+# List of submenus to build packages for
> >> >+SUBMENUS = "utilities_submenu demos_submenu"
> >> >+
> >> >+do_install(){
> >> >+    install -d ${D}${MATRIX_APP_DIR}
> >> >+
> >> >+    for x in ${SUBMENUS}
> >> >+    do
> >> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> >> >+    done
> >> >+}
> >> >+
> >> >+PACKAGES += "${PN}-utility ${PN}-demos"
> >> >+
> >> >+# Make sure app images has been installed
> >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> >> >+
> >> >+# Add the files for each submenu package
> >> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> >> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> >> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> >> >core/matrix/matrix-gui-utility-demos_2.0.bb
> >> >new file mode 100755
> >> >index 0000000..8f769e9
> >> >--- /dev/null
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> >> >demos_2.0.bb
> >> >@@ -0,0 +1,18 @@
> >> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> >> >demos/matrix-apps.git"
> >> >+LICENSE = "CC-BY-SA"
> >> >+
> >> >+require matrix-gui-apps-ks2-git.inc
> >> >+
> >> >+PR = "${INC_PR}.0"
> >> >+
> >> >+inherit allarch
> >> >+
> >> >+S = "${WORKDIR}/git/utility_apps"
> >> >+
> >> >+PACKAGES += "${PN}-utility"
> >> >+
> >> >+# Make sure crypto submenu and app images has been installed.
> >> >Also make sure openssl is available
> >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> >> >submenus-ks2-utility"
> >> >+
> >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> >> >--
> >> >1.7.9.5
> >> >
> >> >_______________________________________________
> >> >meta-arago mailing list
> >> >meta-arago@arago-project.org
> >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >> _______________________________________________
> >> meta-arago mailing list
> >> meta-arago@arago-project.org
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-03 20:01 [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms Sam Nelson
  2014-03-14 12:43 ` Maupin, Chase
@ 2014-03-17 19:58 ` Maupin, Chase
  1 sibling, 0 replies; 10+ messages in thread
From: Maupin, Chase @ 2014-03-17 19:58 UTC (permalink / raw)
  To: Nelson, Sam, meta-arago

>-----Original Message-----
>From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>bounces@arago-project.org] On Behalf Of Nelson, Sam
>Sent: Monday, March 03, 2014 2:02 PM
>To: meta-arago@arago-project.org
>Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
>demos for keystone 2 platforms
>
>- Recipes bring images and menus needed for keystone2 demos
>
>Signed-off-by: Sam Nelson <sam.nelson@ti.com>
>---
> .../matrix/matrix-gui-apps-ks2-git.inc             |   13 +++++
> .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
>++++++++
> .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
>++++++++++++++++++++
> .../matrix/matrix-gui-demos-demos_2.0.bb           |   18 +++++++
> .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
>+++++++++++++++
> .../matrix/matrix-gui-utility-demos_2.0.bb         |   18 +++++++
> 6 files changed, 157 insertions(+)
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-git.inc
> create mode 100644 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-images_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2.inc
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-demos-demos_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-submenus-ks2_2.0.bb
> create mode 100755 meta-arago-extras/recipes-core/matrix/matrix-
>gui-utility-demos_2.0.bb
>
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2-git.inc b/meta-arago-extras/recipes-core/matrix/matrix-
>gui-apps-ks2-git.inc
>new file mode 100755
>index 0000000..4a7b21d
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
>git.inc
>@@ -0,0 +1,13 @@
>+LICENSE = "CC-BY-SA"
>+#By default all Matrix v2 applications use the same license.
>+LICENSE_CHECKSUM = "LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
>+
>+BRANCH ?= "master"
>+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
>+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
>+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
>apps.git;protocol=git;branch=${BRANCH}"
>+INC_PR = "r0"

Why not re-use matrix-gui-apps-git.inc for most of this file and change only the SRCREV and SRC_URI settings?

>+
>+# Pull in the base package for installing matrix applications
>+require matrix-gui-apps-ks2.inc

Re-use the matrix-gui-apps.inc file.  See comments below.

>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-apps-ks2-images_2.0.bb
>new file mode 100644
>index 0000000..cf69d81
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-ks2-
>images_2.0.bb
>@@ -0,0 +1,19 @@
>+DESCRIPTION = "Images package for Matrix GUI v2 Applications for
>Keystone II"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+require matrix-gui-paths.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/images"
>+
>+do_install(){
>+    install -d ${D}${MATRIX_APP_DIR}
>+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
>+}
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-
>apps-ks2.inc
>new file mode 100755
>index 0000000..fb3c866
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
>ks2.inc
>@@ -0,0 +1,51 @@
>+require matrix-gui-paths.inc
>+
>+MATRIX_FILES_DIR ?= "${S}"
>+
>+# Append a generic function to the do_install step that will look
>in the
>+# MATRIX_FILES_DIR for .desktop files and then:
>+#   1.  Install all non-script files into the MATRIX_APP_DIR
>+#   2.  Install all script files into ${bindir}

These comments do not match the function below.

>+do_install_append(){
>+    cd ${MATRIX_FILES_DIR}
>+    applist=`find . -name "*.desktop"`
>+
>+    for app in $applist
>+    do
>+        appdir=`dirname $app`
>+        install -d ${D}${MATRIX_APP_DIR}/$appdir
>+
>+        # Get the matrix metadata
>+        matrix_desktop=`find $appdir -type f -name  "*.desktop"`

Isn't the .desktop file already known as $app at this point?

>+        for m in $matrix_desktop
>+        do
>+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done
>+
>+        matrix_html=`find $appdir -type f -name  "*.html"`
>+        for m in $matrix_html
>+        do
>+            install -m 0644 ${MATRIX_FILES_DIR}/$m
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done
>+
>+        # Get the script files as executables
>+        scripts=`find $appdir -type f -name "*.sh"`
>+        for s in $scripts
>+        do
>+            install -m 0755 ${MATRIX_FILES_DIR}/$s
>${D}${MATRIX_APP_DIR}/$appdir/
>+        done

Why change these from being installed in ${bindir} like other matrix app helper scripts?  This way user's can easily run the scripts as they are already in the PATH.

>+
>+        # Get the cgi files as executables
>+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
>+
>+        for c in $matrix_cgi
>+        do
>+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin

Why create this directory and not use it?

>+            install -m 0755 ${MATRIX_FILES_DIR}/$c
>${D}${MATRIX_APP_DIR}/$c

Is this the right place for your cgi files?

>+        done
>+    done
>+
>+    # Go back to the directory we started from in case there are
>+    # other appends.
>+    cd -

Overall it seems like if your only addition is that you may have cgi files you would be better off adding that support to the orignal .inc file

>+}
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>demos-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-
>gui-demos-demos_2.0.bb
>new file mode 100755
>index 0000000..eb89cca
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
>demos_2.0.bb
>@@ -0,0 +1,18 @@
>+DESCRIPTION = "Demo description for Matrix v2"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/demo_apps"
>+
>+PACKAGES += "${PN}-demos"
>+
>+# Make sure crypto submenu and app images has been installed.
>Also make sure openssl is available
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>submenus-ks2-demos"
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-submenus-ks2_2.0.bb
>new file mode 100755
>index 0000000..2936ad1
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-submenus-
>ks2_2.0.bb
>@@ -0,0 +1,38 @@
>+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
>II"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+require matrix-gui-paths.inc
>+
>+# This package does not use a subdirectory as ${S} so we need to
>+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
>git.inc file
>+LIC_FILES_CHKSUM =
>"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>+
>+PR = "${INC_PR}.0"
>+
>+# These packages make submenus in matrix and are not architecture
>specific
>+inherit allarch
>+
>+S = "${WORKDIR}/git"
>+
>+# List of submenus to build packages for
>+SUBMENUS = "utilities_submenu demos_submenu"
>+
>+do_install(){
>+    install -d ${D}${MATRIX_APP_DIR}
>+
>+    for x in ${SUBMENUS}
>+    do
>+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
>+    done
>+}
>+
>+PACKAGES += "${PN}-utility ${PN}-demos"
>+
>+# Make sure app images has been installed
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
>+
>+# Add the files for each submenu package
>+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
>+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
>diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
>utility-demos_2.0.bb b/meta-arago-extras/recipes-
>core/matrix/matrix-gui-utility-demos_2.0.bb
>new file mode 100755
>index 0000000..8f769e9
>--- /dev/null
>+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
>demos_2.0.bb
>@@ -0,0 +1,18 @@
>+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
>+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
>demos/matrix-apps.git"
>+LICENSE = "CC-BY-SA"
>+
>+require matrix-gui-apps-ks2-git.inc
>+
>+PR = "${INC_PR}.0"
>+
>+inherit allarch
>+
>+S = "${WORKDIR}/git/utility_apps"
>+
>+PACKAGES += "${PN}-utility"
>+
>+# Make sure crypto submenu and app images has been installed.
>Also make sure openssl is available
>+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
>submenus-ks2-utility"
>+
>+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
>--
>1.7.9.5
>
>_______________________________________________
>meta-arago mailing list
>meta-arago@arago-project.org
>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-17 19:57       ` Denys Dmytriyenko
@ 2014-03-17 20:59         ` Cooper Jr., Franklin
  2015-02-20  2:40           ` Denys Dmytriyenko
  0 siblings, 1 reply; 10+ messages in thread
From: Cooper Jr., Franklin @ 2014-03-17 20:59 UTC (permalink / raw)
  To: Dmytriyenko, Denys, Maupin, Chase; +Cc: meta-arago



> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> Sent: Monday, March 17, 2014 2:57 PM
> To: Maupin, Chase
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix demos for
> keystone 2 platforms
> 
> On Mon, Mar 17, 2014 at 03:46:17PM -0400, Maupin, Chase wrote:
> > >-----Original Message-----
> > >From: Dmytriyenko, Denys
> > >Sent: Monday, March 17, 2014 1:49 PM
> > >To: Maupin, Chase
> > >Cc: Nelson, Sam; meta-arago@arago-project.org
> > >Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > >demos for keystone 2 platforms
> > >
> > >On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> > >> >-----Original Message-----
> > >> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > >> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> > >> >Sent: Monday, March 03, 2014 2:02 PM
> > >> >To: meta-arago@arago-project.org
> > >> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > >> >demos for keystone 2 platforms
> > >> >
> > >> >- Recipes bring images and menus needed for keystone2 demos
> > >>
> > >> NAK
> > >>
> > >> These should be pushed to the shared matrix repositories.  There
> > >is no
> > >> reason to duplicate all these recipes and include files.  The
> > >purpose of
> > >> matrix being modular is to enable you to share and not fork.  If
> > >you add
> > >> these to the shared repo then you can create simple recipes to
> > >package you
> > >> individual demos and install those for ks2 devices.  Please
> > >refer to how
> > >> this is done for other devices.
> > >
> > >I don't get it. Specifically due to matrix being modular, it should
> > >be possible to have additional demos in separate repositories - the
> > >way it's done here. And the corresponding recipes that pull from
> > >those additional repositories and package them up for matrix to use.
> > >I don't see this as being a fork, I see this as being an out-of-tree
> > >extenstion. Getting those extensions in to the main tree has some
> > >benefits, but reduces control...
[Franklin] We generally have this only occur when there is a non SDK team that owns their application and Matrix demo that is pulled into an SDK. A good example would be the Wilink demos. But there is no reasons why any team that owns their own sdk can't sure the Matrix repo.
> >
> > It also tends to reduce re-use.  I think it is first worth looking
> > into adding to the shared repo first.
> 
> Then we should outline the process to get 3rd-party changes to the matrix
> repositories (patches, pull requests). Is Franklin the current owner?
[Franklin] No issue in adding things to matrix or the matrix apps repo. 
In terms of processes we can follow the same approach we have taken for all of our scripts which is just sending patches to this mailing list. 

> 
> 
> > Also, even if you want an external repo for additions it seems strange that
> > the .inc files needed to be copied as well.  For example:
> 
> Indeed, I was not commenting on the recipes themselves, but on the method
> to
> add additional functionality to the matrix. Recipes do require more work.
> 
> 
> > Matrix-gui-apps-ks2-git.inc is almost a direct copy of
> > matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a
> mimimum
> > why not inherit matrix-gui-apps-git.inc and change those few settings for
> > your demos, images, submenu, etc recipes to inherit.
> >
> > For matrix-gui-apps-ks2.inc there are also a few changes that could either
> > be common or look wrong and don't match the comments above the install
> > function.  I'll send some additional feedback on that section.  The main
> > changes I see are:
> > 	1. changing the .sh scripts to be installed into the
> > 	${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is
> > 	contrary to all other matrix apps.
> > 	2. there seem to be .cgi files added and they are being installed into
> > 	the root of the ${MATRIX_APP_DIR} even though the directory
> > 	${MATRIX_APP_DIR}/$appdir/cgi-bin was created.
> >
> > These changed could be rolled into the generic .inc instead of forking.
> >
> > >
> > >
> > >> As a side note, if you were going to do tis you should break
> > >this up into
> > >> separate patches for the different recipes, not one mega patch
> > >adding
> > >> everything.  Also, you have recipes like matrix-gui-demos-demos
> > >which
> > >> doesn't seem right.
> > >>
> > >> >
> > >> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > >> >---
> > >> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
> > >+++++
> > >> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> > >> >++++++++
> > >> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> > >> >++++++++++++++++++++
> > >> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
> > >+++++++
> > >> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> > >> >+++++++++++++++
> > >> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
> > >+++++++
> > >> > 6 files changed, 157 insertions(+)
> > >> > create mode 100755 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-apps-ks2-git.inc
> > >> > create mode 100644 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-apps-ks2-images_2.0.bb
> > >> > create mode 100755 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-apps-ks2.inc
> > >> > create mode 100755 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-demos-demos_2.0.bb
> > >> > create mode 100755 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-submenus-ks2_2.0.bb
> > >> > create mode 100755 meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-utility-demos_2.0.bb
> > >> >
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >apps-ks2-git.inc b/meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-apps-ks2-git.inc
> > >> >new file mode 100755
> > >> >index 0000000..4a7b21d
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > >ks2-
> > >> >git.inc
> > >> >@@ -0,0 +1,13 @@
> > >> >+LICENSE = "CC-BY-SA"
> > >> >+#By default all Matrix v2 applications use the same license.
> > >> >+LICENSE_CHECKSUM =
> > >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > >> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> > >> >+
> > >> >+BRANCH ?= "master"
> > >> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> > >> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> > >> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> > >> >apps.git;protocol=git;branch=${BRANCH}"
> > >> >+INC_PR = "r0"
> > >> >+
> > >> >+# Pull in the base package for installing matrix applications
> > >> >+require matrix-gui-apps-ks2.inc
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> > >> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> > >> >new file mode 100644
> > >> >index 0000000..cf69d81
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > >ks2-
> > >> >images_2.0.bb
> > >> >@@ -0,0 +1,19 @@
> > >> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
> > >for
> > >> >Keystone II"
> > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > >> >demos/matrix-apps.git"
> > >> >+LICENSE = "CC-BY-SA"
> > >> >+
> > >> >+require matrix-gui-apps-ks2-git.inc
> > >> >+require matrix-gui-paths.inc
> > >> >+
> > >> >+PR = "${INC_PR}.0"
> > >> >+
> > >> >+inherit allarch
> > >> >+
> > >> >+S = "${WORKDIR}/git/images"
> > >> >+
> > >> >+do_install(){
> > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > >> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> > >> >+}
> > >> >+
> > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> > >gui-
> > >> >apps-ks2.inc
> > >> >new file mode 100755
> > >> >index 0000000..fb3c866
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > >> >ks2.inc
> > >> >@@ -0,0 +1,51 @@
> > >> >+require matrix-gui-paths.inc
> > >> >+
> > >> >+MATRIX_FILES_DIR ?= "${S}"
> > >> >+
> > >> >+# Append a generic function to the do_install step that will
> > >look
> > >> >in the
> > >> >+# MATRIX_FILES_DIR for .desktop files and then:
> > >> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> > >> >+#   2.  Install all script files into ${bindir}
> > >> >+do_install_append(){
> > >> >+    cd ${MATRIX_FILES_DIR}
> > >> >+    applist=`find . -name "*.desktop"`
> > >> >+
> > >> >+    for app in $applist
> > >> >+    do
> > >> >+        appdir=`dirname $app`
> > >> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> > >> >+
> > >> >+        # Get the matrix metadata
> > >> >+        matrix_desktop=`find $appdir -type f -name
> > >"*.desktop"`
> > >> >+        for m in $matrix_desktop
> > >> >+        do
> > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > >> >+        done
> > >> >+
> > >> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> > >> >+        for m in $matrix_html
> > >> >+        do
> > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > >> >+        done
> > >> >+
> > >> >+        # Get the script files as executables
> > >> >+        scripts=`find $appdir -type f -name "*.sh"`
> > >> >+        for s in $scripts
> > >> >+        do
> > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > >> >+        done
> > >> >+
> > >> >+        # Get the cgi files as executables
> > >> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> > >> >+
> > >> >+        for c in $matrix_cgi
> > >> >+        do
> > >> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> > >> >${D}${MATRIX_APP_DIR}/$c
> > >> >+        done
> > >> >+    done
> > >> >+
> > >> >+    # Go back to the directory we started from in case there
> > >are
> > >> >+    # other appends.
> > >> >+    cd -
> > >> >+}
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
> > >core/matrix/matrix-
> > >> >gui-demos-demos_2.0.bb
> > >> >new file mode 100755
> > >> >index 0000000..eb89cca
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> > >> >demos_2.0.bb
> > >> >@@ -0,0 +1,18 @@
> > >> >+DESCRIPTION = "Demo description for Matrix v2"
> > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > >> >demos/matrix-apps.git"
> > >> >+LICENSE = "CC-BY-SA"
> > >> >+
> > >> >+require matrix-gui-apps-ks2-git.inc
> > >> >+
> > >> >+PR = "${INC_PR}.0"
> > >> >+
> > >> >+inherit allarch
> > >> >+
> > >> >+S = "${WORKDIR}/git/demo_apps"
> > >> >+
> > >> >+PACKAGES += "${PN}-demos"
> > >> >+
> > >> >+# Make sure crypto submenu and app images has been installed.
> > >> >Also make sure openssl is available
> > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > >> >submenus-ks2-demos"
> > >> >+
> > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> > >> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> > >> >new file mode 100755
> > >> >index 0000000..2936ad1
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >submenus-
> > >> >ks2_2.0.bb
> > >> >@@ -0,0 +1,38 @@
> > >> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> > >> >II"
> > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > >> >demos/matrix-apps.git"
> > >> >+LICENSE = "CC-BY-SA"
> > >> >+
> > >> >+require matrix-gui-apps-ks2-git.inc
> > >> >+require matrix-gui-paths.inc
> > >> >+
> > >> >+# This package does not use a subdirectory as ${S} so we need
> > >to
> > >> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> > >> >git.inc file
> > >> >+LIC_FILES_CHKSUM =
> > >> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > >> >+
> > >> >+PR = "${INC_PR}.0"
> > >> >+
> > >> >+# These packages make submenus in matrix and are not
> > >architecture
> > >> >specific
> > >> >+inherit allarch
> > >> >+
> > >> >+S = "${WORKDIR}/git"
> > >> >+
> > >> >+# List of submenus to build packages for
> > >> >+SUBMENUS = "utilities_submenu demos_submenu"
> > >> >+
> > >> >+do_install(){
> > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > >> >+
> > >> >+    for x in ${SUBMENUS}
> > >> >+    do
> > >> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> > >> >+    done
> > >> >+}
> > >> >+
> > >> >+PACKAGES += "${PN}-utility ${PN}-demos"
> > >> >+
> > >> >+# Make sure app images has been installed
> > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> > >> >+
> > >> >+# Add the files for each submenu package
> > >> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> > >> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> > >> >core/matrix/matrix-gui-utility-demos_2.0.bb
> > >> >new file mode 100755
> > >> >index 0000000..8f769e9
> > >> >--- /dev/null
> > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> > >> >demos_2.0.bb
> > >> >@@ -0,0 +1,18 @@
> > >> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > >> >demos/matrix-apps.git"
> > >> >+LICENSE = "CC-BY-SA"
> > >> >+
> > >> >+require matrix-gui-apps-ks2-git.inc
> > >> >+
> > >> >+PR = "${INC_PR}.0"
> > >> >+
> > >> >+inherit allarch
> > >> >+
> > >> >+S = "${WORKDIR}/git/utility_apps"
> > >> >+
> > >> >+PACKAGES += "${PN}-utility"
> > >> >+
> > >> >+# Make sure crypto submenu and app images has been installed.
> > >> >Also make sure openssl is available
> > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > >> >submenus-ks2-utility"
> > >> >+
> > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > >> >--
> > >> >1.7.9.5
> > >> >
> > >> >_______________________________________________
> > >> >meta-arago mailing list
> > >> >meta-arago@arago-project.org
> > >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > >> _______________________________________________
> > >> meta-arago mailing list
> > >> meta-arago@arago-project.org
> > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2014-03-17 20:59         ` Cooper Jr., Franklin
@ 2015-02-20  2:40           ` Denys Dmytriyenko
  2015-02-20  2:42             ` Denys Dmytriyenko
  0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2015-02-20  2:40 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago

Franklin, Sam,

Have you guys agreed on anything re: matrix-apps?


On Mon, Mar 17, 2014 at 04:59:36PM -0400, Cooper Jr., Franklin wrote:
> 
> 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> > Sent: Monday, March 17, 2014 2:57 PM
> > To: Maupin, Chase
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix demos for
> > keystone 2 platforms
> > 
> > On Mon, Mar 17, 2014 at 03:46:17PM -0400, Maupin, Chase wrote:
> > > >-----Original Message-----
> > > >From: Dmytriyenko, Denys
> > > >Sent: Monday, March 17, 2014 1:49 PM
> > > >To: Maupin, Chase
> > > >Cc: Nelson, Sam; meta-arago@arago-project.org
> > > >Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > >demos for keystone 2 platforms
> > > >
> > > >On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> > > >> >-----Original Message-----
> > > >> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > >> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> > > >> >Sent: Monday, March 03, 2014 2:02 PM
> > > >> >To: meta-arago@arago-project.org
> > > >> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > >> >demos for keystone 2 platforms
> > > >> >
> > > >> >- Recipes bring images and menus needed for keystone2 demos
> > > >>
> > > >> NAK
> > > >>
> > > >> These should be pushed to the shared matrix repositories.  There
> > > >is no
> > > >> reason to duplicate all these recipes and include files.  The
> > > >purpose of
> > > >> matrix being modular is to enable you to share and not fork.  If
> > > >you add
> > > >> these to the shared repo then you can create simple recipes to
> > > >package you
> > > >> individual demos and install those for ks2 devices.  Please
> > > >refer to how
> > > >> this is done for other devices.
> > > >
> > > >I don't get it. Specifically due to matrix being modular, it should
> > > >be possible to have additional demos in separate repositories - the
> > > >way it's done here. And the corresponding recipes that pull from
> > > >those additional repositories and package them up for matrix to use.
> > > >I don't see this as being a fork, I see this as being an out-of-tree
> > > >extenstion. Getting those extensions in to the main tree has some
> > > >benefits, but reduces control...

> [Franklin] We generally have this only occur when there is a non SDK team 
> that owns their application and Matrix demo that is pulled into an SDK. A 
> good example would be the Wilink demos. But there is no reasons why any team 
> that owns their own sdk can't sure the Matrix repo.
> 
> 
> > > It also tends to reduce re-use.  I think it is first worth looking
> > > into adding to the shared repo first.
> > 
> > Then we should outline the process to get 3rd-party changes to the matrix
> > repositories (patches, pull requests). Is Franklin the current owner?

> [Franklin] No issue in adding things to matrix or the matrix apps repo. 
> In terms of processes we can follow the same approach we have taken for all 
> of our scripts which is just sending patches to this mailing list.
> 
> > 
> > 
> > > Also, even if you want an external repo for additions it seems strange that
> > > the .inc files needed to be copied as well.  For example:
> > 
> > Indeed, I was not commenting on the recipes themselves, but on the method
> > to
> > add additional functionality to the matrix. Recipes do require more work.
> > 
> > 
> > > Matrix-gui-apps-ks2-git.inc is almost a direct copy of
> > > matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a
> > mimimum
> > > why not inherit matrix-gui-apps-git.inc and change those few settings for
> > > your demos, images, submenu, etc recipes to inherit.
> > >
> > > For matrix-gui-apps-ks2.inc there are also a few changes that could either
> > > be common or look wrong and don't match the comments above the install
> > > function.  I'll send some additional feedback on that section.  The main
> > > changes I see are:
> > > 	1. changing the .sh scripts to be installed into the
> > > 	${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is
> > > 	contrary to all other matrix apps.
> > > 	2. there seem to be .cgi files added and they are being installed into
> > > 	the root of the ${MATRIX_APP_DIR} even though the directory
> > > 	${MATRIX_APP_DIR}/$appdir/cgi-bin was created.
> > >
> > > These changed could be rolled into the generic .inc instead of forking.
> > >
> > > >
> > > >
> > > >> As a side note, if you were going to do tis you should break
> > > >this up into
> > > >> separate patches for the different recipes, not one mega patch
> > > >adding
> > > >> everything.  Also, you have recipes like matrix-gui-demos-demos
> > > >which
> > > >> doesn't seem right.
> > > >>
> > > >> >
> > > >> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > >> >---
> > > >> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
> > > >+++++
> > > >> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> > > >> >++++++++
> > > >> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> > > >> >++++++++++++++++++++
> > > >> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
> > > >+++++++
> > > >> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> > > >> >+++++++++++++++
> > > >> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
> > > >+++++++
> > > >> > 6 files changed, 157 insertions(+)
> > > >> > create mode 100755 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-apps-ks2-git.inc
> > > >> > create mode 100644 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-apps-ks2-images_2.0.bb
> > > >> > create mode 100755 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-apps-ks2.inc
> > > >> > create mode 100755 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-demos-demos_2.0.bb
> > > >> > create mode 100755 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-submenus-ks2_2.0.bb
> > > >> > create mode 100755 meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-utility-demos_2.0.bb
> > > >> >
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >apps-ks2-git.inc b/meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-apps-ks2-git.inc
> > > >> >new file mode 100755
> > > >> >index 0000000..4a7b21d
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > >ks2-
> > > >> >git.inc
> > > >> >@@ -0,0 +1,13 @@
> > > >> >+LICENSE = "CC-BY-SA"
> > > >> >+#By default all Matrix v2 applications use the same license.
> > > >> >+LICENSE_CHECKSUM =
> > > >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > >> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> > > >> >+
> > > >> >+BRANCH ?= "master"
> > > >> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> > > >> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> > > >> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> > > >> >apps.git;protocol=git;branch=${BRANCH}"
> > > >> >+INC_PR = "r0"
> > > >> >+
> > > >> >+# Pull in the base package for installing matrix applications
> > > >> >+require matrix-gui-apps-ks2.inc
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> > > >> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> > > >> >new file mode 100644
> > > >> >index 0000000..cf69d81
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > >ks2-
> > > >> >images_2.0.bb
> > > >> >@@ -0,0 +1,19 @@
> > > >> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
> > > >for
> > > >> >Keystone II"
> > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > >> >demos/matrix-apps.git"
> > > >> >+LICENSE = "CC-BY-SA"
> > > >> >+
> > > >> >+require matrix-gui-apps-ks2-git.inc
> > > >> >+require matrix-gui-paths.inc
> > > >> >+
> > > >> >+PR = "${INC_PR}.0"
> > > >> >+
> > > >> >+inherit allarch
> > > >> >+
> > > >> >+S = "${WORKDIR}/git/images"
> > > >> >+
> > > >> >+do_install(){
> > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > >> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> > > >> >+}
> > > >> >+
> > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> > > >gui-
> > > >> >apps-ks2.inc
> > > >> >new file mode 100755
> > > >> >index 0000000..fb3c866
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > >> >ks2.inc
> > > >> >@@ -0,0 +1,51 @@
> > > >> >+require matrix-gui-paths.inc
> > > >> >+
> > > >> >+MATRIX_FILES_DIR ?= "${S}"
> > > >> >+
> > > >> >+# Append a generic function to the do_install step that will
> > > >look
> > > >> >in the
> > > >> >+# MATRIX_FILES_DIR for .desktop files and then:
> > > >> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> > > >> >+#   2.  Install all script files into ${bindir}
> > > >> >+do_install_append(){
> > > >> >+    cd ${MATRIX_FILES_DIR}
> > > >> >+    applist=`find . -name "*.desktop"`
> > > >> >+
> > > >> >+    for app in $applist
> > > >> >+    do
> > > >> >+        appdir=`dirname $app`
> > > >> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> > > >> >+
> > > >> >+        # Get the matrix metadata
> > > >> >+        matrix_desktop=`find $appdir -type f -name
> > > >"*.desktop"`
> > > >> >+        for m in $matrix_desktop
> > > >> >+        do
> > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > >> >+        done
> > > >> >+
> > > >> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> > > >> >+        for m in $matrix_html
> > > >> >+        do
> > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > >> >+        done
> > > >> >+
> > > >> >+        # Get the script files as executables
> > > >> >+        scripts=`find $appdir -type f -name "*.sh"`
> > > >> >+        for s in $scripts
> > > >> >+        do
> > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > >> >+        done
> > > >> >+
> > > >> >+        # Get the cgi files as executables
> > > >> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> > > >> >+
> > > >> >+        for c in $matrix_cgi
> > > >> >+        do
> > > >> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> > > >> >${D}${MATRIX_APP_DIR}/$c
> > > >> >+        done
> > > >> >+    done
> > > >> >+
> > > >> >+    # Go back to the directory we started from in case there
> > > >are
> > > >> >+    # other appends.
> > > >> >+    cd -
> > > >> >+}
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
> > > >core/matrix/matrix-
> > > >> >gui-demos-demos_2.0.bb
> > > >> >new file mode 100755
> > > >> >index 0000000..eb89cca
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> > > >> >demos_2.0.bb
> > > >> >@@ -0,0 +1,18 @@
> > > >> >+DESCRIPTION = "Demo description for Matrix v2"
> > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > >> >demos/matrix-apps.git"
> > > >> >+LICENSE = "CC-BY-SA"
> > > >> >+
> > > >> >+require matrix-gui-apps-ks2-git.inc
> > > >> >+
> > > >> >+PR = "${INC_PR}.0"
> > > >> >+
> > > >> >+inherit allarch
> > > >> >+
> > > >> >+S = "${WORKDIR}/git/demo_apps"
> > > >> >+
> > > >> >+PACKAGES += "${PN}-demos"
> > > >> >+
> > > >> >+# Make sure crypto submenu and app images has been installed.
> > > >> >Also make sure openssl is available
> > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > >> >submenus-ks2-demos"
> > > >> >+
> > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> > > >> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> > > >> >new file mode 100755
> > > >> >index 0000000..2936ad1
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >submenus-
> > > >> >ks2_2.0.bb
> > > >> >@@ -0,0 +1,38 @@
> > > >> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> > > >> >II"
> > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > >> >demos/matrix-apps.git"
> > > >> >+LICENSE = "CC-BY-SA"
> > > >> >+
> > > >> >+require matrix-gui-apps-ks2-git.inc
> > > >> >+require matrix-gui-paths.inc
> > > >> >+
> > > >> >+# This package does not use a subdirectory as ${S} so we need
> > > >to
> > > >> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> > > >> >git.inc file
> > > >> >+LIC_FILES_CHKSUM =
> > > >> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > >> >+
> > > >> >+PR = "${INC_PR}.0"
> > > >> >+
> > > >> >+# These packages make submenus in matrix and are not
> > > >architecture
> > > >> >specific
> > > >> >+inherit allarch
> > > >> >+
> > > >> >+S = "${WORKDIR}/git"
> > > >> >+
> > > >> >+# List of submenus to build packages for
> > > >> >+SUBMENUS = "utilities_submenu demos_submenu"
> > > >> >+
> > > >> >+do_install(){
> > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > >> >+
> > > >> >+    for x in ${SUBMENUS}
> > > >> >+    do
> > > >> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> > > >> >+    done
> > > >> >+}
> > > >> >+
> > > >> >+PACKAGES += "${PN}-utility ${PN}-demos"
> > > >> >+
> > > >> >+# Make sure app images has been installed
> > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> > > >> >+
> > > >> >+# Add the files for each submenu package
> > > >> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> > > >> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > >> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> > > >> >core/matrix/matrix-gui-utility-demos_2.0.bb
> > > >> >new file mode 100755
> > > >> >index 0000000..8f769e9
> > > >> >--- /dev/null
> > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> > > >> >demos_2.0.bb
> > > >> >@@ -0,0 +1,18 @@
> > > >> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > >> >demos/matrix-apps.git"
> > > >> >+LICENSE = "CC-BY-SA"
> > > >> >+
> > > >> >+require matrix-gui-apps-ks2-git.inc
> > > >> >+
> > > >> >+PR = "${INC_PR}.0"
> > > >> >+
> > > >> >+inherit allarch
> > > >> >+
> > > >> >+S = "${WORKDIR}/git/utility_apps"
> > > >> >+
> > > >> >+PACKAGES += "${PN}-utility"
> > > >> >+
> > > >> >+# Make sure crypto submenu and app images has been installed.
> > > >> >Also make sure openssl is available
> > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > >> >submenus-ks2-utility"
> > > >> >+
> > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > >> >--
> > > >> >1.7.9.5
> > > >> >
> > > >> >_______________________________________________
> > > >> >meta-arago mailing list
> > > >> >meta-arago@arago-project.org
> > > >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > >> _______________________________________________
> > > >> meta-arago mailing list
> > > >> meta-arago@arago-project.org
> > > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2015-02-20  2:40           ` Denys Dmytriyenko
@ 2015-02-20  2:42             ` Denys Dmytriyenko
  2015-02-20  2:45               ` Nelson, Sam
  0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2015-02-20  2:42 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago

Oops, sorry, pulled an older version of the thread... I believe there was a 
more recent discussion on the same matter though. Still, I was wondering if 
you reached any agreement about hosting and supplying keystone specific demos


On Thu, Feb 19, 2015 at 09:40:50PM -0500, Denys Dmytriyenko wrote:
> Franklin, Sam,
> 
> Have you guys agreed on anything re: matrix-apps?
> 
> 
> On Mon, Mar 17, 2014 at 04:59:36PM -0400, Cooper Jr., Franklin wrote:
> > 
> > 
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> > > Sent: Monday, March 17, 2014 2:57 PM
> > > To: Maupin, Chase
> > > Cc: meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix demos for
> > > keystone 2 platforms
> > > 
> > > On Mon, Mar 17, 2014 at 03:46:17PM -0400, Maupin, Chase wrote:
> > > > >-----Original Message-----
> > > > >From: Dmytriyenko, Denys
> > > > >Sent: Monday, March 17, 2014 1:49 PM
> > > > >To: Maupin, Chase
> > > > >Cc: Nelson, Sam; meta-arago@arago-project.org
> > > > >Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > > >demos for keystone 2 platforms
> > > > >
> > > > >On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> > > > >> >-----Original Message-----
> > > > >> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > >> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> > > > >> >Sent: Monday, March 03, 2014 2:02 PM
> > > > >> >To: meta-arago@arago-project.org
> > > > >> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > > >> >demos for keystone 2 platforms
> > > > >> >
> > > > >> >- Recipes bring images and menus needed for keystone2 demos
> > > > >>
> > > > >> NAK
> > > > >>
> > > > >> These should be pushed to the shared matrix repositories.  There
> > > > >is no
> > > > >> reason to duplicate all these recipes and include files.  The
> > > > >purpose of
> > > > >> matrix being modular is to enable you to share and not fork.  If
> > > > >you add
> > > > >> these to the shared repo then you can create simple recipes to
> > > > >package you
> > > > >> individual demos and install those for ks2 devices.  Please
> > > > >refer to how
> > > > >> this is done for other devices.
> > > > >
> > > > >I don't get it. Specifically due to matrix being modular, it should
> > > > >be possible to have additional demos in separate repositories - the
> > > > >way it's done here. And the corresponding recipes that pull from
> > > > >those additional repositories and package them up for matrix to use.
> > > > >I don't see this as being a fork, I see this as being an out-of-tree
> > > > >extenstion. Getting those extensions in to the main tree has some
> > > > >benefits, but reduces control...
> 
> > [Franklin] We generally have this only occur when there is a non SDK team 
> > that owns their application and Matrix demo that is pulled into an SDK. A 
> > good example would be the Wilink demos. But there is no reasons why any team 
> > that owns their own sdk can't sure the Matrix repo.
> > 
> > 
> > > > It also tends to reduce re-use.  I think it is first worth looking
> > > > into adding to the shared repo first.
> > > 
> > > Then we should outline the process to get 3rd-party changes to the matrix
> > > repositories (patches, pull requests). Is Franklin the current owner?
> 
> > [Franklin] No issue in adding things to matrix or the matrix apps repo. 
> > In terms of processes we can follow the same approach we have taken for all 
> > of our scripts which is just sending patches to this mailing list.
> > 
> > > 
> > > 
> > > > Also, even if you want an external repo for additions it seems strange that
> > > > the .inc files needed to be copied as well.  For example:
> > > 
> > > Indeed, I was not commenting on the recipes themselves, but on the method
> > > to
> > > add additional functionality to the matrix. Recipes do require more work.
> > > 
> > > 
> > > > Matrix-gui-apps-ks2-git.inc is almost a direct copy of
> > > > matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a
> > > mimimum
> > > > why not inherit matrix-gui-apps-git.inc and change those few settings for
> > > > your demos, images, submenu, etc recipes to inherit.
> > > >
> > > > For matrix-gui-apps-ks2.inc there are also a few changes that could either
> > > > be common or look wrong and don't match the comments above the install
> > > > function.  I'll send some additional feedback on that section.  The main
> > > > changes I see are:
> > > > 	1. changing the .sh scripts to be installed into the
> > > > 	${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is
> > > > 	contrary to all other matrix apps.
> > > > 	2. there seem to be .cgi files added and they are being installed into
> > > > 	the root of the ${MATRIX_APP_DIR} even though the directory
> > > > 	${MATRIX_APP_DIR}/$appdir/cgi-bin was created.
> > > >
> > > > These changed could be rolled into the generic .inc instead of forking.
> > > >
> > > > >
> > > > >
> > > > >> As a side note, if you were going to do tis you should break
> > > > >this up into
> > > > >> separate patches for the different recipes, not one mega patch
> > > > >adding
> > > > >> everything.  Also, you have recipes like matrix-gui-demos-demos
> > > > >which
> > > > >> doesn't seem right.
> > > > >>
> > > > >> >
> > > > >> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > >> >---
> > > > >> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
> > > > >+++++
> > > > >> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> > > > >> >++++++++
> > > > >> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> > > > >> >++++++++++++++++++++
> > > > >> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
> > > > >+++++++
> > > > >> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> > > > >> >+++++++++++++++
> > > > >> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
> > > > >+++++++
> > > > >> > 6 files changed, 157 insertions(+)
> > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-apps-ks2-git.inc
> > > > >> > create mode 100644 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-apps-ks2-images_2.0.bb
> > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-apps-ks2.inc
> > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-demos-demos_2.0.bb
> > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-submenus-ks2_2.0.bb
> > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-utility-demos_2.0.bb
> > > > >> >
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >apps-ks2-git.inc b/meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-apps-ks2-git.inc
> > > > >> >new file mode 100755
> > > > >> >index 0000000..4a7b21d
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > >ks2-
> > > > >> >git.inc
> > > > >> >@@ -0,0 +1,13 @@
> > > > >> >+LICENSE = "CC-BY-SA"
> > > > >> >+#By default all Matrix v2 applications use the same license.
> > > > >> >+LICENSE_CHECKSUM =
> > > > >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > > >> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> > > > >> >+
> > > > >> >+BRANCH ?= "master"
> > > > >> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> > > > >> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> > > > >> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> > > > >> >apps.git;protocol=git;branch=${BRANCH}"
> > > > >> >+INC_PR = "r0"
> > > > >> >+
> > > > >> >+# Pull in the base package for installing matrix applications
> > > > >> >+require matrix-gui-apps-ks2.inc
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> > > > >> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> > > > >> >new file mode 100644
> > > > >> >index 0000000..cf69d81
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > >ks2-
> > > > >> >images_2.0.bb
> > > > >> >@@ -0,0 +1,19 @@
> > > > >> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
> > > > >for
> > > > >> >Keystone II"
> > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > >> >demos/matrix-apps.git"
> > > > >> >+LICENSE = "CC-BY-SA"
> > > > >> >+
> > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > >> >+require matrix-gui-paths.inc
> > > > >> >+
> > > > >> >+PR = "${INC_PR}.0"
> > > > >> >+
> > > > >> >+inherit allarch
> > > > >> >+
> > > > >> >+S = "${WORKDIR}/git/images"
> > > > >> >+
> > > > >> >+do_install(){
> > > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > > >> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> > > > >> >+}
> > > > >> >+
> > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> > > > >gui-
> > > > >> >apps-ks2.inc
> > > > >> >new file mode 100755
> > > > >> >index 0000000..fb3c866
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > >> >ks2.inc
> > > > >> >@@ -0,0 +1,51 @@
> > > > >> >+require matrix-gui-paths.inc
> > > > >> >+
> > > > >> >+MATRIX_FILES_DIR ?= "${S}"
> > > > >> >+
> > > > >> >+# Append a generic function to the do_install step that will
> > > > >look
> > > > >> >in the
> > > > >> >+# MATRIX_FILES_DIR for .desktop files and then:
> > > > >> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> > > > >> >+#   2.  Install all script files into ${bindir}
> > > > >> >+do_install_append(){
> > > > >> >+    cd ${MATRIX_FILES_DIR}
> > > > >> >+    applist=`find . -name "*.desktop"`
> > > > >> >+
> > > > >> >+    for app in $applist
> > > > >> >+    do
> > > > >> >+        appdir=`dirname $app`
> > > > >> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> > > > >> >+
> > > > >> >+        # Get the matrix metadata
> > > > >> >+        matrix_desktop=`find $appdir -type f -name
> > > > >"*.desktop"`
> > > > >> >+        for m in $matrix_desktop
> > > > >> >+        do
> > > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > >> >+        done
> > > > >> >+
> > > > >> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> > > > >> >+        for m in $matrix_html
> > > > >> >+        do
> > > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > >> >+        done
> > > > >> >+
> > > > >> >+        # Get the script files as executables
> > > > >> >+        scripts=`find $appdir -type f -name "*.sh"`
> > > > >> >+        for s in $scripts
> > > > >> >+        do
> > > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > >> >+        done
> > > > >> >+
> > > > >> >+        # Get the cgi files as executables
> > > > >> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> > > > >> >+
> > > > >> >+        for c in $matrix_cgi
> > > > >> >+        do
> > > > >> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> > > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> > > > >> >${D}${MATRIX_APP_DIR}/$c
> > > > >> >+        done
> > > > >> >+    done
> > > > >> >+
> > > > >> >+    # Go back to the directory we started from in case there
> > > > >are
> > > > >> >+    # other appends.
> > > > >> >+    cd -
> > > > >> >+}
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
> > > > >core/matrix/matrix-
> > > > >> >gui-demos-demos_2.0.bb
> > > > >> >new file mode 100755
> > > > >> >index 0000000..eb89cca
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> > > > >> >demos_2.0.bb
> > > > >> >@@ -0,0 +1,18 @@
> > > > >> >+DESCRIPTION = "Demo description for Matrix v2"
> > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > >> >demos/matrix-apps.git"
> > > > >> >+LICENSE = "CC-BY-SA"
> > > > >> >+
> > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > >> >+
> > > > >> >+PR = "${INC_PR}.0"
> > > > >> >+
> > > > >> >+inherit allarch
> > > > >> >+
> > > > >> >+S = "${WORKDIR}/git/demo_apps"
> > > > >> >+
> > > > >> >+PACKAGES += "${PN}-demos"
> > > > >> >+
> > > > >> >+# Make sure crypto submenu and app images has been installed.
> > > > >> >Also make sure openssl is available
> > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > > >> >submenus-ks2-demos"
> > > > >> >+
> > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> > > > >> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> > > > >> >new file mode 100755
> > > > >> >index 0000000..2936ad1
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >submenus-
> > > > >> >ks2_2.0.bb
> > > > >> >@@ -0,0 +1,38 @@
> > > > >> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> > > > >> >II"
> > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > >> >demos/matrix-apps.git"
> > > > >> >+LICENSE = "CC-BY-SA"
> > > > >> >+
> > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > >> >+require matrix-gui-paths.inc
> > > > >> >+
> > > > >> >+# This package does not use a subdirectory as ${S} so we need
> > > > >to
> > > > >> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> > > > >> >git.inc file
> > > > >> >+LIC_FILES_CHKSUM =
> > > > >> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > > >> >+
> > > > >> >+PR = "${INC_PR}.0"
> > > > >> >+
> > > > >> >+# These packages make submenus in matrix and are not
> > > > >architecture
> > > > >> >specific
> > > > >> >+inherit allarch
> > > > >> >+
> > > > >> >+S = "${WORKDIR}/git"
> > > > >> >+
> > > > >> >+# List of submenus to build packages for
> > > > >> >+SUBMENUS = "utilities_submenu demos_submenu"
> > > > >> >+
> > > > >> >+do_install(){
> > > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > > >> >+
> > > > >> >+    for x in ${SUBMENUS}
> > > > >> >+    do
> > > > >> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> > > > >> >+    done
> > > > >> >+}
> > > > >> >+
> > > > >> >+PACKAGES += "${PN}-utility ${PN}-demos"
> > > > >> >+
> > > > >> >+# Make sure app images has been installed
> > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> > > > >> >+
> > > > >> >+# Add the files for each submenu package
> > > > >> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> > > > >> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > >> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> > > > >> >core/matrix/matrix-gui-utility-demos_2.0.bb
> > > > >> >new file mode 100755
> > > > >> >index 0000000..8f769e9
> > > > >> >--- /dev/null
> > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> > > > >> >demos_2.0.bb
> > > > >> >@@ -0,0 +1,18 @@
> > > > >> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > >> >demos/matrix-apps.git"
> > > > >> >+LICENSE = "CC-BY-SA"
> > > > >> >+
> > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > >> >+
> > > > >> >+PR = "${INC_PR}.0"
> > > > >> >+
> > > > >> >+inherit allarch
> > > > >> >+
> > > > >> >+S = "${WORKDIR}/git/utility_apps"
> > > > >> >+
> > > > >> >+PACKAGES += "${PN}-utility"
> > > > >> >+
> > > > >> >+# Make sure crypto submenu and app images has been installed.
> > > > >> >Also make sure openssl is available
> > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > > >> >submenus-ks2-utility"
> > > > >> >+
> > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > >> >--
> > > > >> >1.7.9.5
> > > > >> >
> > > > >> >_______________________________________________
> > > > >> >meta-arago mailing list
> > > > >> >meta-arago@arago-project.org
> > > > >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > >> _______________________________________________
> > > > >> meta-arago mailing list
> > > > >> meta-arago@arago-project.org
> > > > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms
  2015-02-20  2:42             ` Denys Dmytriyenko
@ 2015-02-20  2:45               ` Nelson, Sam
  0 siblings, 0 replies; 10+ messages in thread
From: Nelson, Sam @ 2015-02-20  2:45 UTC (permalink / raw)
  To: Dmytriyenko, Denys, Cooper Jr., Franklin; +Cc: meta-arago

Yes.  We are planning to send patches to the shared matrix repositories.
These old patches can be ignored.
With regards,
Sam

> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> Sent: Thursday, February 19, 2015 9:43 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix demos for
> keystone 2 platforms
> 
> Oops, sorry, pulled an older version of the thread... I believe there was a
> more recent discussion on the same matter though. Still, I was wondering if
> you reached any agreement about hosting and supplying keystone specific
> demos
> 
> 
> On Thu, Feb 19, 2015 at 09:40:50PM -0500, Denys Dmytriyenko wrote:
> > Franklin, Sam,
> >
> > Have you guys agreed on anything re: matrix-apps?
> >
> >
> > On Mon, Mar 17, 2014 at 04:59:36PM -0400, Cooper Jr., Franklin wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> > > > Sent: Monday, March 17, 2014 2:57 PM
> > > > To: Maupin, Chase
> > > > Cc: meta-arago@arago-project.org
> > > > Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> demos for
> > > > keystone 2 platforms
> > > >
> > > > On Mon, Mar 17, 2014 at 03:46:17PM -0400, Maupin, Chase wrote:
> > > > > >-----Original Message-----
> > > > > >From: Dmytriyenko, Denys
> > > > > >Sent: Monday, March 17, 2014 1:49 PM
> > > > > >To: Maupin, Chase
> > > > > >Cc: Nelson, Sam; meta-arago@arago-project.org
> > > > > >Subject: Re: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > > > >demos for keystone 2 platforms
> > > > > >
> > > > > >On Fri, Mar 14, 2014 at 12:43:17PM +0000, Maupin, Chase wrote:
> > > > > >> >-----Original Message-----
> > > > > >> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > > >> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> > > > > >> >Sent: Monday, March 03, 2014 2:02 PM
> > > > > >> >To: meta-arago@arago-project.org
> > > > > >> >Subject: [meta-arago] [PATCH] matrix-demos: Recipes for Matrix
> > > > > >> >demos for keystone 2 platforms
> > > > > >> >
> > > > > >> >- Recipes bring images and menus needed for keystone2 demos
> > > > > >>
> > > > > >> NAK
> > > > > >>
> > > > > >> These should be pushed to the shared matrix repositories.  There
> > > > > >is no
> > > > > >> reason to duplicate all these recipes and include files.  The
> > > > > >purpose of
> > > > > >> matrix being modular is to enable you to share and not fork.  If
> > > > > >you add
> > > > > >> these to the shared repo then you can create simple recipes to
> > > > > >package you
> > > > > >> individual demos and install those for ks2 devices.  Please
> > > > > >refer to how
> > > > > >> this is done for other devices.
> > > > > >
> > > > > >I don't get it. Specifically due to matrix being modular, it should
> > > > > >be possible to have additional demos in separate repositories - the
> > > > > >way it's done here. And the corresponding recipes that pull from
> > > > > >those additional repositories and package them up for matrix to use.
> > > > > >I don't see this as being a fork, I see this as being an out-of-tree
> > > > > >extenstion. Getting those extensions in to the main tree has some
> > > > > >benefits, but reduces control...
> >
> > > [Franklin] We generally have this only occur when there is a non SDK team
> > > that owns their application and Matrix demo that is pulled into an SDK. A
> > > good example would be the Wilink demos. But there is no reasons why any
> team
> > > that owns their own sdk can't sure the Matrix repo.
> > >
> > >
> > > > > It also tends to reduce re-use.  I think it is first worth looking
> > > > > into adding to the shared repo first.
> > > >
> > > > Then we should outline the process to get 3rd-party changes to the matrix
> > > > repositories (patches, pull requests). Is Franklin the current owner?
> >
> > > [Franklin] No issue in adding things to matrix or the matrix apps repo.
> > > In terms of processes we can follow the same approach we have taken for
> all
> > > of our scripts which is just sending patches to this mailing list.
> > >
> > > >
> > > >
> > > > > Also, even if you want an external repo for additions it seems strange
> that
> > > > > the .inc files needed to be copied as well.  For example:
> > > >
> > > > Indeed, I was not commenting on the recipes themselves, but on the
> method
> > > > to
> > > > add additional functionality to the matrix. Recipes do require more work.
> > > >
> > > >
> > > > > Matrix-gui-apps-ks2-git.inc is almost a direct copy of
> > > > > matrix-gui-apps-git.inc with the SRC_URI and SRCREV changed.  At a
> > > > mimimum
> > > > > why not inherit matrix-gui-apps-git.inc and change those few settings
> for
> > > > > your demos, images, submenu, etc recipes to inherit.
> > > > >
> > > > > For matrix-gui-apps-ks2.inc there are also a few changes that could
> either
> > > > > be common or look wrong and don't match the comments above the
> install
> > > > > function.  I'll send some additional feedback on that section.  The main
> > > > > changes I see are:
> > > > > 	1. changing the .sh scripts to be installed into the
> > > > > 	${MATRIX_APP_DIR}/$appdir/ directory instead of ${bindir}.  This is
> > > > > 	contrary to all other matrix apps.
> > > > > 	2. there seem to be .cgi files added and they are being installed into
> > > > > 	the root of the ${MATRIX_APP_DIR} even though the directory
> > > > > 	${MATRIX_APP_DIR}/$appdir/cgi-bin was created.
> > > > >
> > > > > These changed could be rolled into the generic .inc instead of forking.
> > > > >
> > > > > >
> > > > > >
> > > > > >> As a side note, if you were going to do tis you should break
> > > > > >this up into
> > > > > >> separate patches for the different recipes, not one mega patch
> > > > > >adding
> > > > > >> everything.  Also, you have recipes like matrix-gui-demos-demos
> > > > > >which
> > > > > >> doesn't seem right.
> > > > > >>
> > > > > >> >
> > > > > >> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > > >> >---
> > > > > >> > .../matrix/matrix-gui-apps-ks2-git.inc             |   13
> > > > > >+++++
> > > > > >> > .../matrix/matrix-gui-apps-ks2-images_2.0.bb       |   19
> > > > > >> >++++++++
> > > > > >> > .../recipes-core/matrix/matrix-gui-apps-ks2.inc    |   51
> > > > > >> >++++++++++++++++++++
> > > > > >> > .../matrix/matrix-gui-demos-demos_2.0.bb           |   18
> > > > > >+++++++
> > > > > >> > .../matrix/matrix-gui-submenus-ks2_2.0.bb          |   38
> > > > > >> >+++++++++++++++
> > > > > >> > .../matrix/matrix-gui-utility-demos_2.0.bb         |   18
> > > > > >+++++++
> > > > > >> > 6 files changed, 157 insertions(+)
> > > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-apps-ks2-git.inc
> > > > > >> > create mode 100644 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-apps-ks2-images_2.0.bb
> > > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-apps-ks2.inc
> > > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-demos-demos_2.0.bb
> > > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-submenus-ks2_2.0.bb
> > > > > >> > create mode 100755 meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-utility-demos_2.0.bb
> > > > > >> >
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >apps-ks2-git.inc b/meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-apps-ks2-git.inc
> > > > > >> >new file mode 100755
> > > > > >> >index 0000000..4a7b21d
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > > >ks2-
> > > > > >> >git.inc
> > > > > >> >@@ -0,0 +1,13 @@
> > > > > >> >+LICENSE = "CC-BY-SA"
> > > > > >> >+#By default all Matrix v2 applications use the same license.
> > > > > >> >+LICENSE_CHECKSUM =
> > > > > >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > > > >> >+LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> > > > > >> >+
> > > > > >> >+BRANCH ?= "master"
> > > > > >> >+# commit corresponds to DEV.MATRIX.APPS.01.00.00.02
> > > > > >> >+SRCREV = "7af0ec5ad5ca9464a981b1c9683ec66bb075331d"
> > > > > >> >+SRC_URI = "git://git.ti.com/keystone-demos/matrix-
> > > > > >> >apps.git;protocol=git;branch=${BRANCH}"
> > > > > >> >+INC_PR = "r0"
> > > > > >> >+
> > > > > >> >+# Pull in the base package for installing matrix applications
> > > > > >> >+require matrix-gui-apps-ks2.inc
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >apps-ks2-images_2.0.bb b/meta-arago-extras/recipes-
> > > > > >> >core/matrix/matrix-gui-apps-ks2-images_2.0.bb
> > > > > >> >new file mode 100644
> > > > > >> >index 0000000..cf69d81
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > > >ks2-
> > > > > >> >images_2.0.bb
> > > > > >> >@@ -0,0 +1,19 @@
> > > > > >> >+DESCRIPTION = "Images package for Matrix GUI v2 Applications
> > > > > >for
> > > > > >> >Keystone II"
> > > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > > >> >demos/matrix-apps.git"
> > > > > >> >+LICENSE = "CC-BY-SA"
> > > > > >> >+
> > > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > > >> >+require matrix-gui-paths.inc
> > > > > >> >+
> > > > > >> >+PR = "${INC_PR}.0"
> > > > > >> >+
> > > > > >> >+inherit allarch
> > > > > >> >+
> > > > > >> >+S = "${WORKDIR}/git/images"
> > > > > >> >+
> > > > > >> >+do_install(){
> > > > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > > > >> >+    cp -rf ${S}/ ${D}${MATRIX_APP_DIR}
> > > > > >> >+}
> > > > > >> >+
> > > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >apps-ks2.inc b/meta-arago-extras/recipes-core/matrix/matrix-
> > > > > >gui-
> > > > > >> >apps-ks2.inc
> > > > > >> >new file mode 100755
> > > > > >> >index 0000000..fb3c866
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > > > > >> >ks2.inc
> > > > > >> >@@ -0,0 +1,51 @@
> > > > > >> >+require matrix-gui-paths.inc
> > > > > >> >+
> > > > > >> >+MATRIX_FILES_DIR ?= "${S}"
> > > > > >> >+
> > > > > >> >+# Append a generic function to the do_install step that will
> > > > > >look
> > > > > >> >in the
> > > > > >> >+# MATRIX_FILES_DIR for .desktop files and then:
> > > > > >> >+#   1.  Install all non-script files into the MATRIX_APP_DIR
> > > > > >> >+#   2.  Install all script files into ${bindir}
> > > > > >> >+do_install_append(){
> > > > > >> >+    cd ${MATRIX_FILES_DIR}
> > > > > >> >+    applist=`find . -name "*.desktop"`
> > > > > >> >+
> > > > > >> >+    for app in $applist
> > > > > >> >+    do
> > > > > >> >+        appdir=`dirname $app`
> > > > > >> >+        install -d ${D}${MATRIX_APP_DIR}/$appdir
> > > > > >> >+
> > > > > >> >+        # Get the matrix metadata
> > > > > >> >+        matrix_desktop=`find $appdir -type f -name
> > > > > >"*.desktop"`
> > > > > >> >+        for m in $matrix_desktop
> > > > > >> >+        do
> > > > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > > >> >+        done
> > > > > >> >+
> > > > > >> >+        matrix_html=`find $appdir -type f -name  "*.html"`
> > > > > >> >+        for m in $matrix_html
> > > > > >> >+        do
> > > > > >> >+            install -m 0644 ${MATRIX_FILES_DIR}/$m
> > > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > > >> >+        done
> > > > > >> >+
> > > > > >> >+        # Get the script files as executables
> > > > > >> >+        scripts=`find $appdir -type f -name "*.sh"`
> > > > > >> >+        for s in $scripts
> > > > > >> >+        do
> > > > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$s
> > > > > >> >${D}${MATRIX_APP_DIR}/$appdir/
> > > > > >> >+        done
> > > > > >> >+
> > > > > >> >+        # Get the cgi files as executables
> > > > > >> >+        matrix_cgi=`find $appdir -type f -name "*.cgi"`
> > > > > >> >+
> > > > > >> >+        for c in $matrix_cgi
> > > > > >> >+        do
> > > > > >> >+            install -d ${D}${MATRIX_APP_DIR}/$appdir/cgi-bin
> > > > > >> >+            install -m 0755 ${MATRIX_FILES_DIR}/$c
> > > > > >> >${D}${MATRIX_APP_DIR}/$c
> > > > > >> >+        done
> > > > > >> >+    done
> > > > > >> >+
> > > > > >> >+    # Go back to the directory we started from in case there
> > > > > >are
> > > > > >> >+    # other appends.
> > > > > >> >+    cd -
> > > > > >> >+}
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >demos-demos_2.0.bb b/meta-arago-extras/recipes-
> > > > > >core/matrix/matrix-
> > > > > >> >gui-demos-demos_2.0.bb
> > > > > >> >new file mode 100755
> > > > > >> >index 0000000..eb89cca
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-demos-
> > > > > >> >demos_2.0.bb
> > > > > >> >@@ -0,0 +1,18 @@
> > > > > >> >+DESCRIPTION = "Demo description for Matrix v2"
> > > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > > >> >demos/matrix-apps.git"
> > > > > >> >+LICENSE = "CC-BY-SA"
> > > > > >> >+
> > > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > > >> >+
> > > > > >> >+PR = "${INC_PR}.0"
> > > > > >> >+
> > > > > >> >+inherit allarch
> > > > > >> >+
> > > > > >> >+S = "${WORKDIR}/git/demo_apps"
> > > > > >> >+
> > > > > >> >+PACKAGES += "${PN}-demos"
> > > > > >> >+
> > > > > >> >+# Make sure crypto submenu and app images has been installed.
> > > > > >> >Also make sure openssl is available
> > > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > > > >> >submenus-ks2-demos"
> > > > > >> >+
> > > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >submenus-ks2_2.0.bb b/meta-arago-extras/recipes-
> > > > > >> >core/matrix/matrix-gui-submenus-ks2_2.0.bb
> > > > > >> >new file mode 100755
> > > > > >> >index 0000000..2936ad1
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >submenus-
> > > > > >> >ks2_2.0.bb
> > > > > >> >@@ -0,0 +1,38 @@
> > > > > >> >+DESCRIPTION = "Submenu packages for Matrix GUI v2 for Keystone
> > > > > >> >II"
> > > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > > >> >demos/matrix-apps.git"
> > > > > >> >+LICENSE = "CC-BY-SA"
> > > > > >> >+
> > > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > > >> >+require matrix-gui-paths.inc
> > > > > >> >+
> > > > > >> >+# This package does not use a subdirectory as ${S} so we need
> > > > > >to
> > > > > >> >+# reset the LIC_FILES_CHKSUM setting from the matrix-gui-apps-
> > > > > >> >git.inc file
> > > > > >> >+LIC_FILES_CHKSUM =
> > > > > >> >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > > > >> >+
> > > > > >> >+PR = "${INC_PR}.0"
> > > > > >> >+
> > > > > >> >+# These packages make submenus in matrix and are not
> > > > > >architecture
> > > > > >> >specific
> > > > > >> >+inherit allarch
> > > > > >> >+
> > > > > >> >+S = "${WORKDIR}/git"
> > > > > >> >+
> > > > > >> >+# List of submenus to build packages for
> > > > > >> >+SUBMENUS = "utilities_submenu demos_submenu"
> > > > > >> >+
> > > > > >> >+do_install(){
> > > > > >> >+    install -d ${D}${MATRIX_APP_DIR}
> > > > > >> >+
> > > > > >> >+    for x in ${SUBMENUS}
> > > > > >> >+    do
> > > > > >> >+        cp -rf ${S}/$x ${D}${MATRIX_APP_DIR}/
> > > > > >> >+    done
> > > > > >> >+}
> > > > > >> >+
> > > > > >> >+PACKAGES += "${PN}-utility ${PN}-demos"
> > > > > >> >+
> > > > > >> >+# Make sure app images has been installed
> > > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images"
> > > > > >> >+
> > > > > >> >+# Add the files for each submenu package
> > > > > >> >+FILES_${PN}-utility = "${MATRIX_APP_DIR}/utilities_submenu/*"
> > > > > >> >+FILES_${PN}-demos = "${MATRIX_APP_DIR}/demos_submenu/*"
> > > > > >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > > > > >> >utility-demos_2.0.bb b/meta-arago-extras/recipes-
> > > > > >> >core/matrix/matrix-gui-utility-demos_2.0.bb
> > > > > >> >new file mode 100755
> > > > > >> >index 0000000..8f769e9
> > > > > >> >--- /dev/null
> > > > > >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-utility-
> > > > > >> >demos_2.0.bb
> > > > > >> >@@ -0,0 +1,18 @@
> > > > > >> >+DESCRIPTION = "Utility app demo descriptions for Matrix v2"
> > > > > >> >+HOMEPAGE = "http://git.ti.com/cgit/cgit.cgi/keystone-
> > > > > >> >demos/matrix-apps.git"
> > > > > >> >+LICENSE = "CC-BY-SA"
> > > > > >> >+
> > > > > >> >+require matrix-gui-apps-ks2-git.inc
> > > > > >> >+
> > > > > >> >+PR = "${INC_PR}.0"
> > > > > >> >+
> > > > > >> >+inherit allarch
> > > > > >> >+
> > > > > >> >+S = "${WORKDIR}/git/utility_apps"
> > > > > >> >+
> > > > > >> >+PACKAGES += "${PN}-utility"
> > > > > >> >+
> > > > > >> >+# Make sure crypto submenu and app images has been installed.
> > > > > >> >Also make sure openssl is available
> > > > > >> >+RDEPENDS_${PN} += "matrix-gui-apps-ks2-images matrix-gui-
> > > > > >> >submenus-ks2-utility"
> > > > > >> >+
> > > > > >> >+FILES_${PN} += "${MATRIX_BASE_DIR}/*"
> > > > > >> >--
> > > > > >> >1.7.9.5
> > > > > >> >
> > > > > >> >_______________________________________________
> > > > > >> >meta-arago mailing list
> > > > > >> >meta-arago@arago-project.org
> > > > > >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > > >> _______________________________________________
> > > > > >> meta-arago mailing list
> > > > > >> meta-arago@arago-project.org
> > > > > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2015-02-20  2:45 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-03 20:01 [PATCH] matrix-demos: Recipes for Matrix demos for keystone 2 platforms Sam Nelson
2014-03-14 12:43 ` Maupin, Chase
2014-03-17 18:48   ` Denys Dmytriyenko
2014-03-17 19:46     ` Maupin, Chase
2014-03-17 19:57       ` Denys Dmytriyenko
2014-03-17 20:59         ` Cooper Jr., Franklin
2015-02-20  2:40           ` Denys Dmytriyenko
2015-02-20  2:42             ` Denys Dmytriyenko
2015-02-20  2:45               ` Nelson, Sam
2014-03-17 19:58 ` Maupin, Chase

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.