lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jon Scott Stevens <...@latchkey.com>
Subject Re: cvs commit: jakarta-lucene build.xml
Date Wed, 27 Feb 2002 20:09:03 GMT
on 2/27/02 10:08 AM, "Dmitry Serebrennikov" <dmitrys@earthlink.net> wrote:

> As a new and cluless user of Ant, here's my vote:
>   +1 on not requiring editing of build.xml
>   +1 on keeping default properties in build.xml
>   +1 on providing build.properties.sample with comments as to what can
> be overriden and how
>   +1 on *no* default.properties
> 
> I prefer build.properties.sample to be there because it gives me as a
> cluless user a clear idea as to what to do to customize things. If
> simply default.properties file is there, I wouldn't even think that some
> other file called 'build.properties" could be created to customize
> things (even if after reading the build.xml I could figure out that
> there is an include statement or something). I think having
> default.properties file would confuse things a bit more than required.
> 
> The BUILD, COMPILE, or build.readme files can actually explain the fact
> that ant should be used for building, how to customize settings, what
> the ant targets are, how to run ant, and what version of ant is needed.
> 

Those same files can also explain to create a build.properties to override
the default.properties instead of having to rename a file...I think you are
also overlooking the fact that by having defaults in build.xml you need to
also maintain the same defaults in build.properties.sample. That is bad.

Here is the text I use in Scarab's README.txt file to explain things...I'm
sure it could be cleaned up to be more easily understood...

Anyway, I'm happy to continue to discuss this, but I think I have provided a
very strong and compelling case for the reasons why I like build systems to
do things the way that I explained. I also already voted -1 towards having a
build.properties.sample as well as storing default properties in the
build.xml. Are people overlooking that vote now? Yes, I'm more than happy to
also provide a patch and do the work.

Thanks,

-jon

-------------------------------------------------------------------------
| S E T T I N G S                                                       |
-------------------------------------------------------------------------

The Scarab build process depends on having a few properties which are
defined in the build/default.properties. These properties should be set
accordingly *before* you build Scarab. If you would like to customize
these settings, then you should not edit the build/default.properties.

The settings in the build/default.properties are fairly well documented.

Instead, one should create a ~/build.properties and/or a
build/build.properties file and place the properties in those files
which override the properties in the build/default.properties. The build
system will take the first property it can find and use that. It will
first look for the ~/build.properties and then look for the
build/default.properties.

Chances are that you are going to have to define your own database and
mail server properties as well as a few other properties. Please look in
the scarab/build/default.properties for a list of things that you can
define.


--
To unsubscribe, e-mail:   <mailto:lucene-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:lucene-dev-help@jakarta.apache.org>


Mime
View raw message