Sorry, forgot the patch and the ChangeLog

2011-05-08  Doug Kwan  <dougk...@google.com>

        * configure.ac: Propagate LDFLAGS_FOR_TARGET.
        * configure: Regenerated.
        * Makefile.tpl (LDFLAGS_FOR_TARGET): Use LDFLAGS_FOR_TARGET
        value from configure.
        * Makefile.in: Regenerated.


On Sun, May 8, 2011 at 7:31 PM, Doug Kwan (關振德) <dougk...@google.com> wrote:
> Hi,
>
>    The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to
> build.  This patch does that.  The patch was tested by bootstrapping
> natively on x86_64 linux.  Do I also need to submit this to binutils
> as well?
>
> -Doug
>
Index: configure.ac
===================================================================
--- configure.ac        (revision 173552)
+++ configure.ac        (working copy)
@@ -2110,6 +2110,8 @@ if test "x$CXXFLAGS_FOR_TARGET" = x; then
 fi
 AC_SUBST(CXXFLAGS_FOR_TARGET)
 
+AC_SUBST(LDFLAGS_FOR_TARGET)
+
 # Handle --with-headers=XXX.  If the value is not "yes", the contents of
 # the named directory are copied to $(tooldir)/sys-include.
 if test x"${with_headers}" != x && test x"${with_headers}" != xno ; then
Index: Makefile.tpl
===================================================================
--- Makefile.tpl        (revision 173552)
+++ Makefile.tpl        (working copy)
@@ -494,7 +494,7 @@ CXXFLAGS_FOR_TARGET = @CXXFLAGS_FOR_TARGET@
 
 LIBCFLAGS_FOR_TARGET = $(CFLAGS_FOR_TARGET)
 LIBCXXFLAGS_FOR_TARGET = $(CXXFLAGS_FOR_TARGET) -fno-implicit-templates
-LDFLAGS_FOR_TARGET = 
+LDFLAGS_FOR_TARGET = @LDFLAGS_FOR_TARGET@
 GOCFLAGS_FOR_TARGET = -O2 -g
 
 FLAGS_FOR_TARGET = @FLAGS_FOR_TARGET@

Reply via email to