[glib: 2/4] gconvert: Don't differentiate between GNU iconv and macOS iconv
- From: Nirbheek Chauhan <nirbheekc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [glib: 2/4] gconvert: Don't differentiate between GNU iconv and macOS iconv
- Date: Fri, 12 Apr 2019 06:03:10 +0000 (UTC)
commit ace32cd8e2a012f33ebd780e53631b5a2cd03c23
Author: Nirbheek Chauhan <nirbheek centricular com>
Date: Thu Apr 4 05:12:35 2019 +0530
gconvert: Don't differentiate between GNU iconv and macOS iconv
Both provide iconv_open, and in fact, we weren't using the difference
anywhere in glib at all.
glib/gconvert.c | 8 --------
meson.build | 12 +-----------
meson_options.txt | 4 ++--
3 files changed, 3 insertions(+), 21 deletions(-)
---
diff --git a/glib/gconvert.c b/glib/gconvert.c
index 707867951..c5857df1c 100644
--- a/glib/gconvert.c
+++ b/glib/gconvert.c
@@ -52,14 +52,6 @@
#include "glibintl.h"
-#if defined(USE_LIBICONV_GNU) && !defined (_LIBICONV_H)
-#error GNU libiconv in use but included iconv.h not from libiconv
-#endif
-#if !defined(USE_LIBICONV_GNU) && defined (_LIBICONV_H) \
- && !defined (__APPLE_CC__) && !defined (__LP_64__)
-#error GNU libiconv not in use but included iconv.h is from libiconv
-#endif
-
/**
* SECTION:conversions
diff --git a/meson.build b/meson.build
index 0ec80c177..85433fb3b 100644
--- a/meson.build
+++ b/meson.build
@@ -1690,9 +1690,6 @@ glibconfig_conf.set10('G_HAVE_GROWING_STACK', growing_stack)
# Tests for iconv
#
-# USE_LIBICONV_GNU: Using GNU libiconv
-# USE_LIBICONV_NATIVE: Using a native impl of iconv in a separate library
-#
# We should never use the MinGW C library's iconv because it may not be
# available in the actual runtime environment. On Windows, we always use
# the built-in implementation
@@ -1701,7 +1698,6 @@ if host_system == 'windows'
libiconv = []
# We have a #include "win_iconv.c" in gconvert.c on Windows, so we don't need
# any external library for it
- glib_conf.set('USE_LIBICONV_NATIVE', true)
if iconv_opt != 'auto'
warning('-Diconv was set to @0@, which was ignored')
endif
@@ -1711,13 +1707,7 @@ else
libiconv = []
found_iconv = true
endif
- if not found_iconv and ['auto', 'native'].contains(iconv_opt) and cc.has_header_symbol('iconv.h',
'iconv_open')
- glib_conf.set('USE_LIBICONV_NATIVE', true)
- libiconv = [cc.find_library('iconv')]
- found_iconv = true
- endif
- if not found_iconv and ['auto', 'gnu'].contains(iconv_opt) and cc.has_header_symbol('iconv.h',
'libiconv_open')
- glib_conf.set('USE_LIBICONV_GNU', true)
+ if not found_iconv and ['auto', 'external'].contains(iconv_opt) and cc.has_header_symbol('iconv.h',
'iconv_open')
libiconv = [cc.find_library('iconv')]
found_iconv = true
endif
diff --git a/meson_options.txt b/meson_options.txt
index 0841f6bfe..2c831e37e 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -5,9 +5,9 @@ option('runtime_libdir',
option('iconv',
type : 'combo',
- choices : ['auto', 'libc', 'native', 'gnu'],
+ choices : ['auto', 'libc', 'external'],
value : 'auto',
- description : 'iconv implementation to use (\'libc\' = \'Part of the C stdlib\'; \'gnu\' = \'GNU\'s
iconv\'; \'native\' = \'A separate iconv\'; \'auto\' = \'Auto-detect which iconv is available\')')
+ description : 'iconv implementation to use (\'libc\' = \'Part of the C library\'; \'external\' =
\'External libiconv\'; \'auto\' = \'Auto-detect which iconv is available\')')
option('charsetalias_dir',
type : 'string',
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]