Index: /jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java
===================================================================
--- /jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java	(revision 680075)
+++ /jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java	(working copy)
@@ -625,9 +625,10 @@
                     && CLUSTER_ELEMENT.equals(child.getNodeName())) {
                 Element element = (Element) child;
 
-                String id = getAttribute(element, ID_ATTRIBUTE, null);
-                long syncDelay = Long.parseLong(
-                        getAttribute(element, SYNC_DELAY_ATTRIBUTE, DEFAULT_SYNC_DELAY));
+                String id = replaceVariables(getAttribute(
+                    element, ID_ATTRIBUTE, null));
+                long syncDelay = Long.parseLong(replaceVariables(getAttribute(
+                    element, SYNC_DELAY_ATTRIBUTE, DEFAULT_SYNC_DELAY)));
 
                 JournalConfig jc = parseJournalConfig(element);
                 return new ClusterConfig(id, syncDelay, jc);
Index: /jackrabbit-core/src/test/java/org/apache/jackrabbit/core/config/RepositoryConfigTest.java
===================================================================
--- /jackrabbit-core/src/test/java/org/apache/jackrabbit/core/config/RepositoryConfigTest.java	(revision 680075)
+++ /jackrabbit-core/src/test/java/org/apache/jackrabbit/core/config/RepositoryConfigTest.java	(working copy)
@@ -238,4 +238,37 @@
         }
     }
 
+
+    /**
+     * Test that the repository configuration file is correctly parsed.
+     */
+    public void testRepositoryConfigWithSystemVariables() throws Exception {
+
+        System.setProperty("systemtest", "testvalue");
+        System.setProperty("systemtestlong", "11");
+
+        InputStream repoxml = getClass()
+            .getResourceAsStream(
+                "/org/apache/jackrabbit/core/config/repository-test-systemproperties.xml");
+        RepositoryConfig config = RepositoryConfig.create(
+            repoxml, REPOSITORY_HOME);
+
+        assertEquals("testvalue", config.getDefaultWorkspaceName());
+        assertEquals(1, config.getWorkspaceConfigs().size());
+        WorkspaceConfig workspaceConfig = (WorkspaceConfig) config
+            .getWorkspaceConfigs().iterator().next();
+        assertEquals("testvalue", workspaceConfig.getName());
+
+        ClusterConfig clusterConfig = config.getClusterConfig();
+        assertEquals("testvalue", clusterConfig.getId());
+        assertEquals(11, clusterConfig.getSyncDelay());
+
+
+        JournalConfig journalConfig = clusterConfig.getJournalConfig();
+
+        assertEquals("testvalue", journalConfig.getParameters().getProperty(
+            "test"));
+
+    }
+
 }
