commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jon Scott Stevens <>
Subject Re: [PATCH] beanutils build system
Date Mon, 25 Mar 2002 00:45:25 GMT
on 3/22/02 9:46 PM, "Jeff Turner" <> wrote:

> Hi Jon,
> Would you be happy (ie, would it build on your system) if we applied the
> patch with the following modifications?
> 1) Source ${basedir}/ before
> ${user.home}/
> Many people have a ~/ with default values, which only
> needs to be overridden by ./ for unusual cases. Changing
> the order breaks these people's setups. So I'm proposing that the order
> be:
> <property file="${basedir}/" />
> <property file="${user.home}/" />
> <property file="${user.home}/" />
> <property file="${basedir}/" />

-1. The order that I have it in is done on purpose and has worked well for a
long time for Scarab, Turbine, Lucene, etc. The point is that the
~/ is like a globally defined set of props.

I don't see how changing the order would break people's setups.

> 2) Not delete Ie, will
> default the values to ${lib.repo}/.., but people without a jar
> repository can still 'cp' and
> have things working.


If anything you should do:


> I quite like the idea. Keeps build.xml clean, and
> you don't lose your place in build.xml when looking up the value of a
> property.

The whole point is that you don't have to look at build.xml to change
properties. Much like you shouldn't have to look at a ./configure script.

> If we're all happy with this, the change should be done to all projects,
> not just beanutils (yes I volunteer).

I have already started doing that for all of the projects that Scarab has a
dependency on (which is quite a lot of projects at this point...31 .jar



To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message