[Building Sakai] WARNs about placeholder.properties

[Anthony Whyte]

The default K1 sakai-configuration.xml looks for several local  
properties files besides kernel.properties and the config project's  
default.sakai.properties files.  I believe the warnings are generated  
by the Spring framework's property factory bean when properties files  
are not located.

Besides placeholder.properties other WARN messages are typically  
generated for local.properties and security.properties.  What is your  
view on these properties file references?

Anth



K1 component-manager/src/main/bundle/org/sakaiproject/config/sakai- 
configuration.xml

[excerpt]

<!--
		The "${sakai.home}/sakai-configuration.xml" file can override this  
bean to change which properties
		files are read, or to plug a DB-backed or JMX-backed properties  
factory bean into the "properties"
		slot.
		
		However, if you do overwrite the SakaiProperties bean, make sure to  
include the "parent" attribute!
	-->
	<bean id="org.sakaiproject.component.SakaiProperties"  
parent="org.sakaiproject.component.DefaultSakaiProperties">
		<property name="locations">
			<list merge="true">
				<value>file:${sakai.home}placeholder.properties</value>
				<value>file:${sakai.home}sakai.properties</value>
				<value>file:${sakai.home}local.properties</value>
				<value>file:${sakai.security}security.properties</value>
			</list>
		</property>
	</bean>



On Mar 11, 2009, at 10:06 PM, Ray Davis wrote:

> 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
>>
>
> _______________________________________________
> sakai-dev mailing list
> sakai-dev at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>
> TO UNSUBSCRIBE: send email to sakai-dev- 
> unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"
>
>