On Mon, 2011-07-11 at 13:58 -0700, Ian Lance Taylor wrote: > Eric Botcazou <ebotca...@adacore.com> writes: > > >> If there is an up to date patch, I'm happy to review it if it would > >> help. But perhaps an Ada maintainer would prefer to do the review, I > >> don't know what the usual policy is as I've never approved an Ada patch. > > > > The patch is here: http://gcc.gnu.org/ml/gcc-patches/2011-07/msg00845.html > > Thanks. > > I'm not sure that always setting LINKER=$(CXX) in gnattools/Makefile.in > is correct. It seems to me that that should vary based on > --enable-build-with-cxx, as is done in gcc/Makefile.in. But I don't > know much about building Ada, and perhaps this is right.
Hi, Paolo did suggest something like that: http://gcc.gnu.org/ml/gcc/2009-06/msg00639.html << I think you should rather do "CC=../../xgcc -B../../" \ + "CXX=../../g++ -B../../" \ "CFLAGS=$(CFLAGS) $(WARN_CFLAGS)" \ + "CXXFLAGS=$(CXXFLAGS) $(WARN_CFLAGS)" \ and copy the setting of COMPILER and LINKER from gcc/Makefile.in into gcc/ada/gcc-interface/Makefile.in: ENABLE_BUILD_WITH_CXX = @ENABLE_BUILD_WITH_CXX@ ifneq ($(ENABLE_BUILD_WITH_CXX),yes) COMPILER = $(CC) COMPILER_FLAGS = $(CFLAGS) LINKER = $(CC) LINKER_FLAGS = $(CFLAGS) else COMPILER = $(CXX) COMPILER_FLAGS = $(CXXFLAGS) LINKER = $(CXX) LINKER_FLAGS = $(CXXFLAGS) endif Paolo >> Now I don't know if we intend to support build without CXX in the near future :). Sincerely, Laurent