> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, June 04, 2014 10:23 AM
> To: Maupin, Chase
> Cc: Cooper Jr., Franklin; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 05/11] matrix-gui: Add camera submenu
> 
> On Wed, Jun 04, 2014 at 02:32:37PM +0000, Maupin, Chase wrote:
> > >-----Original Message-----
> > >From: meta-arago-boun...@arago-project.org [mailto:meta-arago-
> > >boun...@arago-project.org] On Behalf Of Cooper Jr., Franklin
> > >Sent: Thursday, May 29, 2014 1:38 PM
> > >To: meta-arago@arago-project.org
> > >Cc: Cooper Jr., Franklin
> > >Subject: [meta-arago] [PATCH 05/11] matrix-gui: Add camera submenu
> > >
> > >Signed-off-by: Franklin S. Cooper Jr <fcoo...@ti.com>
> > >---
> > > .../recipes-core/matrix/matrix-gui-apps-git.inc    |    4 ++--
> > > .../recipes-core/matrix/matrix-gui-submenus_2.0.bb |    5 +++--
> > > 2 files changed, 5 insertions(+), 4 deletions(-)
> > >
> > >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >apps-git.inc b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >apps-git.inc
> > >index fe4b3f0..c985859 100644
> > >--- a/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > >git.inc
> > >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-apps-
> > >git.inc
> > >@@ -4,9 +4,9 @@ LICENSE_CHECKSUM =
> > >"LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > > LIC_FILES_CHKSUM := "file://../${LICENSE_CHECKSUM}"
> > >
> > > SRC_URI = "git://gitorious.org/matrix-gui-v2/matrix-gui-v2-
> > >apps.git;protocol=git;branch=${BRANCH}"
> > >-SRCREV = "ea1592722e9b00a6324d031af1fa23a3683ff7a6"
> > >+SRCREV = "6ba41c4070cbc723d758f5d493859ae6470cdceb"
> > > BRANCH = "master"
> > >-INC_PR = "r22"
> > >+INC_PR = "r23"
> > >
> > > # Pull in the base package for installing matrix applications
> > >require matrix-gui-apps.inc diff --git
> > >a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >submenus_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-
> > >gui-submenus_2.0.bb
> > >index d108500..d794277 100644
> > >--- a/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >submenus_2.0.bb
> > >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-
> > >submenus_2.0.bb
> > >@@ -8,7 +8,7 @@ require matrix-gui-apps-git.inc  # reset the
> > >LIC_FILES_CHKSUM setting from the matrix-gui-apps- git.inc file
> > >LIC_FILES_CHKSUM =
> > >"file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> > >
> > >-PR = "${INC_PR}.9"
> > >+PR = "${INC_PR}.10"
> >
> > Just a nit, but if you bumped the INC_PR already then do you really
> > need to bump this PR?  I don't think this is worth submitting a v2 for
> though.
> 
> From my perspective - if you made any changes in the .inc file, you bump
> INC_PR and then if you made any changes to the actual recipe, you bump it
> own PR. That way you can track the history of changes in each file
> separately...
[Franklin] Your both right. I'll fix this.
> 
> 
> > > # These packages make submenus in matrix and are not architecture
> > >specific  inherit allarch @@ -27,7 +27,7 @@ do_install(){
> > >     done
> > > }
> > >
> > >-PACKAGES += "${PN}-arm ${PN}-3d ${PN}-cryptos ${PN}-display
> > >${PN}-ethernet ${PN}-multimedia ${PN}-power ${PN}-pru ${PN}-qt4
> > >${PN}-settings ${PN}-usb ${PN}-wifi ${PN}-oprofile"
> > >+PACKAGES += "${PN}-arm ${PN}-3d ${PN}-cryptos ${PN}-display
> > >${PN}-ethernet ${PN}-multimedia ${PN}-power ${PN}-pru ${PN}-qt4
> > >${PN}-settings ${PN}-usb ${PN}-wifi ${PN}-oprofile ${PN}-camera"
> > >
> > > # Make sure app images has been installed  RDEPENDS_${PN} +=
> > >"matrix-gui-apps-images"
> > >@@ -46,3 +46,4 @@ FILES_${PN}-settings =
> > >"${MATRIX_APP_DIR}/settings_submenu/*"
> > > FILES_${PN}-usb = "${MATRIX_APP_DIR}/usb_submenu/*"
> > > FILES_${PN}-wifi = "${MATRIX_APP_DIR}/wifi_submenu/*"
> > > FILES_${PN}-oprofile = "${MATRIX_APP_DIR}/oprofile_submenu/*"
> > >+FILES_${PN}-camera = "${MATRIX_APP_DIR}/camera_submenu/*"
> > >--
> > >1.7.0.4
> > >
> > >_______________________________________________
> > >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

Reply via email to