This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: autoconf 2.50 updates - libjava


>>>>> "Zack" == Zack Weinberg <zackw@stanford.edu> writes:

Zack> Continuing to break things up by directory.  libjava isn't set
Zack> up for cross compilation, which makes life easier (no
Zack> LIB_AC_PROG_CC to deal with).  These are pretty straightforward.

Actually, libjava can be cross-compiled.  I don't do it frequently,
but it has been done in the past, and I know that Anthony has done
some work on that recently.

We don't use LIB_AC_PROG_CC.  Instead we do this:

    dnl This shouldn't be needed, as long as top-level dependencies are
    dnl defined correctly and shared-library paths are set up so that
    dnl execution tests succeed.  FIXME.
    define([AC_PROG_CC_WORKS],[])
    define([AC_PROG_CXX_WORKS],[])

You will also get warnings from configure about not using certain
macros when to cross-compiling.  Those macros are actually only
invoked in the native case.

Does this approach continue to work with 2.50?

Zack> 	* configure.host: Rename AM_RUNTESTFLAGS to GCJ_RUNTESTFLAGS.

This is "wrong" in a certain pedantic sense.  But since we already
violate the convention (we set RUNTESTFLAGS ourselves in
testsuite/Makefile.am) it probably doesn't matter.

Zack> +# This can't be called RUNTESTFLAGS, or automake pukes.
Zack> +# It can't be called AM_RUNTESTFLAGS either, or autoconf 2.50 pukes.
Zack> +# Aaargh.
Zack> +GCJ_RUNTESTFLAGS= 
 
You can use this invocation if you prefer not to change the name.

ifdef([m4_pattern_allow], [m4_pattern_allow([^AM_RUNTESTFLAGS])])

The next automake release will do this automatically.


Anyway, assuming the cross-compilation hack continues to work, then
this patch is ok.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]