All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] libidn2: Fix charset.alias issue with musl
@ 2018-08-03 17:30 Khem Raj
  2018-08-03 18:16 ` Burton, Ross
  0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2018-08-03 17:30 UTC (permalink / raw)
  To: openembedded-core

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@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@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



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

* Re: [PATCH] libidn2: Fix charset.alias issue with musl
  2018-08-03 17:30 [PATCH] libidn2: Fix charset.alias issue with musl Khem Raj
@ 2018-08-03 18:16 ` Burton, Ross
  2018-08-03 18:38   ` Khem Raj
  0 siblings, 1 reply; 3+ messages in thread
From: Burton, Ross @ 2018-08-03 18:16 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core

Has that actually been fixed in gnulib yet?

Ross

On 3 August 2018 at 18:30, Khem Raj <raj.khem@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@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@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@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


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

* Re: [PATCH] libidn2: Fix charset.alias issue with musl
  2018-08-03 18:16 ` Burton, Ross
@ 2018-08-03 18:38   ` Khem Raj
  0 siblings, 0 replies; 3+ messages in thread
From: Khem Raj @ 2018-08-03 18:38 UTC (permalink / raw)
  To: Burton, Ross; +Cc: Patches and discussions about the oe-core layer

On Fri, Aug 3, 2018 at 11:17 AM Burton, Ross <ross.burton@intel.com> wrote:
>
> Has that actually been fixed in gnulib yet?
>

haven't check lately but few months back it is not fixed yet.

> Ross
>
> On 3 August 2018 at 18:30, Khem Raj <raj.khem@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@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@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@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core


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

end of thread, other threads:[~2018-08-03 18:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-03 17:30 [PATCH] libidn2: Fix charset.alias issue with musl Khem Raj
2018-08-03 18:16 ` Burton, Ross
2018-08-03 18:38   ` Khem Raj

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.