forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mathieu Champlon <m.champ...@free.fr>
Subject Re: [jira] Closed: (FOR-913) failure when ${project.home}/forrest.properties does not exist
Date Thu, 03 Aug 2006 03:57:35 GMT
>
> Ross Gardler closed FOR-913.
> ----------------------------
>
>     Resolution: Invalid
>
> forrest.properties has nothing to do with ant. It provides configuration values for forrest.
>
> A forrest.properties file is required.
>   
Hello !

First of all sorry for having added an issue when I should probably have 
taken this to the mailing-list first...

What I was trying to explain is that a forrest.properties file seems 
indeed required while its content isn't.
If you seed a new project and remove everything from the 
forrest.properties file, it builds fine (using default configuration).
But if you remove the (now empty) file, the build breaks.

In the context of several projects with an automated/centralized build 
process, it isn't very convenient to maintain a forrest.properties file 
for each project.
For example if the proxy configuration must be changed, it's much easier 
to change it in one place rather than going through every project.
The feature is already there in forrest : for instance a (centralized) 
script can simply pass custom configuration values to 'forrest' using -D 
options.
If the whole build process is already scripted, it makes sense to 
integrate forrest in it, and to try and remove configuration 
duplications, thus ending with obsolete forrest.properties files.

I understand that it might not be considered a bug, especially with such 
an easy workaround (an empty forrest.properties file).
Maybe this could be a feature request ?
If cannot think of any drawback but if you prefer that the 
${project.home}/forrest.properties file should really be required and 
must not be made optional, then so be it...

By the way the patch is really simple, involving only a couple of java 
lines in forrest.

Thanks.

MAT.

Mime
View raw message