Completely overlooked those...

I found the anonymous function you mentioned. How can this be modified to check 
for the target class?

Thanks,
Jake

-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Monday, February 23, 2015 4:15 PM
To: Stiffler, Jacob
Cc: meta-ti@yoctoproject.org
Subject: Re: [meta-ti] [PATCH v2 2/2] ti-cgt6x: Add verion 8.0.0

What about the rest of the comments?


On Mon, Feb 23, 2015 at 10:42:52AM -0500, Stiffler, Jacob wrote:
> This is the license string given by the OSRB. I will submit a v3 replacing 
> 'and' with '&'.
> 
> -----Original Message-----
> From: Dmytriyenko, Denys 
> Sent: Friday, February 20, 2015 12:25 PM
> To: Stiffler, Jacob
> Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH v2 2/2] ti-cgt6x: Add verion 8.0.0
> 
> On Thu, Feb 05, 2015 at 10:28:29AM -0500, Jacob Stiffler wrote:
> > * Supports native and nativesdk classes as well as target.
> > 
> > Signed-off-by: Jacob Stiffler <j-stiff...@ti.com>
> > ---
> >  recipes-ti/devtools/ti-cgt6x_8.0.0.bb |   88 
> > +++++++++++++++++++++++++++++++++
> >  1 file changed, 88 insertions(+)
> >  create mode 100644 recipes-ti/devtools/ti-cgt6x_8.0.0.bb
> > 
> > diff --git a/recipes-ti/devtools/ti-cgt6x_8.0.0.bb 
> > b/recipes-ti/devtools/ti-cgt6x_8.0.0.bb
> > new file mode 100644
> > index 0000000..c56a629
> > --- /dev/null
> > +++ b/recipes-ti/devtools/ti-cgt6x_8.0.0.bb
> > @@ -0,0 +1,88 @@
> > +DESCRIPTION = "TI DSP Code Generation Tools"
> > +HOMEPAGE = 
> > "https://www-a.ti.com/downloads/sds_support/TICodegenerationTools/download.htm";
> > +LICENSE = "TI TSPA with portions under Thai Open Source Software Center & 
> > BSD-3-Clause & BSL-1.0 & Hewlett-Packard & AFL-3.0 and MIT and BSD-2-Clause 
> > and Public Domain and other similar"
> 
> ^^^ This is a very strange license. Have you consulted with OSRB? Should 
> "and" 
> be replaced with "&" at the end?
> 
> 
> > +
> > +LIC_FILES_CHKSUM = 
> > "file://ti-cgt-c6000_${PV}/LICENSE.txt;md5=b6311962635a4f15630e36ec2d875eca"
> > +LIC_FILES_CHKSUM_class-target = 
> > "file://usr/share/doc/ti/cgt-c6x/LICENSE.txt;md5=b6311962635a4f15630e36ec2d875eca"
> > +
> > +PE = "1"
> > +
> > +require ../includes/ti-unpack.inc
> > +
> > +BINFILE = "ti_cgt_c6000_${PV}_linux_installer_x86.bin"
> > +BINFILE_NAME = "cgt6x_x86_installer"
> > +TI_BIN_UNPK_ARGS = "--prefix ${S}"
> > +TI_BIN_UNPK_CMDS = ""
> > +
> > +BINFILE_class-target = 
> > "ti_cgt_c6000_${PV}_armlinuxa8hf_busybox_installer.sh"
> > +BINFILE_NAME_class-target = "cgt6x_arm_installer"
> 
> Now, since target portion is very ARM specific, would be nice to add this:
> 
> COMPATIBLE_HOST_class-target = "arm.*-linux"
> 
> Also, you may want to copy the anonymous function that checks for hardfp from 
> libgles recipes. You might want to modify it to only check the target case 
> though, to not block native and nativesdk cases... Let me know if you have 
> any issues here.
> 
> 
> > +SRC_URI = 
> > "http://software-dl.ti.com/codegen/esd/cgt_public_sw/C6000/Production/${PV}/${BINFILE};name=${BINFILE_NAME}";
> > +
> > +SRC_URI[cgt6x_x86_installer.md5sum] = "ff83845672090bf0a669bba7b0cadf56"
> > +SRC_URI[cgt6x_x86_installer.sha256sum] = 
> > "297f77a00b211b3c4afa23afe7319e98b2400b5d2eb0f8c8cfe3ef7d7aa4d709"
> > +
> > +SRC_URI[cgt6x_arm_installer.md5sum] = "13961e5a1da4c310ea8c8707ab981fff"
> > +SRC_URI[cgt6x_arm_installer.sha256sum] = 
> > "cb24ae05baffbd44d6dedba68743be09fd883e04403e069c437c486533d8fffb"
> > +
> > +do_install() {
> > +    install -d ${D}${bindir}
> > +    for binfile in ${S}/ti-cgt-c6000_${PV}/bin/*; do
> > +        install -m 755 ${binfile} ${D}${bindir}
> > +    done
> > +
> > +    install -d ${D}${datadir}/ti/cgt-c6x/include
> > +    for includefile in ${S}/ti-cgt-c6000_${PV}/include/*; do
> > +        install -m 644 ${includefile} ${D}${datadir}/ti/cgt-c6x/include
> > +    done
> > +
> > +    install -d ${D}${datadir}/ti/cgt-c6x/lib
> > +    for libfile in ${S}/ti-cgt-c6000_${PV}/lib/*.a; do
> > +        install -m 644 ${libfile} ${D}${datadir}/ti/cgt-c6x/lib
> > +    done
> > +
> > +    for libfile in ${S}/ti-cgt-c6000_${PV}/lib/*.cmd; do
> > +        install -m 644 ${libfile} ${D}${datadir}/ti/cgt-c6x/lib
> > +    done
> > +
> > +    for libfile in ${S}/ti-cgt-c6000_${PV}/lib/*.lib; do
> > +        install -m 644 ${libfile} ${D}${datadir}/ti/cgt-c6x/lib
> > +    done
> > +
> > +    install -d ${D}${datadir}/ti/cgt-c6x/lib/src
> > +    for srcfile in ${S}/ti-cgt-c6000_${PV}/lib/src/*; do
> > +        install -m 644 ${srcfile} ${D}${datadir}/ti/cgt-c6x/lib/src
> > +    done
> > +
> > +    install -d ${D}${datadir}/man/man1
> > +    for manfile in ${S}/ti-cgt-c6000_${PV}/man/man1/*; do
> > +        install -m 644 ${manfile} ${D}${datadir}/man/man1
> > +    done
> > +
> > +    install -d ${D}${datadir}/doc/ti/cgt-c6x
> > +    for docfile in ${S}/ti-cgt-c6000_${PV}/*.txt; do
> > +        install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
> > +    done
> > +
> > +    for docfile in ${S}/ti-cgt-c6000_${PV}/*.htm; do
> > +        install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
> > +    done
> > +
> > +    for docfile in ${S}/ti-cgt-c6000_${PV}/*.spdx; do
> > +        install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
> > +    done
> > +
> > +    for docfile in ${S}/ti-cgt-c6000_${PV}/doc/*; do
> > +        install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
> > +    done
> > +
> > +}
> > +
> > +do_install_class-target() {
> > +    ${WORKDIR}/${BINFILE} --prefix ${D} }
> > +
> > +FILES_${PN} += "${datadir}/ti/*"
> > +
> > +BBCLASSEXTEND = "native nativesdk"
> > +
> > --
> > 1.7.9.5
> > 
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
-- 
_______________________________________________
meta-ti mailing list
meta-ti@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-ti

Reply via email to