| Message ID | 20201014221939.196958-1-brogers@suse.com |
|---|---|
| State | New |
| Headers | show |
| Series | meson.build: don't condition iconv detection on library detection | expand |
On Thu, Oct 15, 2020 at 6:19 AM Bruce Rogers <brogers@suse.com> wrote: > > It isn't necessarily the case that use of iconv requires an additional > library. For that reason we shouldn't conditionalize iconv detection on > libiconv.found. > > Fixes: 5285e593c33 (configure: Fixes ncursesw detection under msys2/mingw by convert them to meson) > > Signed-off-by: Bruce Rogers <brogers@suse.com> > --- > meson.build | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/meson.build b/meson.build > index 1a4a482492..84c8ec9541 100644 > --- a/meson.build > +++ b/meson.build > @@ -431,15 +431,13 @@ if not get_option('iconv').disabled() > libiconv = cc.find_library('iconv', > required: false, > static: enable_static) > - if libiconv.found() > - if cc.links(''' > - #include <iconv.h> > - int main(void) { > - iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > - return conv != (iconv_t) -1; > - }''', dependencies: [libiconv]) > - iconv = declare_dependency(dependencies: [libiconv]) > - endif > + if cc.links(''' > + #include <iconv.h> > + int main(void) { > + iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > + return conv != (iconv_t) -1; > + }''', dependencies: [libiconv]) > + iconv = declare_dependency(dependencies: [libiconv]) > endif > endif > if get_option('iconv').enabled() and not iconv.found() > -- > 2.28.0 > Reviewed-by: Yonggang Luo<l <brogers@suse.com>uoyonggang@gmail.com> -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo <div dir="ltr"><br><br>On Thu, Oct 15, 2020 at 6:19 AM Bruce Rogers <<a href="mailto:brogers@suse.com">brogers@suse.com</a>> wrote:<br>><br>> It isn't necessarily the case that use of iconv requires an additional<br>> library. For that reason we shouldn't conditionalize iconv detection on<br>> libiconv.found.<br>><br>> Fixes: 5285e593c33 (configure: Fixes ncursesw detection under msys2/mingw by convert them to meson)<br>><br>> Signed-off-by: Bruce Rogers <<a href="mailto:brogers@suse.com">brogers@suse.com</a>><br>> ---<br>> meson.build | 16 +++++++---------<br>> 1 file changed, 7 insertions(+), 9 deletions(-)<br>><br>> diff --git a/meson.build b/meson.build<br>> index 1a4a482492..84c8ec9541 100644<br>> --- a/meson.build<br>> +++ b/meson.build<br>> @@ -431,15 +431,13 @@ if not get_option('iconv').disabled()<br>> libiconv = cc.find_library('iconv',<br>> required: false,<br>> static: enable_static)<br>> - if libiconv.found()<br>> - if cc.links('''<br>> - #include <iconv.h><br>> - int main(void) {<br>> - iconv_t conv = iconv_open("WCHAR_T", "UCS-2");<br>> - return conv != (iconv_t) -1;<br>> - }''', dependencies: [libiconv])<br>> - iconv = declare_dependency(dependencies: [libiconv])<br>> - endif<br>> + if cc.links('''<br>> + #include <iconv.h><br>> + int main(void) {<br>> + iconv_t conv = iconv_open("WCHAR_T", "UCS-2");<br>> + return conv != (iconv_t) -1;<br>> + }''', dependencies: [libiconv])<br>> + iconv = declare_dependency(dependencies: [libiconv])<br>> endif<br>> endif<br>> if get_option('iconv').enabled() and not iconv.found()<br>> --<br>> 2.28.0<br>><br>Reviewed-by: Yonggang Luo<<a href="mailto:brogers@suse.com" target="_blank">l</a><a href="mailto:uoyonggang@gmail.com">uoyonggang@gmail.com</a>><br><br>--<br> 此致<br>礼<br>罗勇刚<br>Yours<br> sincerely,<br>Yonggang Luo</div>
On 10/15/20 6:33 AM, 罗勇刚(Yonggang Luo) wrote: > > > On Thu, Oct 15, 2020 at 6:19 AM Bruce Rogers <brogers@suse.com > <mailto:brogers@suse.com>> wrote: > > > > It isn't necessarily the case that use of iconv requires an additional > > library. For that reason we shouldn't conditionalize iconv detection on > > libiconv.found. > > > > Fixes: 5285e593c33 (configure: Fixes ncursesw detection under > msys2/mingw by convert them to meson) > > > > Signed-off-by: Bruce Rogers <brogers@suse.com <mailto:brogers@suse.com>> > > --- > > meson.build | 16 +++++++--------- > > 1 file changed, 7 insertions(+), 9 deletions(-) > > > > diff --git a/meson.build b/meson.build > > index 1a4a482492..84c8ec9541 100644 > > --- a/meson.build > > +++ b/meson.build > > @@ -431,15 +431,13 @@ if not get_option('iconv').disabled() > > libiconv = cc.find_library('iconv', > > required: false, > > static: enable_static) > > - if libiconv.found() > > - if cc.links(''' > > - #include <iconv.h> > > - int main(void) { > > - iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > > - return conv != (iconv_t) -1; > > - }''', dependencies: [libiconv]) > > - iconv = declare_dependency(dependencies: [libiconv]) > > - endif > > + if cc.links(''' > > + #include <iconv.h> > > + int main(void) { > > + iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > > + return conv != (iconv_t) -1; > > + }''', dependencies: [libiconv]) > > + iconv = declare_dependency(dependencies: [libiconv]) > > endif > > endif > > if get_option('iconv').enabled() and not iconv.found() > > -- > > 2.28.0 > > > Reviewed-by: Yonggang Luo<l > <mailto:brogers@suse.com>uoyonggang@gmail.com <mailto:uoyonggang@gmail.com>> Something is certainly wrong here =) I suppose you meant: Reviewed-by:Yonggang Luo <luoyonggang@gmail.com> > > -- > 此致 > 礼 > 罗勇刚 > Yours > sincerely, > Yonggang Luo
On 15/10/20 00:19, Bruce Rogers wrote: > It isn't necessarily the case that use of iconv requires an additional > library. For that reason we shouldn't conditionalize iconv detection on > libiconv.found. > > Fixes: 5285e593c33 (configure: Fixes ncursesw detection under msys2/mingw by convert them to meson) > > Signed-off-by: Bruce Rogers <brogers@suse.com> > --- > meson.build | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/meson.build b/meson.build > index 1a4a482492..84c8ec9541 100644 > --- a/meson.build > +++ b/meson.build > @@ -431,15 +431,13 @@ if not get_option('iconv').disabled() > libiconv = cc.find_library('iconv', > required: false, > static: enable_static) > - if libiconv.found() > - if cc.links(''' > - #include <iconv.h> > - int main(void) { > - iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > - return conv != (iconv_t) -1; > - }''', dependencies: [libiconv]) > - iconv = declare_dependency(dependencies: [libiconv]) > - endif > + if cc.links(''' > + #include <iconv.h> > + int main(void) { > + iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); > + return conv != (iconv_t) -1; > + }''', dependencies: [libiconv]) > + iconv = declare_dependency(dependencies: [libiconv]) > endif > endif > if get_option('iconv').enabled() and not iconv.found() > Queued, thanks. Paolo
diff --git a/meson.build b/meson.build index 1a4a482492..84c8ec9541 100644 --- a/meson.build +++ b/meson.build @@ -431,15 +431,13 @@ if not get_option('iconv').disabled() libiconv = cc.find_library('iconv', required: false, static: enable_static) - if libiconv.found() - if cc.links(''' - #include <iconv.h> - int main(void) { - iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); - return conv != (iconv_t) -1; - }''', dependencies: [libiconv]) - iconv = declare_dependency(dependencies: [libiconv]) - endif + if cc.links(''' + #include <iconv.h> + int main(void) { + iconv_t conv = iconv_open("WCHAR_T", "UCS-2"); + return conv != (iconv_t) -1; + }''', dependencies: [libiconv]) + iconv = declare_dependency(dependencies: [libiconv]) endif endif if get_option('iconv').enabled() and not iconv.found()
It isn't necessarily the case that use of iconv requires an additional library. For that reason we shouldn't conditionalize iconv detection on libiconv.found. Fixes: 5285e593c33 (configure: Fixes ncursesw detection under msys2/mingw by convert them to meson) Signed-off-by: Bruce Rogers <brogers@suse.com> --- meson.build | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-)