Message ID | 20220427052715.7642-1-br015@umbiko.net |
---|---|
State | Superseded |
Headers | show |
Series | [1/1] package/mpd: fix iconv detection w/ libiconv installed | expand |
On 27/04/2022 07:27, Andreas Ziegler wrote: > Fix build failures caused by meson failing to properly detect iconv() when > libiconv is installed and a missing link dependency. > > Upstream pull request: > https://github.com/MusicPlayerDaemon/MPD/pull/1515 > > Fixes the following build failures: > http://autobuild.buildroot.net/results/7a0/7a0fe4e9248ed96a5c4934361de16e0b59a51d50/ > > Signed-off-by: Andreas Ziegler <br015@umbiko.net> > --- > ...detection-when-libiconv-is-installed.patch | 71 +++++++++++++++++++ > 1 file changed, 71 insertions(+) > create mode 100644 package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch > > diff --git a/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch b/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch > new file mode 100644 > index 0000000000..6121bc9740 > --- /dev/null > +++ b/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch > @@ -0,0 +1,71 @@ > +From 3882a5a263caa681778a21b1f5f13a1b64536796 Mon Sep 17 00:00:00 2001 > +From: aeolio <git@aeolio.de> > +Date: Wed, 20 Apr 2022 16:10:39 +0200 > +Subject: [PATCH 1/2] src/lib/icu: fix iconv() detection when libiconv is Remove the N/M (git format-patch -N). > + installed > + Add a Signed-off-by line for yourself. This is a short way for you to assert that you are entitled to contribute the patch under the upstream project's license. See http://elinux.org/Developer_Certificate_Of_Origin for more details. > +--- > + src/lib/icu/meson.build | 11 +++++++++-- > + 1 file changed, 9 insertions(+), 2 deletions(-) > + > +diff --git a/src/lib/icu/meson.build b/src/lib/icu/meson.build > +index 59215e704b..972c1fda3f 100644 > +--- a/src/lib/icu/meson.build > ++++ b/src/lib/icu/meson.build > +@@ -12,17 +12,23 @@ if is_windows > + icu_sources += 'Win32.cxx' > + endif > + > ++iconv_dep = [] > + if icu_dep.found() > + icu_sources += [ > + 'Util.cxx', > + 'Init.cxx', > + ] > + elif not get_option('iconv').disabled() > +- have_iconv = compiler.has_function('iconv', prefix : '#include <iconv.h>') > +- conf.set('HAVE_ICONV', have_iconv) > ++ # an installed iconv library will make the builtin iconf() unavailable, > ++ # so search for the library first and pass it as (possible) dependency > ++ iconv_dep = compiler.find_library('libiconv', required: false) > ++ have_iconv = compiler.has_function('iconv', > ++ dependencies: iconv_dep, > ++ prefix : '#include <iconv.h>') > + if not have_iconv and get_option('iconv').enabled() > + error('iconv() not available') > + endif > ++ conf.set('HAVE_ICONV', have_iconv) > + endif > + > + icu = static_library( > +@@ -31,6 +37,7 @@ icu = static_library( > + include_directories: inc, > + dependencies: [ > + icu_dep, > ++ iconv_dep, > + fmt_dep, > + ], > + ) > + > +From ee39af34199aa875e109f7002a401d6ebbd66c22 Mon Sep 17 00:00:00 2001 > +From: aeolio <git@aeolio.de> > +Date: Sun, 24 Apr 2022 04:14:17 +0000 > +Subject: [PATCH 2/2] fix typo in comment Only one patch per patch file please. Though I don't see why you even need this patch. Regards, Arnout > + > +--- > + src/lib/icu/meson.build | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/src/lib/icu/meson.build b/src/lib/icu/meson.build > +index 972c1fda3f..6e489876b8 100644 > +--- a/src/lib/icu/meson.build > ++++ b/src/lib/icu/meson.build > +@@ -19,7 +19,7 @@ if icu_dep.found() > + 'Init.cxx', > + ] > + elif not get_option('iconv').disabled() > +- # an installed iconv library will make the builtin iconf() unavailable, > ++ # an installed iconv library will make the builtin iconv() unavailable, > + # so search for the library first and pass it as (possible) dependency > + iconv_dep = compiler.find_library('libiconv', required: false) > + have_iconv = compiler.has_function('iconv',
diff --git a/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch b/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch new file mode 100644 index 0000000000..6121bc9740 --- /dev/null +++ b/package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch @@ -0,0 +1,71 @@ +From 3882a5a263caa681778a21b1f5f13a1b64536796 Mon Sep 17 00:00:00 2001 +From: aeolio <git@aeolio.de> +Date: Wed, 20 Apr 2022 16:10:39 +0200 +Subject: [PATCH 1/2] src/lib/icu: fix iconv() detection when libiconv is + installed + +--- + src/lib/icu/meson.build | 11 +++++++++-- + 1 file changed, 9 insertions(+), 2 deletions(-) + +diff --git a/src/lib/icu/meson.build b/src/lib/icu/meson.build +index 59215e704b..972c1fda3f 100644 +--- a/src/lib/icu/meson.build ++++ b/src/lib/icu/meson.build +@@ -12,17 +12,23 @@ if is_windows + icu_sources += 'Win32.cxx' + endif + ++iconv_dep = [] + if icu_dep.found() + icu_sources += [ + 'Util.cxx', + 'Init.cxx', + ] + elif not get_option('iconv').disabled() +- have_iconv = compiler.has_function('iconv', prefix : '#include <iconv.h>') +- conf.set('HAVE_ICONV', have_iconv) ++ # an installed iconv library will make the builtin iconf() unavailable, ++ # so search for the library first and pass it as (possible) dependency ++ iconv_dep = compiler.find_library('libiconv', required: false) ++ have_iconv = compiler.has_function('iconv', ++ dependencies: iconv_dep, ++ prefix : '#include <iconv.h>') + if not have_iconv and get_option('iconv').enabled() + error('iconv() not available') + endif ++ conf.set('HAVE_ICONV', have_iconv) + endif + + icu = static_library( +@@ -31,6 +37,7 @@ icu = static_library( + include_directories: inc, + dependencies: [ + icu_dep, ++ iconv_dep, + fmt_dep, + ], + ) + +From ee39af34199aa875e109f7002a401d6ebbd66c22 Mon Sep 17 00:00:00 2001 +From: aeolio <git@aeolio.de> +Date: Sun, 24 Apr 2022 04:14:17 +0000 +Subject: [PATCH 2/2] fix typo in comment + +--- + src/lib/icu/meson.build | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/src/lib/icu/meson.build b/src/lib/icu/meson.build +index 972c1fda3f..6e489876b8 100644 +--- a/src/lib/icu/meson.build ++++ b/src/lib/icu/meson.build +@@ -19,7 +19,7 @@ if icu_dep.found() + 'Init.cxx', + ] + elif not get_option('iconv').disabled() +- # an installed iconv library will make the builtin iconf() unavailable, ++ # an installed iconv library will make the builtin iconv() unavailable, + # so search for the library first and pass it as (possible) dependency + iconv_dep = compiler.find_library('libiconv', required: false) + have_iconv = compiler.has_function('iconv',
Fix build failures caused by meson failing to properly detect iconv() when libiconv is installed and a missing link dependency. Upstream pull request: https://github.com/MusicPlayerDaemon/MPD/pull/1515 Fixes the following build failures: http://autobuild.buildroot.net/results/7a0/7a0fe4e9248ed96a5c4934361de16e0b59a51d50/ Signed-off-by: Andreas Ziegler <br015@umbiko.net> --- ...detection-when-libiconv-is-installed.patch | 71 +++++++++++++++++++ 1 file changed, 71 insertions(+) create mode 100644 package/mpd/0002-fix-iconv-detection-when-libiconv-is-installed.patch