Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0

[Denys Dmytriyenko <denys@ti.com>]

On Tue, May 19, 2015 at 08:12:43AM -0400, Jacob Stiffler wrote:
> 
> 
> On 5/18/2015 3:07 PM, Denys Dmytriyenko wrote:
> >On Mon, May 18, 2015 at 03:00:37PM -0400, Jacob Stiffler wrote:
> >>
> >>On 5/18/2015 1:41 PM, Denys Dmytriyenko wrote:
> >>>On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
> >>>>* c6x DSP development files required to support remote gdb debugging.
> >>>>
> >>>>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>>>---
> >>>>  recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
> >>>>  1 file changed, 46 insertions(+)
> >>>>  create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>
> >>>>diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>new file mode 100644
> >>>>index 0000000..d21f03f
> >>>>--- /dev/null
> >>>>+++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>@@ -0,0 +1,46 @@
> >>>>+DESCRIPTION="TI gdb-server to be used with c6xgdb."
> >>>>+
> >>>>+LICENSE = "BSD"
> >>>>+LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
> >>>>+
> >>>>+include gdbc6x.inc
> >>>>+
> >>>>+COMPATIBLE_MACHINE = "dra7xx"
> >>>>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>>>+
> >>>>+PR = "${INC_PR}.0"
> >>>>+
> >>>>+require recipes-ti/includes/ti-paths.inc
> >>>>+
> >>>>+DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
> >>>>+
> >>>>+export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
> >>>>+export FC_DIR = "${FC_INSTALL_DIR}"
> >>>>+export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> >>>>+export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
> >>>>+export XDC_DIR = "${XDC_INSTALL_DIR}"
> >>>>+
> >>>>+PLATFORM = ""
> >>>>+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> >>>>+
> >>>>+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> >>>>+
> >>>>+PARALLEL_MAKE = ""
> >>>>+
> >>>>+S = "${WORKDIR}/git/gdbserver-c6x/src"
> >>>>+
> >>>>+do_install() {
> >>>>+    install -d ${D}${datadir}/ti/gdbc6x/include
> >>>>+    install -d ${D}${datadir}/ti/gdbc6x/lib
> >>>>+    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
> >>>>+    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
> >>>>+}
> >>>>+
> >>>>+ALLOW_EMPTY_${PN} = "1"
> >>>Why?
> >>>
> >>This recipes only popultaes the -dev and -staticdev packages.
> >So, why do you need an empty package then? If you only need -dev and
> >-staticdev added as a dependency for the devkit, this shouldn't be required.
> >Unless there's a need to depend on the main package someplace, in which case
> >you would create an empty one...
> >
> 
> Without this, I am seeing an error when populating the devkit:
> 
> Collected errors:
>  * satisfy_dependencies_for: Cannot satisfy the following
> dependencies for packagegroup-arago-toolchain-tisdk-target:
>  *      libulm (= 1.2.0-r0.1) *         gdbserver-c6x (= 1.1.0-r0.1) *
>  * opkg_install_cmd: Cannot install package
> packagegroup-arago-toolchain-tisdk-target.

I guess we could break the default dependency between -dev and the main 
package, but an empty package would be fine in this case too.


> >>>>+FILES_${PN}-dev += "\
> >>>>+    ${datadir}/ti/gdbc6x \
> >>>>+"
> >>>>+
> >>>>+do_configure[noexec] = "1"
> >>>>+do_qa_configure[noexec] = "1"
> >>>Why?
> >>>
> >>>
> >>>>-- 
> >>>>1.7.9.5
> >>>>
> >>>>-- 
> >>>>_______________________________________________
> >>>>meta-ti mailing list
> >>>>meta-ti@yoctoproject.org
> >>>>https://lists.yoctoproject.org/listinfo/meta-ti
>