[OE-core] [PATCH] libidn2: Fix charset.alias issue with musl

[Burton, Ross]

Has that actually been fixed in gnulib yet?

Ross

On 3 August 2018 at 18:30, Khem Raj <raj.khem at gmail.com> wrote:
> Fixes
> ERROR: libidn2-2.0.5-r0 do_package: QA Issue: libidn2: Files/directories were installed but not shipped in any packag
> e:
>   /usr/lib/charset.alias
> Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or del
> ete them within do_install.
>
> Signed-off-by: Khem Raj <raj.khem at gmail.com>
> ---
>  ...charset_alias-when-building-for-musl.patch | 23 +++++++++++++++++++
>  meta/recipes-extended/libidn/libidn2_2.0.5.bb |  4 +++-
>  2 files changed, 26 insertions(+), 1 deletion(-)
>  create mode 100644 meta/recipes-extended/libidn/libidn2/Unset-need_charset_alias-when-building-for-musl.patch
>
> diff --git a/meta/recipes-extended/libidn/libidn2/Unset-need_charset_alias-when-building-for-musl.patch b/meta/recipes-extended/libidn/libidn2/Unset-need_charset_alias-when-building-for-musl.patch
> new file mode 100644
> index 0000000000..ace50bbfde
> --- /dev/null
> +++ b/meta/recipes-extended/libidn/libidn2/Unset-need_charset_alias-when-building-for-musl.patch
> @@ -0,0 +1,23 @@
> +Unset need_charset_alias when building for musl
> +
> +localcharset uses ac_cv_gnu_library_2_1 from glibc21.m4
> +which actually shoudl be fixed in gnulib and then all downstream
> +projects will get it eventually. For now we apply the fix to
> +coreutils
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Khem Raj <raj.khem at gmail.com>
> +Index: libidn2-2.0.5/unistring/Makefile.am
> +===================================================================
> +--- libidn2-2.0.5.orig/unistring/Makefile.am
> ++++ libidn2-2.0.5/unistring/Makefile.am
> +@@ -293,7 +293,7 @@ install-exec-localcharset: all-local
> +         case '$(host_os)' in \
> +           darwin[56]*) \
> +             need_charset_alias=true ;; \
> +-          darwin* | cygwin* | mingw* | pw32* | cegcc*) \
> ++          darwin* | cygwin* | mingw* | pw32* | cegcc* | linux-musl*) \
> +             need_charset_alias=false ;; \
> +           *) \
> +             need_charset_alias=true ;; \
> diff --git a/meta/recipes-extended/libidn/libidn2_2.0.5.bb b/meta/recipes-extended/libidn/libidn2_2.0.5.bb
> index aaf9ecd7e7..0d7bddbc7f 100644
> --- a/meta/recipes-extended/libidn/libidn2_2.0.5.bb
> +++ b/meta/recipes-extended/libidn/libidn2_2.0.5.bb
> @@ -9,7 +9,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=ab90e75ef97cc6318ce4f2fbda62fe4d \
>                      file://src/idn2.c;endline=16;md5=0283aec28e049f5bcaaeee52aa865874 \
>                      file://lib/idn2.h.in;endline=27;md5=c2cd28d3f87260f157f022eabb83714f"
>
> -SRC_URI = "${GNU_MIRROR}/libidn/${BPN}-${PV}.tar.gz"
> +SRC_URI = "${GNU_MIRROR}/libidn/${BPN}-${PV}.tar.gz \
> +           file://Unset-need_charset_alias-when-building-for-musl.patch \
> +          "
>  SRC_URI[md5sum] = "eaf9a5b9d03b0cce3760f34b3124eb36"
>  SRC_URI[sha256sum] = "53f69170886f1fa6fa5b332439c7a77a7d22626a82ef17e2c1224858bb4ca2b8"
>
> --
> 2.18.0
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core