You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2006/03/09 15:50:18 UTC
svn commit: r384518 - in
/incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework:
Felix.java SystemBundle.java
Author: rickhall
Date: Thu Mar 9 06:50:16 2006
New Revision: 384518
URL: http://svn.apache.org/viewcvs?rev=384518&view=rev
Log:
Cleaned up the System Bundle's dependency on the framework version constant.
Modified:
incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/Felix.java
incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/SystemBundle.java
Modified: incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/Felix.java
URL: http://svn.apache.org/viewcvs/incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/Felix.java?rev=384518&r1=384517&r2=384518&view=diff
==============================================================================
--- incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/Felix.java (original)
+++ incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/Felix.java Thu Mar 9 06:50:16 2006
@@ -3326,42 +3326,30 @@
m_configMutable.put(
- FelixConstants.FELIX_VERSION_PROPERTY, getVersion() );
+ FelixConstants.FELIX_VERSION_PROPERTY, getFrameworkVersion());
}
-
- private static final String FELIX_VERSION_VALUE;
-
- static
- {
- FELIX_VERSION_VALUE = getVersion0();
- }
-
-
- private static String getVersion0()
+ /**
+ * Read the framework version from the property file.
+ * @return the framework version as a string.
+ **/
+ private static String getFrameworkVersion()
{
// The framework version property.
Properties props = new Properties();
- InputStream in = Felix.class.getResourceAsStream( "Felix.properties" );
+ InputStream in = Felix.class.getResourceAsStream("Felix.properties");
try
{
- props.load( in );
+ props.load(in);
}
- catch ( IOException e )
+ catch (IOException ex)
{
- e.printStackTrace();
+ ex.printStackTrace();
}
- return props.getProperty( FelixConstants.FELIX_VERSION_PROPERTY, "unknown" );
+ return props.getProperty(FelixConstants.FELIX_VERSION_PROPERTY, "unknown");
}
-
- public static String getVersion()
- {
- return FELIX_VERSION_VALUE;
- }
-
-
private void processAutoProperties()
{
// The auto-install property specifies a space-delimited list of
Modified: incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/SystemBundle.java
URL: http://svn.apache.org/viewcvs/incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/SystemBundle.java?rev=384518&r1=384517&r2=384518&view=diff
==============================================================================
--- incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/SystemBundle.java (original)
+++ incubator/felix/trunk/org.apache.felix.framework/src/main/java/org/apache/felix/framework/SystemBundle.java Thu Mar 9 06:50:16 2006
@@ -126,8 +126,10 @@
// Initialize header map as a case insensitive map.
Map map = new StringMap(false);
- map.put(FelixConstants.BUNDLE_VERSION, Felix.getVersion() );
- map.put(FelixConstants.BUNDLE_SYMBOLICNAME, FelixConstants.SYSTEM_BUNDLE_SYMBOLICNAME);
+ map.put(FelixConstants.BUNDLE_VERSION,
+ getFelix().getConfig().get(FelixConstants.FELIX_VERSION_PROPERTY));
+ map.put(FelixConstants.BUNDLE_SYMBOLICNAME,
+ FelixConstants.SYSTEM_BUNDLE_SYMBOLICNAME);
map.put(FelixConstants.BUNDLE_NAME, "System Bundle");
map.put(FelixConstants.BUNDLE_DESCRIPTION,
"This bundle is system specific; it implements various system services.");