Skip site navigation (1) Skip section navigation (2)

Re: [PATCHES] the build

From: Barry Lind <blind(at)xythos(dot)com>
To: Nic Ferrier <nferrier(at)tapsellferrier(dot)co(dot)uk>
Cc: pgsql-jdbc(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [PATCHES] the build
Date: 2003-04-16 04:58:37
Message-ID: 3E9CE2FD.60406@xythos.com (view raw or flat)
Thread:
Lists: pgsql-jdbcpgsql-patches
Nic,

I don't understand this patch.  From the ant doc I see the following:

 >>It is possible to use different compilers. This can be specified by
 >>either setting the global build.compiler property, which will affect
 >>all <javac> tasks throughout the build, or by setting the compiler
 >>attribute, specific to the current <javac> task. Valid values for
 >>either the build.compiler property or the compiler attribute are:
 >>
 >>    * classic (the standard compiler of JDK 1.1/1.2) – javac1.1 and
 >>javac1.2 can be used as aliases.
 >>    * modern (the standard compiler of JDK 1.3/1.4) – javac1.3 and
 >>javac1.4 can be used as aliases.
 >>    * jikes (the Jikes compiler).
 >>    * jvc (the Command-Line Compiler from Microsoft's SDK for Java /
 >>Visual J++) – microsoft can be used as an alias.
 >>    * kjc (the kopi compiler).
 >>    * gcj (the gcj compiler from gcc).
 >>    * sj (Symantec java compiler) – symantec can be used as an alias.
 >>    * extJavac (run either modern or classic in a JVM of its own).

I don't see 'javac' as a valid option for this parameter.  So I don't 
understand why your patch is attempting to set that value.

thanks,
--Barry



Nic Ferrier wrote:
> Barry Lind <blind(at)xythos(dot)com> writes:
> 
> 
>>Nic,
>>
>>Unfortunately I am not well versed in either ant or autoconf configury.
>>  I personally build the different versions by changing the JAVA_HOME
>>environment variable to point to the jdk I wish to use for building.
>>Ant then picks it up and does the 'right' thing.  I personally haven't
>>tried building with gcj yet and I don't know if ant has support for gcj.
>>
>>In general I would say that the driver has moved away from autoconf/make
>>and towards ant as the build mechanism.  The role make now plays is
>>essentially just the wrapper around ant that ties into the overall
>>postgres build mechanism.
> 
> 
> Ok, first I should say, I'm a sometime GCJ hacker. I'm not heavily
> involved with it, but I am heavily involved with GNU and so I use it a
> lot.
> 
> So I have an interest here.
> 
> 
> The next thing is: GCJ doesn't have a concept of JAVA_HOME. I don't
> think jikes or kaffe or kopi has any such concept either.
> 
> So that means that JAVAC detection should be either via a normal
> autoconf tool or something simpler, like the attached patch.
> 
> 
> 
> But that doesn't solve the issue of version selection. I think this
> is a fundamental build issue: how do you choose which version of the
> driver to build?
> 
> As I say, my view is that we should test the target java's version,
> and not the version that is being used to run ant.
> 
> Would you (or anyone else) have any objection in principle to that?
> 
> 
> Nic
> 
> 
> 
> 
> ------------------------------------------------------------------------
> 
> Index: src/Makefile.global.in
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/src/Makefile.global.in,v
> retrieving revision 1.161
> diff -p -u -r1.161 Makefile.global.in
> --- src/Makefile.global.in	2003/04/04 20:42:11	1.161
> +++ src/Makefile.global.in	2003/04/15 16:59:41
> @@ -177,6 +177,9 @@ ifeq ($(GCC), yes)
>    CFLAGS += -Wall -Wmissing-prototypes -Wmissing-declarations
>  endif
>  
> +JAVAC	= javac
> +
> +
>  # Kind-of compilers
>  
>  YACC = @YACC@
> Index: src/interfaces/jdbc/Makefile
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/src/interfaces/jdbc/Makefile,v
> retrieving revision 1.38
> diff -p -u -r1.38 Makefile
> --- src/interfaces/jdbc/Makefile	2003/02/12 06:13:04	1.38
> +++ src/interfaces/jdbc/Makefile	2003/04/15 16:59:41
> @@ -22,9 +22,10 @@ build.properties: $(top_builddir)/src/Ma
>  	@echo fullversion=$(VERSION) >> build.properties
>  	@echo def_pgport=$(DEF_PGPORT) >> build.properties
>  	@echo enable_debug=$(enable_debug) >> build.properties
> +	@echo build.compiler=$(JAVAC) >> build.properties
>  
> 
> 
> ------------------------------------------------------------------------
> 
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
> 
> http://www.postgresql.org/docs/faqs/FAQ.html


In response to

Responses

pgsql-patches by date

Next:From: Tatsuo IshiiDate: 2003-04-16 05:43:08
Subject: Re: PostgreSQL vs. MySQL in Japan
Previous:From: Barry LindDate: 2003-04-16 04:43:08
Subject: Re: the build

pgsql-jdbc by date

Next:From: Kris JurkaDate: 2003-04-16 07:20:24
Subject: Re: Index metadata not sorted by ORDINAL_POSITION
Previous:From: Barry LindDate: 2003-04-16 04:43:08
Subject: Re: the build

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group