Re: [PATCH 2/4] external-arm-toolchain: Refine dev libraies packaging

["Denys Dmytriyenko" <denis@denix.org>]

On Tue, Jun 02, 2020 at 04:34:52PM +0530, Sumit Garg wrote:
> OE core by default package all libraies installed as "${libdir}/lib*.so"
> or "${libdir}/*.la" as part of "FILES_${PN}-dev" but in case of external
> Arm tool-set recipe, multiple packages are provided which needs to
> include specific libraries. So split dev libraries packaging accordingly
> and only package remaining libraries as part of "FILES_${PN}-dev" which
> aren't provided as part of other packages.
> 
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> ---
>  .../external-arm-toolchain.bb                      | 23 ++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/meta-arm-toolchain/recipes-devtools/external-arm-toolchain/external-arm-toolchain.bb b/meta-arm-toolchain/recipes-devtools/external-arm-toolchain/external-arm-toolchain.bb
> index 04d8923..7f5a6f5 100644
> --- a/meta-arm-toolchain/recipes-devtools/external-arm-toolchain/external-arm-toolchain.bb
> +++ b/meta-arm-toolchain/recipes-devtools/external-arm-toolchain/external-arm-toolchain.bb
> @@ -545,6 +545,29 @@ FILES_${PN} += "\
>  	${base_libdir}/libpcprofile.so \
>      "
>  
> +FILES_${PN}-dev_remove = "\

Shouldn't be using _remove here. It will probably break anyones .bbappends on 
this recipe and will make it hard making any adjustments.

If you check git logs, you'll see that every time we needed to change the 
default lists defined by glibc-package.inc in OE-Core, we had to re-create 
them from scratch.


> +	${libdir}/lib*.so \
> +	${libdir}/*.la \
> +    "
> +FILES_${PN}-dev += "\
> +	${libdir}/libcrypt.so \
> +	${libdir}/libc.so \
> +	${libdir}/libm.so \
> +	${libdir}/libpthread.so \
> +	${libdir}/libresolv.so \
> +	${libdir}/librt.so \
> +	${libdir}/libutil.so \
> +	${libdir}/libnss_files.so \
> +	${libdir}/libnss_compat.so \
> +	${libdir}/libnss_dns.so \
> +	${libdir}/libnss_hesiod.so \
> +	${libdir}/libnss_db.so \
> +	${libdir}/libdl.so \
> +	${libdir}/libanl.so \
> +	${libdir}/libBrokenLocale.so \
> +	${libdir}/libthread_db.so \
> +    "
> +
>  FILES_${PN}-dbg += "${base_libdir}/debug"
>  
>  # From gcc-runtime.inc
> -- 
> 2.7.4
>