commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brian E. Dunbar" <bdun...@dunbarconsulting.org>
Subject RE: [Configuration]Missing Jars for Build and Bug with ClassPropertiesConfiguration
Date Sun, 22 Jun 2003 02:33:35 GMT
Thanks. I am not fully up to speed on Maven yet.

I will send patches in the future.

Brian

-----Original Message-----
From: EPugh@upstate.com [mailto:EPugh@upstate.com]
Sent: Saturday, June 21, 2003 6:13 PM
To: commons-dev@jakarta.apache.org
Subject: RE: [Configuration]Missing Jars for Build and Bug with
ClassPropertiesConfiguration


Brian,

I committed your fixes.  Instead of editing the maven.xml, I tweaked the
project.xml testResources section instead.  I also added you to the
project.xml file, figure some free advertising for your work was a good
thing!

One more thing, if you can submit patches versus whole files, that is much
easier to apply.

Thanks!
Eric Pugh

-----Original Message-----
From: Brian E. Dunbar [mailto:bdunbar@dunbarconsulting.org]
Sent: Friday, June 20, 2003 6:23 PM
To: Jakarta Commons Developers List
Subject: RE: [Configuration]Missing Jars for Build and Bug with
ClassPropertiesConfiguration


Some of the files appeared to have gotten stripped in transfer. Here are the
missing files.

Brian



-----Original Message-----
From: Brian E. Dunbar [mailto:bdunbar@dunbarconsulting.org]
Sent: Friday, June 20, 2003 5:14 PM
To: Jakarta Commons Developers List
Subject: RE: [Configuration]Missing Jars for Build and Bug with
ClassPropertiesConfiguration


Eric,

I have made the changes and provided a test case as you suggested.

The files I have attached contain the following:

maven.xml-Added copy target to copy the test.properties file to
testClasspath.properties at the root of the classpath.

project.xml-removed the TestClassProperties from being excluded from tests.

TestClassPropertiesConfiguration.java-Added new unit test to test loading
from the classpath in various path formats.

TestBasePropertiesConfiguration.java-Added a test to test properties that
reference another property using the ${property} notation. This was
unrelated to the original changes that I proposed.

test.properties-Added two properties for testing references to previous
properties.

If you put the above changes in place and run maven you can see that it
fails the tests. Place the following files in place and everything should
test OK.

ClassPropertiesConfiguration.java-Changes to first attempt to load the
properties file via the provided class. If that fails it attempts to load
from the classloader.

Let me know if you have any questions,

Brian Dunbar




---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message