Index: /jackrabbit-core/src/test/resources/org/apache/jackrabbit/core/config/repository-test-systemproperties.xml
===================================================================
--- /jackrabbit-core/src/test/resources/org/apache/jackrabbit/core/config/repository-test-systemproperties.xml	(revision 0)
+++ /jackrabbit-core/src/test/resources/org/apache/jackrabbit/core/config/repository-test-systemproperties.xml	(revision 0)
@@ -0,0 +1,148 @@
+<?xml version="1.0"?>
+<!--
+   Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  See the NOTICE file distributed with
+   this work for additional information regarding copyright ownership.
+   The ASF licenses this file to You under the Apache License, Version 2.0
+   (the "License"); you may not use this file except in compliance with
+   the License.  You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+-->
+<!DOCTYPE Repository PUBLIC "-//The Apache Software Foundation//DTD Jackrabbit 1.5//EN"
+                            "http://jackrabbit.apache.org/dtd/repository-1.5.dtd">
+<!-- Example Repository Configuration File
+     Used by
+     - org.apache.jackrabbit.core.config.RepositoryConfigTest.java
+     -
+-->
+<Repository>
+    <!--
+        virtual file system where the repository stores global state
+        (e.g. registered namespaces, custom node types, etc.)
+    -->
+    <FileSystem class="org.apache.jackrabbit.core.fs.local.LocalFileSystem">
+        <param name="path" value="${rep.home}/repository"/>
+    </FileSystem>
+
+    <!--
+        security configuration
+    -->
+    <Security appName="Jackrabbit">
+        <!--
+            security manager:
+            class: FQN of class implementing the JackrabbitSecurityManager interface
+        -->
+        <SecurityManager class="org.apache.jackrabbit.core.security.simple.SimpleSecurityManager" workspaceName="security">
+            <!--
+            workspace access:
+            class: FQN of class implementing the WorkspaceAccessManager interface
+            -->
+            <!-- <WorkspaceAccessManager class="..."/> -->
+            <!-- <param name="config" value="${rep.home}/security.xml"/> -->
+        </SecurityManager>
+
+        <!--
+            access manager:
+            class: FQN of class implementing the AccessManager interface
+        -->
+        <AccessManager class="org.apache.jackrabbit.core.security.simple.SimpleAccessManager">
+            <!-- <param name="config" value="${rep.home}/access.xml"/> -->
+        </AccessManager>
+
+        <LoginModule class="org.apache.jackrabbit.core.security.simple.SimpleLoginModule">
+           <!-- anonymous user id -->
+           <param name="anonymousId" value="anonymous"/>
+           <!-- administrator user id (default value if param is missing is 'admin') -->
+           <param name="adminId" value="admin"/>
+           <!--
+              default user name to be used instead of the anonymous user
+              when no login credentials are provided (unset by default)
+           -->
+           <!-- <param name="defaultUserId" value="superuser"/> -->
+        </LoginModule>
+    </Security>
+
+    <!--
+        location of workspaces root directory and name of default workspace
+    -->
+    <Workspaces rootPath="${rep.home}/workspaces" defaultWorkspace="${systemtest}"/>
+    <!--
+        workspace configuration template:
+        used to create the initial workspace if there's no workspace yet
+    -->
+    <Workspace name="${systemtest}">
+        <!--
+            virtual file system of the workspace:
+            class: FQN of class implementing the FileSystem interface
+        -->
+        <FileSystem class="org.apache.jackrabbit.core.fs.local.LocalFileSystem">
+            <param name="path" value="${wsp.home}"/>
+        </FileSystem>
+        <!--
+            persistence manager of the workspace:
+            class: FQN of class implementing the PersistenceManager interface
+        -->
+        <PersistenceManager class="org.apache.jackrabbit.core.persistence.bundle.DerbyPersistenceManager">
+          <param name="url" value="jdbc:derby:${wsp.home}/db;create=true"/>
+          <param name="schemaObjectPrefix" value="${wsp.name}_"/>
+        </PersistenceManager>
+        <!--
+            Search index and the file system it uses.
+            class: FQN of class implementing the QueryHandler interface
+        -->
+        <SearchIndex class="org.apache.jackrabbit.core.query.lucene.SearchIndex">
+            <param name="path" value="${wsp.home}/index"/>
+            <param name="textFilterClasses" value="org.apache.jackrabbit.extractor.MsWordTextExtractor,org.apache.jackrabbit.extractor.MsExcelTextExtractor,org.apache.jackrabbit.extractor.MsPowerPointTextExtractor,org.apache.jackrabbit.extractor.PdfTextExtractor,org.apache.jackrabbit.extractor.OpenOfficeTextExtractor,org.apache.jackrabbit.extractor.RTFTextExtractor,org.apache.jackrabbit.extractor.HTMLTextExtractor,org.apache.jackrabbit.extractor.XMLTextExtractor"/>
+            <param name="extractorPoolSize" value="2"/>
+            <param name="supportHighlighting" value="true"/>
+        </SearchIndex>
+    </Workspace>
+
+    <!--
+        Configures the versioning
+    -->
+    <Versioning rootPath="${rep.home}/version">
+        <!--
+            Configures the filesystem to use for versioning for the respective
+            persistence manager
+        -->
+        <FileSystem class="org.apache.jackrabbit.core.fs.local.LocalFileSystem">
+            <param name="path" value="${rep.home}/version" />
+        </FileSystem>
+
+        <!--
+            Configures the persistence manager to be used for persisting version state.
+            Please note that the current versioning implementation is based on
+            a 'normal' persistence manager, but this could change in future
+            implementations.
+        -->
+        <PersistenceManager class="org.apache.jackrabbit.core.persistence.bundle.DerbyPersistenceManager">
+          <param name="url" value="jdbc:derby:${rep.home}/version/db;create=true"/>
+          <param name="schemaObjectPrefix" value="version_"/>
+        </PersistenceManager>
+    </Versioning>
+
+    <!--
+        Search index for content that is shared repository wide
+        (/jcr:system tree, contains mainly versions)
+    -->
+    <SearchIndex class="org.apache.jackrabbit.core.query.lucene.SearchIndex">
+        <param name="path" value="${rep.home}/repository/index"/>
+        <param name="textFilterClasses" value="org.apache.jackrabbit.extractor.MsWordTextExtractor,org.apache.jackrabbit.extractor.MsExcelTextExtractor,org.apache.jackrabbit.extractor.MsPowerPointTextExtractor,org.apache.jackrabbit.extractor.PdfTextExtractor,org.apache.jackrabbit.extractor.OpenOfficeTextExtractor,org.apache.jackrabbit.extractor.RTFTextExtractor,org.apache.jackrabbit.extractor.HTMLTextExtractor,org.apache.jackrabbit.extractor.XMLTextExtractor"/>
+        <param name="extractorPoolSize" value="2"/>
+        <param name="supportHighlighting" value="true"/>
+    </SearchIndex>
+  <Cluster id="${systemtest}" syncDelay="${systemtestlong}">
+    <Journal class="org.apache.jackrabbit.core.journal.JNDIDatabaseJournal">
+      <param name="schema" value="oracle" />
+      <param name="test" value="${systemtest}" />
+    </Journal>
+  </Cluster>
+</Repository>

Property changes on: /jackrabbit-core/src/test/resources/org/apache/jackrabbit/core/config/repository-test-systemproperties.xml
___________________________________________________________________
Name: svn:eol-style
   + native
Name: svn:keywords
   + Author Date Id Revision
Name: svn:mime-type
   + text/plain