On Thu, 16 Aug 2001 21:06, Charles Benett wrote: > Peter Donald wrote: > > On Thu, 16 Aug 2001 19:19, Paul Hammant wrote: > > > Trying to reapply this patch (as per invitation to commit it myself) > > > and it does not work. Depite trying numerous combinations of the > > > <exclude> for org/apache/log/output/db/**, it never excludes the > > > package from compilation !!?! > > > > > > Can somone else test this to make sure I am not going insane.. > > > > Try adding > > <exclude name="org/apache/log/output/test/DBTargetTestlet.java" > > unless="javax.sql.present"/> > > > > The compiler is probably trying to include the classes because they are > > referenced from that file. > > Done. But there's still an issue with junit - build scripts aren't > picking the jar.
hmm .. seems to work fine here. Picks up junit from ../jakarta-avalon/tools/lib/junit-3.7.jar > > Charles > > > > - Paul > > > > > > Paul Hammant wrote: > > > > Compiling Logkit is fails if you don't have javax.sql and javax.jms > > > > in lib. > > > > > > > > Patch follows. > > > > > > > > - PH > > > > > > > > C:/Apache-CVS/cvs.exe diff -r HEAD build.xml > > > > Index: build.xml > > > > =================================================================== > > > > RCS file: /home/cvspublic/jakarta-avalon-logkit/build.xml,v > > > > retrieving revision 1.38 > > > > diff -r1.38 build.xml > > > > 155a156,163 > > > > > > > > > <available property="javax.sql.present" > > > > > classname="javax.sql.DataSource" > > > > > classpathref="project.class.path" /> > > > > > > > > > > <available property="javax.jms.present" > > > > > classname="javax.jms.TopicConnection" > > > > > classpathref="project.class.path" /> > > > > > > > > 167a176,179 > > > > > > > > > <exclude name="org/apache/log/output/db/**" > > > > > unless="javax.sql.present"/> > > > > > > > > > <exclude name="org/apache/log/output/jms/**" > > > > > > > > > unless="javax.jms.present"/> > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > -- > > Cheers, > > > > Pete > > > > *-----------------------------------------------------* > > * "Faced with the choice between changing one's mind, * > > * and proving that there is no need to do so - almost * > > * everyone gets busy on the proof." * > > * - John Kenneth Galbraith * > > *-----------------------------------------------------* > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] -- Cheers, Pete *-----------------------------------------------------* * "Faced with the choice between changing one's mind, * * and proving that there is no need to do so - almost * * everyone gets busy on the proof." * * - John Kenneth Galbraith * *-----------------------------------------------------* --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
