Re: [PATCH v2 2/2] ti-cgt6x: Add verion 8.0.0

[Denys Dmytriyenko <denys@ti.com>]

You can check CLASSOVERRIDE variable to be "class-target".


On Tue, Feb 24, 2015 at 08:26:55AM -0500, Stiffler, Jacob wrote:
> 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-stiffler@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