Skip to content

Commit 15c540f

Browse files
committed
Merge branch 'dp/i18n-libcharset'
* dp/i18n-libcharset: Makefile: introduce CHARSET_LIB to link with -lcharset
2 parents d31f378 + b522528 commit 15c540f

File tree

3 files changed

+16
-0
lines changed

3 files changed

+16
-0
lines changed

Makefile

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,10 @@ all::
5656
# FreeBSD can use either, but MinGW and some others need to use
5757
# libcharset.h's locale_charset() instead.
5858
#
59+
# Define CHARSET_LIB to you need to link with library other than -liconv to
60+
# use locale_charset() function. On some platforms this needs to set to
61+
# -lcharset
62+
#
5963
# Define LIBC_CONTAINS_LIBINTL if your gettext implementation doesn't
6064
# need -lintl when linking.
6165
#
@@ -1701,6 +1705,7 @@ endif
17011705

17021706
ifdef HAVE_LIBCHARSET_H
17031707
BASIC_CFLAGS += -DHAVE_LIBCHARSET_H
1708+
EXTLIBS += $(CHARSET_LIB)
17041709
endif
17051710

17061711
ifdef HAVE_DEV_TTY

config.mak.in

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,3 +74,4 @@ SNPRINTF_RETURNS_BOGUS=@SNPRINTF_RETURNS_BOGUS@
7474
NO_PTHREADS=@NO_PTHREADS@
7575
PTHREAD_CFLAGS=@PTHREAD_CFLAGS@
7676
PTHREAD_LIBS=@PTHREAD_LIBS@
77+
CHARSET_LIB=@CHARSET_LIB@

configure.ac

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -836,6 +836,16 @@ AC_CHECK_HEADER([libcharset.h],
836836
[HAVE_LIBCHARSET_H=YesPlease],
837837
[HAVE_LIBCHARSET_H=])
838838
AC_SUBST(HAVE_LIBCHARSET_H)
839+
# Define CHARSET_LIB if libiconv does not export the locale_charset symbol
840+
# and libcharset does
841+
CHARSET_LIB=
842+
AC_CHECK_LIB([iconv], [locale_charset],
843+
[],
844+
[AC_CHECK_LIB([charset], [locale_charset],
845+
[CHARSET_LIB=-lcharset])
846+
]
847+
)
848+
AC_SUBST(CHARSET_LIB)
839849
#
840850
# Define NO_STRCASESTR if you don't have strcasestr.
841851
GIT_CHECK_FUNC(strcasestr,

0 commit comments

Comments
 (0)