Skip to content

Commit 346c157

Browse files
author
ripley
committed
revert r87916 as it does not build on Windows
git-svn-id: https://svn.r-project.org/R/trunk@87928 00db46b3-68df-0310-9c12-caf00c1e9a41
1 parent a87eca9 commit 346c157

File tree

6 files changed

+108
-374
lines changed

6 files changed

+108
-374
lines changed

configure

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -667,7 +667,7 @@ BUILD_LIBINTL_FALSE
667667
BUILD_LIBINTL_TRUE
668668
USE_NLS_FALSE
669669
USE_NLS_TRUE
670-
XTRA_INTL_CPPFLAGS
670+
LIBINTL_PKGS
671671
POSUB
672672
LTLIBINTL
673673
LIBINTL
@@ -63696,7 +63696,9 @@ printf "%s\n" "#define HAVE_DCGETTEXT 1" >>confdefs.h
6369663696

6369763697

6369863698
if test -n "$INTL_MACOSX_LIBS"; then
63699-
XTRA_INTL_CPPFLAGS=-I/System/Library/Frameworks/CoreFoundation.framework/Headers
63699+
LIBINTL_PKGS =
63700+
else
63701+
LIBINTL_PKGS = $(LIBINTL)
6370063702
fi
6370163703
echo "Finished configuring src/extra/intl directory"
6370263704
echo

configure.ac

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2926,14 +2926,21 @@ if test "${USE_NLS}" = "yes"; then
29262926
AM_GNU_GETTEXT_VERSION(0.16.1)
29272927
AM_GNU_GETTEXT([no-libtool], [need-ngettext], [../extra/intl])
29282928
if test -n "$INTL_MACOSX_LIBS"; then
2929-
XTRA_INTL_CPPFLAGS=-I/System/Library/Frameworks/CoreFoundation.framework/Headers
2929+
dnl This is no longer valid (if it ever was)
2930+
dnl XTRA_INTL_CPPFLAGS=-I/System/Library/Frameworks/CoreFoundation.framework/Headers
2931+
dnl On a macOS build which uses static libraries, libintk_dgettext
2932+
dnl etc are provided by libR.dylib
2933+
LIBINTL_PKGS =
2934+
else
2935+
LIBINTL_PKGS = $(LIBINTL)
29302936
fi
29312937
echo "Finished configuring src/extra/intl directory"
29322938
echo
29332939
else
29342940
USE_INCLUDED_LIBINTL=no
29352941
fi
2936-
AC_SUBST(XTRA_INTL_CPPFLAGS)
2942+
dnl AC_SUBST(XTRA_INTL_CPPFLAGS)
2943+
AC_SUBST(LIBINTL_PKGS)
29372944

29382945
AM_CONDITIONAL(USE_NLS, [test "x${USE_NLS}" = xyes])
29392946
AM_CONDITIONAL(BUILD_LIBINTL, [test "x${USE_INCLUDED_LIBINTL}" = xyes])

etc/Makeconf.in

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,15 @@ JAVA_LIBS = @JAVA_LIBS0@
9292
JAVA_LD_LIBRARY_PATH = @JAVA_LD_LIBRARY_PATH@
9393
LAPACK_LIBS = @LAPACK_LIBS@
9494
LDFLAGS = @LDFLAGS@
95-
## we only need this is if it is external, as otherwise link to R
96-
LIBINTL= @LIBINTL@
95+
## We only need this is if it is external, as otherwise link to R
96+
## Oo macOS, where libintl is static and so compiled in, can still link to R
97+
LIBINTL = @LIBINTL_PKGS@
9798
LIBM = @LIBM@
9899
LIBR0 = @LIBR0@
99100
LIBR1 = @LIBR1@
100101
LIBR = @LIBR0@ @LIBR1@
102+
## FIXME: LIBS is approporiate for building R,
103+
## but not for packages if static libse are used.
101104
LIBS = @LIBS@
102105
## needed by R CMD config
103106
LIBnn = @LIBnn@

0 commit comments

Comments
 (0)