[Building Sakai] WARNs about placeholder.properties

[Ray Davis]

I should have added that any 2.6 deployments that *do* depend on a 
"placeholder.properties" file can explicitly load it using the same 
sakai-configuration.xml file:

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns="http://www.springframework.org/schema/beans"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns:util="http://www.springframework.org/schema/util"
     xsi:schemaLocation="http://www.springframework.org/schema/beans 
http://www.springframework.org/schema/beans/spring-beans-2.0.xsd
     http://www.springframework.org/schema/util 
http://www.springframework.org/schema/util/spring-util-2.0.xsd">
   <!-- Change the list of local properties files. (The default 
sakai.properties will still be read.) -->
   <bean id="org.sakaiproject.component.SakaiProperties" 
parent="org.sakaiproject.component.DefaultSakaiProperties">
     <property name="locations">
       <list merge="true">
         <value>file:${sakai.home}sakai.properties</value>
         <value>file:${sakai.home}placeholder.properties</value>
       </list>
     </property>
   </bean>
</beans>

Best,
Ray

On 3/11/2009 12:57 PM, Ray Davis wrote:
> +1. The last time I was working in the configuration system, I left the 
> old checks in place just so as to eliminate backwards incompatibility in 
> my changes. Cleaning them out now as a separate step sounds good to me.
> 
> On my local deployments, BTW, I use the new sakai-configuration.xml to 
> override the defaults:
>... snip ...
> 
> Best,
> Ray
> 
> On 3/11/2009 12:46 PM, Seth Theriault wrote:
>> Hello,
>>
>> More thana few people have reported this WARN from 2.6 code:
>>
>> 2009-03-11 15:00:25,149 WARN main org.springframework.beans.factory.config.PropertiesFactoryBean - Could not load properties from URL [file:/spare/sakai-qa/sakai-demo/sakai/placeholder.properties]: /spare/sakai-qa/sakai-demo/sakai/placeholder.properties (No such file or directory)
>>
>> There really isn't a reason for that WARN since the 
>> placeholder.properties file was allegedly deprecated a long time 
>> ago. Specifically, a warning message about its future removal was 
>> in place for almost a year starting in Feb 2007:
>>
>> http://source.sakaiproject.org/viewsvn?view=rev&revision=21066
>>
>> until it was removed just before the move to K1:
>>
>> http://source.sakaiproject.org/viewsvn?view=rev&revision=39741
>>
>> The JIRAs related to r39471 are here:
>> http://jira.sakaiproject.org/jira/browse/SAK-8315
>> http://jira.sakaiproject.org/jira/browse/SAK-12236
>> http://jira.sakaiproject.org/jira/browse/SAK-12237
>>
>> Isn't it about time to put placeholder.properties out of its 
>> misery, since no one "should" be using it anyway?
>>
>> Seth
>