Berin Loritsch wrote:

>We should not be using ant.properties.sample if we are depending
>on their contents.  We should have ant.properties in every excalibur
>product package.  The ".sample" is causing issues.
>
-1  People will have subsequent CVS update probs then.

>
>An alternative is to check for the existence of the file and copy
>the ant.properties.sample to ant.properties and load it.  That way
>we always have the property files we need to build excalibur, but
>we don't run into accidently checking in changes that only work on
>one machine.
>
+1

>
>>-----Original Message-----
>>From: Paul Hammant [mailto:[EMAIL PROTECTED]] 
>>Sent: Friday, March 29, 2002 4:41 AM
>>To: Avalon Development
>>Subject: ant.properties
>>
>>
>>Folks,
>>
>>I am finding it difficult to complete the setup necessary to build 
>>excalibur from scratch.  And I am not a newbie.
>>
>>Given that ant.properties is reuired in a number of places 
>>(or at least 
>>the properties they define are), I though the following may 
>>be useful if 
>>added to pertinent places of the build files :
>>
>>Index: build.xml 
>>===================================================================
>>RCS file: /home/cvs/jakarta-avalon-excalibur/all/build.xml,v
>>retrieving revision 1.118
>>diff -r1.118 build.xml
>>195c195,199
>><   <target name="check-datasource" depends="check-environment" 
>>if="datasource.present">
>>---
>> >   <target name="check-ant-properties" 
>>unless="excalibur-collections.home">
>> >      <fail message="You need an ant.properties to build 
>>excalibur.  
>>Either one in your home dir, or one the jakarta-avalon-excalibur/all 
>>dir. There is an ant.properties.sample that can be copied for 
>>this need."/>
>> >   </target>
>> >
>> >   <target name="check-datasource" depends="check-environment, 
>>check-ant-properties" if="datasource.present">
>>
>>Note that it is checking for a single property from each 
>>build file and 
>>not the presence of the local build file...
>>
>>Thoughts?
>>
>>- Paul
>>
>>
>>--
>>To unsubscribe, e-mail:   
>><mailto:avalon-dev-> [EMAIL PROTECTED]>
>>For 
>>additional commands, 
>>e-mail: <mailto:[EMAIL PROTECTED]>
>>
>
>
>--
>To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
>For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>
>




--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to