build.xml patch

From: mike beachy <beachy(at)marketboy(dot)com>
To: pgsql-jdbc(at)postgresql(dot)org
Subject: build.xml patch
Date: 2002-10-02 16:02:36
Message-ID: 20021002120235.A52128@cloud9.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

I've included a small patch for build.xml.

The scenario that it fixes:

Knowing that the dev code has a lot of bug fixes not present in the
standard releases, J Random User checks out cvs code, goes directly to
src/interfaces/jdbc and types 'ant'. User then gets compile errors due
to ${major} and ${minor} not being filtered during the creation of
Driver.java. After reading the README (a bit out of order, but who can
say they haven't done similar things) he then tries configuring and
running make. Unfortunately, this also fails because once Driver.java is
created, it's never overwritten.

I'm not sure that overwriting Driver.java is a good thing, but checking
for defined major and minor definitely is.

-mike

Index: build.xml
===================================================================
RCS file: /projects/cvsroot/pgsql-server/src/interfaces/jdbc/build.xml,v
retrieving revision 1.29
diff -c -r1.29 build.xml
*** build.xml 2002/09/25 07:01:30 1.29
--- build.xml 2002/09/27 21:19:17
***************
*** 156,165 ****
--- 156,169 ----
<filter token="JDBCCONNECTCLASS" value="${connectclass}" />
<filter token="DEF_PGPORT" value="${def_pgport}" />

+ <fail unless="major" message="'major' undefined. Please follow the directions in README."/>
+ <fail unless="minor" message="'minor' undefined. Please follow the directions in README."/>
+
<!-- Put a check for the current version here -->

<!-- now copy and filter the file -->
<copy file="${package}/Driver.java.in"
+ overwrite="true"
tofile="${package}/Driver.java"
filtering="yes" />

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Barry Lind 2002-10-02 16:27:31 Re: Huh? ErrorCode/SQLstate are null?
Previous Message beh 2002-10-02 13:23:02 Huh? ErrorCode/SQLstate are null?