Skip to content

Commit 4c1ca6b

Browse files
author
ripley
committed
revert r87928 to split it up in smaller pieces
git-svn-id: https://svn.r-project.org/R/trunk@87929 00db46b3-68df-0310-9c12-caf00c1e9a41
1 parent 346c157 commit 4c1ca6b

File tree

5 files changed

+11
-23
lines changed

5 files changed

+11
-23
lines changed

configure

Lines changed: 2 additions & 4 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-
LIBINTL_PKGS
670+
XTRA_INTL_CPPFLAGS
671671
POSUB
672672
LTLIBINTL
673673
LIBINTL
@@ -63696,9 +63696,7 @@ printf "%s\n" "#define HAVE_DCGETTEXT 1" >>confdefs.h
6369663696

6369763697

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

configure.ac

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2926,21 +2926,14 @@ 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-
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)
2929+
XTRA_INTL_CPPFLAGS=-I/System/Library/Frameworks/CoreFoundation.framework/Headers
29362930
fi
29372931
echo "Finished configuring src/extra/intl directory"
29382932
echo
29392933
else
29402934
USE_INCLUDED_LIBINTL=no
29412935
fi
2942-
dnl AC_SUBST(XTRA_INTL_CPPFLAGS)
2943-
AC_SUBST(LIBINTL_PKGS)
2936+
AC_SUBST(XTRA_INTL_CPPFLAGS)
29442937

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

etc/Makeconf.in

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -92,15 +92,12 @@ 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-
## Oo macOS, where libintl is static and so compiled in, can still link to R
97-
LIBINTL = @LIBINTL_PKGS@
95+
## we only need this is if it is external, as otherwise link to R
96+
LIBINTL= @LIBINTL@
9897
LIBM = @LIBM@
9998
LIBR0 = @LIBR0@
10099
LIBR1 = @LIBR1@
101100
LIBR = @LIBR0@ @LIBR1@
102-
## FIXME: LIBS is approporiate for building R,
103-
## but not for packages if static libse are used.
104101
LIBS = @LIBS@
105102
## needed by R CMD config
106103
LIBnn = @LIBnn@

src/extra/tzone/Makefile.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ DEPENDS = $(SOURCES:.c=.d)
1818
OBJECTS = $(SOURCES:.c=.o)
1919
@WANT_R_SHLIB_TRUE@ALL_CFLAGS = $(R_XTRA_CFLAGS) $(CPICFLAGS) $(SHLIB_CFLAGS) $(CFLAGS) @C_VISIBILITY@
2020

21-
ALL_CPPFLAGS=$(R_XTRA_CPPFLAGS) -I$(srcdir) -I$(top_srcdir)/src/main $(CPPFLAGS) $(DEFS)
21+
ALL_CPPFLAGS=$(R_XTRA_CPPFLAGS) -I$(srcdir) -I$(top_srcdir)/src/main $(CPPFLAGS) $(DEFS) @XTRA_INTL_CPPFLAGS@
2222

2323
R all: Makefile
2424
@$(MAKE) Makedeps

src/modules/lapack/Lapack.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/*
22
* R : A Computer Language for Statistical Data Analysis
3-
* Copyright (C) 2001--2025 The R Core Team.
3+
* Copyright (C) 2001--2023 The R Core Team.
44
* Copyright (C) 2003--2010 The R Foundation
55
*
66
* This program is free software; you can redistribute it and/or modify
@@ -263,7 +263,7 @@ static SEXP unscramble(const double* imaginary, int n,
263263
/* Real, general case of eigen */
264264
static SEXP La_rg(SEXP x, SEXP only_values)
265265
{
266-
bool vectors, complexValues;
266+
Rboolean vectors, complexValues;
267267
int i, n, lwork, info, *xdims, ov;
268268
double *work, *wR, *wI, *left, *right, *xvals, tmp;
269269
char jobVL[2] = "N", jobVR[2] = "N";
@@ -306,11 +306,11 @@ static SEXP La_rg(SEXP x, SEXP only_values)
306306
if (info != 0)
307307
error(_("error code %d from Lapack routine '%s'"), info, "dgeev");
308308

309-
complexValues = false;
309+
complexValues = FALSE;
310310
for (i = 0; i < n; i++)
311311
/* This test used to be !=0 for R < 2.3.0. This is OK for 0+0i */
312312
if (fabs(wI[i]) > 10 * R_AccuracyInfo.eps * fabs(wR[i])) {
313-
complexValues = true;
313+
complexValues = TRUE;
314314
break;
315315
}
316316
SEXP ret = PROTECT(allocVector(VECSXP, 2));

0 commit comments

Comments
 (0)