Re: [rocko/master][PATCH 4/7] ti-cgt6x: Update to version 8.3.2

[Denys Dmytriyenko <denys@ti.com>]

On Mon, Mar 11, 2019 at 09:43:45PM -0400, Mahesh Radhakrishnan wrote:
> The license checksum changed due to the version, licenses have not
> changed.
> Signed-off-by: Mahesh Radhakrishnan <m-radhakrishnan2@ti.com>
> ---
>  .../{ti-cgt6x_8.2.2.bb => ti-cgt6x_8.3.2.bb}       | 32 ++++++++--------------
>  1 file changed, 11 insertions(+), 21 deletions(-)
>  rename recipes-ti/devtools/{ti-cgt6x_8.2.2.bb => ti-cgt6x_8.3.2.bb} (70%)
> 
> diff --git a/recipes-ti/devtools/ti-cgt6x_8.2.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> similarity index 70%
> rename from recipes-ti/devtools/ti-cgt6x_8.2.2.bb
> rename to recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> index 53f5252..cf1d4eb 100644
> --- a/recipes-ti/devtools/ti-cgt6x_8.2.2.bb
> +++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> @@ -1,9 +1,9 @@
>  DESCRIPTION = "TI DSP Code Generation Tools"
>  HOMEPAGE = "https://www-a.ti.com/downloads/sds_support/TICodegenerationTools/download.htm"
> -LICENSE = "(TI-TSPA & Thai-Open-Source-Software-Center) & BSD-3-Clause & BSL-1.0 & Hewlett-Packard & AFL-3.0 & MIT & BSD-2-Clause & PD"
> +LICENSE = "(TI-TSPA & Thai-Open-Source-Software-Center) & BSD-3-Clause & BSL-1.0 & Patrick-Powell & AFL-3.0 & MIT & BSD-2-Clause & PD"

In ti-cgt-arm patch #6 you have this text regarding the LICENSE change to 
include Patrick-Powell:

> However Patrick powell license which was missing earlier has been added now.

First of all, this text is missing here in the commit log.
Second, in ti-cgt-arm Patrick-Powell is added, and here it replaces 
Hewlett-Packard. Please double-check which way is correct and update the 
commit log to mention this. Thanks!


> -LIC_FILES_CHKSUM = "file://ti-cgt-c6000_${PV}/C6000_8.2.x_CodeGenerationTools_Manifest.htm;md5=66147c95fc5d3f900b73414f8258c91e"
> -LIC_FILES_CHKSUM_class-target = "file://usr/share/doc/ti/cgt-c6x/C6000_8.2.x_CodeGenerationTools_Manifest.htm;md5=66147c95fc5d3f900b73414f8258c91e"
> +LIC_FILES_CHKSUM = "file://ti-cgt-c6000_${PV}/C6000CodeGenerationTools_8.3.x_manifest.html;md5=d06c6f9acebf78df4108a8535396e9f1"
> +LIC_FILES_CHKSUM_class-target = "file://usr/share/doc/ti/cgt-c6x/C6000CodeGenerationTools_8.3.x_manifest.html;md5=d06c6f9acebf78df4108a8535396e9f1"
>  
>  PE = "1"
>  
> @@ -37,11 +37,13 @@ BINFILE_NAME_class-target = "cgt6x_arm_installer"
>  
>  SRC_URI = "http://software-dl.ti.com/codegen/esd/cgt_public_sw/C6000/${PV}/${BINFILE};name=${BINFILE_NAME}"
>  
> -SRC_URI[cgt6x_x86_installer.md5sum] = "548c841b231c2fbd954218d4e370c995"
> -SRC_URI[cgt6x_x86_installer.sha256sum] = "f269e51eb29de2efd32b3ea93beebd7b00a1a156c923be1a33ba7d785c4c6a24"
> +SRC_URI[cgt6x_x86_installer.md5sum] = "f1f534e6a4bdee6df39e8d0cb458d161"
> +SRC_URI[cgt6x_x86_installer.sha256sum] = "1fba2a8f5532b33a23677771d686a866171ce7a0e567fed50d900d0ecd2e17e6"
>  
> -SRC_URI[cgt6x_arm_installer.md5sum] = "56f5e5464ab87af517a6cdd5fed06f3f"
> -SRC_URI[cgt6x_arm_installer.sha256sum] = "4b2e605a4afc3d4d0ac74013ee32496541abf093aedd70bc7ef040597933b1ea"
> +SRC_URI[cgt6x_arm_installer.md5sum] = "425d82308e71202ad004a36b7ec3cec9"
> +SRC_URI[cgt6x_arm_installer.sha256sum] = "70c387ddde379194fed712087be6cb0ca5f4f0e65d7f29bd7462c38ee011928f"
> +
> +CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
>  
>  do_install() {
>      install -d ${D}${bindir}
> @@ -54,10 +56,7 @@ do_install() {
>          install -m 755 ${binfile} ${D}${datadir}/ti/cgt-c6x/bin
>      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
> +    cp ${CP_ARGS}  ${S}/ti-cgt-c6000_${PV}/include ${D}${datadir}/ti/cgt-c6x/include 

There's an extra white-space at the end of this ^^ line and git apply will 
complain about it.


>      install -d ${D}${datadir}/ti/cgt-c6x/lib
>      for libfile in ${S}/ti-cgt-c6000_${PV}/lib/*.a; do
> @@ -72,11 +71,6 @@ do_install() {
>          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
>          if [ -e ${manfile} ]; then
> @@ -89,7 +83,7 @@ do_install() {
>          install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
>      done
>  
> -    for docfile in ${S}/ti-cgt-c6000_${PV}/*.htm; do
> +    for docfile in ${S}/ti-cgt-c6000_${PV}/*.html; do
>          install -m 644 ${docfile} ${D}${datadir}/doc/ti/cgt-c6x
>      done
>  
> @@ -97,10 +91,6 @@ do_install() {
>          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() {
> -- 
> 1.9.1
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti