[commons-dbcp] Does not build without jndi

[jean-frederic clere <jf...@fujitsu-siemens.com>]

Hi,

I have fixed a problem in commons-dbcp: It does not build with jndi (java1.2).

The exclude of BasicDataSourceFactory does not work because the real path is 
wrong: an extra org is needed in it. 
(org/org/apache/commons/dbcp/BasicDataSourceFactory.java).

A fix for it and a little cleaning of build.properties.sample and adding 
jndi.jar property is what is in attached patch. Please check it and commit it.

Other things:
what does the link:
<link href="http://java.sun.com/products/jdk/1.3/docs/api"/>
Is for? Does commons-dbcp realy needs 1.3?

Do we need sax2.jar?

I have removed these in the patch.


Cheers

Jean-frederic

Re: [commons-dbcp] Does not build without jndi

[Glenn Nielsen <gl...@voyager.apg.more.net>]

Committed to CVS.

Thanks

jean-frederic clere wrote:
> 
> Hi,
> 
> I have fixed a problem in commons-dbcp: It does not build with jndi (java1.2).
> 
> The exclude of BasicDataSourceFactory does not work because the real path is
> wrong: an extra org is needed in it.
> (org/org/apache/commons/dbcp/BasicDataSourceFactory.java).
> 
> A fix for it and a little cleaning of build.properties.sample and adding
> jndi.jar property is what is in attached patch. Please check it and commit it.
> 
> Other things:
> what does the link:
> <link href="http://java.sun.com/products/jdk/1.3/docs/api"/>
> Is for? Does commons-dbcp realy needs 1.3?
> 
> Do we need sax2.jar?
> 
> I have removed these in the patch.
> 
> Cheers
> 
> Jean-frederic
> 
>   ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>                 Name: patch.txt
>    patch.txt    Type: Plain Text (text/plain)
>             Encoding: 7bit
> 
>    Part 1.3Type: Plain Text (text/plain)

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>