Re: PATCH: PR libjava/32078: Update libtool in classpath

["H. J. Lu" <hjl at lucon dot org>]

On Wed, May 30, 2007 at 02:32:03PM -0500, Peter O'Gorman wrote:
> On Wed, 2007-05-30 at 10:37 -0700, H. J. Lu wrote:
> > On Wed, May 30, 2007 at 11:01:00AM -0600, Tom Tromey wrote:
> > > >>>>> "H.J." == H J Lu <hjl@lucon.org> writes:
> > > 
> > > H.J.> This patch + my libtool.m4 hack work for me on Linux/x86-64 with
> > > H.J.> "make", "make check" and "make install".
> > > 
> > > I suppose someone else will have to review the libtool.m4 patch.
> > > I'd appreciate it if this were done quickly.  libjava hasn't built for
> > > several days now.
> > > 
> > > Your change in classpath is ok.  Thank you.
> > 
> > My change in classpath won't work without my libtool.m4 change. I
> > will check in my libtool.m4 change to enable libjava build again. We
> > can always back it out later when a better solution is found.
> 
> I don't see the need to fork libtool.m4, if you can't get aclocal to
> work (I couldn't either when I tried just now) just put this in
> configure.ac ( or in a separate file and m4_include it):
> 
> m4_pushdef([_LT_LANG_GCJ_CONFIG],
> [

Have you tried it? Does it work? Do you have a patch I can try?

When I tried it, it didn't work and the one in libtool.m4 was used.
I think the order of 2 _LT_LANG_GCJ_CONFIGs is important. I couldn't
get it right.


H.J.