Re: [PATCHES] the build

From: Nic Ferrier <nferrier(at)tapsellferrier(dot)co(dot)uk>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Barry Lind <blind(at)xythos(dot)com>, <pgsql-jdbc(at)postgresql(dot)org>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [PATCHES] the build
Date: 2003-04-17 18:51:17
Message-ID: 87d6jlklei.fsf@pooh-sticks-bridge.tapsellferrier.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc pgsql-patches

Peter Eisentraut <peter_e(at)gmx(dot)net> writes:

> Nic Ferrier writes:
>
> > The JAVAC variable in the Makefile.in is therefore designed to allow
> > overriding by the user when running the make, thus:
> >
> > ./configure --with-java ; make JAVAC=gcj
> >
> > will build the jdbc library with gcj.
>
> And then if you make a change and run make again but forget to specify the
> same Java compiler, you get a mess. Specifying variables on the make
> command line has been rejected as error-prone a long time ago.

I agree. If I ruled the world everything would be autotools.

The proposed patch is a hack. But it is a hack with a purpose, the
desire to do this is, after all, not exactly universal right now.

As I'm sure you know, a better solution would be to invent some kind
of autoconf macro to achieve the same purpose, ie: the specification
of the compiler to Ant. If it was done in autoconf then overrides
would be possible at ./configure time.

As Barry has pointed out, what we want the JAVAC variable (we'll
probably change the variable's name) to contain is not the filename
of a compiler executable but a symbolic name to be recognized by Ant;
so none of the existing autotools Java macros is suitable.

And I don't have the time _right_ now to write such a macro. I might
in the future.

So, is the hack method totally unacceptable do you think? If not I'll
submit it as part of a patch for the java build that I'll be doing
this evening.

If it is unacceptable then we'll have to wait for somebody to write a
macro before compiling with gcj becomes easier (and I think that
would be a pity).

Nic

In response to

Browse pgsql-jdbc by date

  From Date Subject
Next Message Daniel Serodio 2003-04-17 20:51:11 Re: Javadoc for Postgresql JDBC driver?
Previous Message Cris 2003-04-17 18:10:52 Table as a column

Browse pgsql-patches by date

  From Date Subject
Next Message Sean Chittenden 2003-04-17 19:48:03 Re: [HACKERS] Are we losing momentum?
Previous Message Bruce Momjian 2003-04-17 18:44:26 Re: Win32 defines