Re: [meta-ti] [master][PATCH] meta-ti: correct and do not skip "installed-vs-shipped" QA check

* Re: [meta-ti] [master][PATCH] meta-ti: correct and do not skip "installed-vs-shipped" QA check
       [not found] <16D7256159B60D66.7393@lists.yoctoproject.org>
@ 2022-03-01 17:32 ` Denys Dmytriyenko
  2022-03-01 17:55   ` Ryan Eatmon
  0 siblings, 1 reply; 2+ messages in thread
From: Denys Dmytriyenko @ 2022-03-01 17:32 UTC (permalink / raw)
  To: meta-ti, Ryan Eatmon

Ping.

On Fri, Feb 25, 2022 at 09:41:25PM +0000, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@konsulko.com>
> 
> Skipping "installed-vs-shipped" QA check is now considered bad practice
> due to possible SDK issues and is being flagged by yocto-check-layer [1]
> script and could affect "Yocto Project Compatible" status for the layer.
> 
> [1] https://git.openembedded.org/openembedded-core/commit/?id=e8baa75535fc888f1d768b23a0140475e832c910
> 
> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> ---
>  meta-ti-extras/recipes-devtools/cuia/cuia_1.0.0.13.bb  | 5 ++++-
>  meta-ti-extras/recipes-ti/vlib/vlib-c66x.inc           | 2 +-
>  meta-ti-extras/recipes-ti/xdais/ti-xdais_6.26.01.03.bb | 2 --
>  3 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/meta-ti-extras/recipes-devtools/cuia/cuia_1.0.0.13.bb b/meta-ti-extras/recipes-devtools/cuia/cuia_1.0.0.13.bb
> index 1cda366d..8b16830f 100644
> --- a/meta-ti-extras/recipes-devtools/cuia/cuia_1.0.0.13.bb
> +++ b/meta-ti-extras/recipes-devtools/cuia/cuia_1.0.0.13.bb
> @@ -22,9 +22,12 @@ do_compile () {
>  do_install() {
>  	cd ${S}
>  	oe_runmake DESTDIR=${D} libdir=${libdir} includedir=${includedir} install
> +	rm -rf ${D}${libdir}/boot.*
> +	rm -rf ${D}${libdir}/gnu.targets.*
> +	rm -rf ${D}${libdir}/syscalls.*
>  }
>  
>  COMPATIBLE_HOST ?= "null"
>  COMPATIBLE_HOST:ti-soc = "(.*)"
>  
> -INSANE_SKIP:${PN} += "textrel installed-vs-shipped"
> +INSANE_SKIP:${PN} += "textrel"
> diff --git a/meta-ti-extras/recipes-ti/vlib/vlib-c66x.inc b/meta-ti-extras/recipes-ti/vlib/vlib-c66x.inc
> index 460964b3..6b5e186c 100644
> --- a/meta-ti-extras/recipes-ti/vlib/vlib-c66x.inc
> +++ b/meta-ti-extras/recipes-ti/vlib/vlib-c66x.inc
> @@ -29,4 +29,4 @@ FILES:${PN}-dev += "${VLIB_C66_INSTALL_DIR_RECIPE}"
>  INHIBIT_PACKAGE_STRIP = "1"
>  INHIBIT_SYSROOT_STRIP = "1"
>  INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> -INSANE_SKIP:${PN}-dev = "installed-vs-shipped arch"
> +INSANE_SKIP:${PN}-dev = "arch"
> diff --git a/meta-ti-extras/recipes-ti/xdais/ti-xdais_6.26.01.03.bb b/meta-ti-extras/recipes-ti/xdais/ti-xdais_6.26.01.03.bb
> index 9fc4cee1..ee9826e6 100644
> --- a/meta-ti-extras/recipes-ti/xdais/ti-xdais_6.26.01.03.bb
> +++ b/meta-ti-extras/recipes-ti/xdais/ti-xdais_6.26.01.03.bb
> @@ -7,5 +7,3 @@ LIC_FILES_CHKSUM = "file://xdais_6_26_01_03_manifest.html;md5=a5a675d49db2f658bb
>  
>  SRC_URI[xdaistarball.md5sum] = "9b2bea3408254e737ec0d819a031adf6"
>  SRC_URI[xdaistarball.sha256sum] = "d7bcbd8bcfe6fdfc5f402e4b125ced6c806dc30bc5296e98d5d43b0b5a237a0d"
> -
> -INSANE_SKIP:${PN} = "installed-vs-shipped"
> -- 
> 2.25.1
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread