Author: robbie
Date: Wed Nov 21 23:27:07 2012
New Revision: 1412359

URL: http://svn.apache.org/viewvc?rev=1412359&view=rev
Log:
QPID-4456: ensure the broker plugins get packaged in the release binary and 
added to the classpath. Create opt directory for later use by users and update 
docs accordingly.

Modified:
    qpid/trunk/qpid/doc/book/src/java-broker/Java-Broker-Stores-BDB-Store.xml
    qpid/trunk/qpid/java/bdbstore/jmx/build.xml
    qpid/trunk/qpid/java/broker-plugins/access-control/build.xml
    qpid/trunk/qpid/java/broker-plugins/management-http/build.xml
    qpid/trunk/qpid/java/broker-plugins/management-jmx/build.xml
    qpid/trunk/qpid/java/broker/bin/qpid-server
    qpid/trunk/qpid/java/broker/build.xml
    qpid/trunk/qpid/java/common.xml
    qpid/trunk/qpid/java/module.xml

Modified: 
qpid/trunk/qpid/doc/book/src/java-broker/Java-Broker-Stores-BDB-Store.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/doc/book/src/java-broker/Java-Broker-Stores-BDB-Store.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/doc/book/src/java-broker/Java-Broker-Stores-BDB-Store.xml 
(original)
+++ qpid/trunk/qpid/doc/book/src/java-broker/Java-Broker-Stores-BDB-Store.xml 
Wed Nov 21 23:27:07 2012
@@ -52,16 +52,14 @@
     <title>Oracle BDB JE jar installation</title>
     <para>
       If you wish to use the BDBMessageStore, copy the 
je-&oracleBdbProductVersion;.jar from within the release
-      downloaded <link 
linkend="Java-Broker-Stores-BDB-Store-BDBJE-Download">above</link> into an 
'opt' sub-directory
+      downloaded <link 
linkend="Java-Broker-Stores-BDB-Store-BDBJE-Download">above</link> into the 
'opt' sub-directory
       of the brokers 'lib' directory.
     </para>
 
     <programlisting>Unix:
-mkdir qpid-broker-&qpidCurrentRelease;/lib/opt
 cp je-&oracleBdbProductVersion;.jar 
qpid-broker-&qpidCurrentRelease;/lib/opt</programlisting>
 
       <programlisting>Windows:
-mkdir qpid-broker-&qpidCurrentRelease;\lib\opt
 copy je-&oracleBdbProductVersion;.jar 
qpid-broker-&qpidCurrentRelease;\lib\opt</programlisting>
   </section>
 

Modified: qpid/trunk/qpid/java/bdbstore/jmx/build.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/bdbstore/jmx/build.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/bdbstore/jmx/build.xml (original)
+++ qpid/trunk/qpid/java/bdbstore/jmx/build.xml Wed Nov 21 23:27:07 2012
@@ -23,6 +23,8 @@
     <property name="module.genpom" value="true"/>
     <property name="module.genpom.args" value="-Sqpid-common=provided 
-Sqpid-broker=provided -Sqpid-broker-plugins-management-jmx=provided 
-Sqpid-management-common=provided -Sqpid-bdbstore=provided -Sje=provided"/>
 
+    <property name="broker.plugin" value="true"/>
+
     <import file="../../module.xml" />
 
     <target name="bundle" depends="bundle-tasks" />

Modified: qpid/trunk/qpid/java/broker-plugins/access-control/build.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-plugins/access-control/build.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker-plugins/access-control/build.xml (original)
+++ qpid/trunk/qpid/java/broker-plugins/access-control/build.xml Wed Nov 21 
23:27:07 2012
@@ -23,6 +23,8 @@
     <property name="module.genpom" value="true"/>
     <property name="module.genpom.args" value="-Sqpid-common=provided 
-Sqpid-broker=provided"/>
 
+    <property name="broker.plugin" value="true"/>
+
     <property name="broker-plugins-access-control.libs" value=""/>
 
     <import file="../../module.xml" />

Modified: qpid/trunk/qpid/java/broker-plugins/management-http/build.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-plugins/management-http/build.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker-plugins/management-http/build.xml (original)
+++ qpid/trunk/qpid/java/broker-plugins/management-http/build.xml Wed Nov 21 
23:27:07 2012
@@ -24,6 +24,8 @@
     <property name="module.genpom" value="true" />
     <property name="module.genpom.args" value="-Sqpid-common=provided 
-Sqpid-broker=provided" />
 
+    <property name="broker.plugin" value="true"/>
+
     <property name="broker-plugins-management-http.libs" value="" />
 
     <import file="../../module.xml" />

Modified: qpid/trunk/qpid/java/broker-plugins/management-jmx/build.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-plugins/management-jmx/build.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker-plugins/management-jmx/build.xml (original)
+++ qpid/trunk/qpid/java/broker-plugins/management-jmx/build.xml Wed Nov 21 
23:27:07 2012
@@ -23,6 +23,8 @@
     <property name="module.genpom" value="true"/>
     <property name="module.genpom.args" value="-Sqpid-common=provided 
-Sqpid-broker=provided -Sqpid-management-common=provided"/>
 
+    <property name="broker.plugin" value="true"/>
+
     <property name="broker-plugins-management-jmx.libs" value=""/>
 
     <import file="../../module.xml" />

Modified: qpid/trunk/qpid/java/broker/bin/qpid-server
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/bin/qpid-server?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/bin/qpid-server (original)
+++ qpid/trunk/qpid/java/broker/bin/qpid-server Wed Nov 21 23:27:07 2012
@@ -33,8 +33,8 @@ if [ -z "$QPID_PNAME" ]; then
     export QPID_PNAME=" -DPNAME=QPBRKR"
 fi
 
-# Set classpath to include the qpid-all manifest jar, and any jars supplied in 
lib/opt
-QPID_LIBS="$QPID_HOME/lib/qpid-all.jar:$QPID_HOME/lib/opt/*"
+# Set classpath to include the qpid-all manifest jar, plus jars in lib/plugins 
and lib/opt
+QPID_LIBS="$QPID_HOME/lib/qpid-all.jar:$QPID_HOME/lib/plugins/*:$QPID_HOME/lib/opt/*"
 
 # Set other variables used by the qpid-run script before calling
 export JAVA=java \

Modified: qpid/trunk/qpid/java/broker/build.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/build.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/build.xml (original)
+++ qpid/trunk/qpid/java/broker/build.xml Wed Nov 21 23:27:07 2012
@@ -48,9 +48,15 @@
         <fixcrlf srcdir="${module.release}/bin" fixlast="true" eol="dos" 
includes="*.bat"/>
     </target>
 
-    <target name="release-bin-other" depends="release-bin-other-bdbstore" 
description="copy broker-plugins into module release">
+    <target name="release-bin-other" 
depends="release-bin-other-lib-opt,release-bin-other-bdbstore,release-bin-copy-broker-plugins"/>
+
+    <target name="release-bin-other-lib-opt" 
depends="release-bin-other-bdbstore" description="make lib/opt dir in the 
module release">
+        <mkdir dir="${module.release}/lib/opt"/>
+    </target>
+
+    <target name="release-bin-copy-broker-plugins" description="copy 
broker-plugins into module release">
         <copy todir="${module.release}/lib/plugins" failonerror="true">
-            <fileset dir="${build.lib}/plugins"/>
+            <fileset dir="${build.lib.broker.plugins}"/>
         </copy>
     </target>
 

Modified: qpid/trunk/qpid/java/common.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/common.xml (original)
+++ qpid/trunk/qpid/java/common.xml Wed Nov 21 23:27:07 2012
@@ -42,7 +42,7 @@
   <property name="build.report"          location="${build}/report"/>
   <property name="build.release"         location="${build}/release"/>
   <property name="build.release.prepare" location="${build.release}/prepare"/>
-  <property name="build.plugins"         location="${build}/lib/plugins"/>
+  <property name="build.lib.broker.plugins"  
location="${build}/lib/broker-plugins"/>
   <property name="build.coverage.report" location="${build}/coverage/report"/>
   <property name="build.coverage.src"    location="${build}/coverage/src"/>
   <property name="build.findbugs"        location="${build}/findbugs"/>

Modified: qpid/trunk/qpid/java/module.xml
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/module.xml?rev=1412359&r1=1412358&r2=1412359&view=diff
==============================================================================
--- qpid/trunk/qpid/java/module.xml (original)
+++ qpid/trunk/qpid/java/module.xml Wed Nov 21 23:27:07 2012
@@ -74,6 +74,7 @@
   <property name="broker.log.prefix" value="BROKER: "/> 
   <property name="broker.log.interleave" value="true"/>
 
+  <property name="module.jar"          
location="${build.lib}/${project.name}-${module.name}-${project.version}.jar"/>
   <property name="module.qpid.jar"     
location="${module.release.lib}/qpid-all.jar"/>
   <basename property="qpid.jar.name"  file="${module.qpid.jar}"/>
 
@@ -151,15 +152,6 @@
     <pathelement location="${module.test.src}"/>
   </path>
 
-  <condition property="module.jar"
-             
value="${build.plugins}/${project.name}-${module.name}-${project.version}.jar"
-             
else="${build.lib}/${project.name}-${module.name}-${project.version}.jar">
-    <and>
-      <isset property="module.plugin"/>
-      <istrue value="${module.plugin}"/>
-    </and>
-  </condition>
-
   <property name="module.test.jar"
             
location="${build.lib}/${project.name}-${module.name}-tests-${project.version}.jar"/>
 
@@ -248,7 +240,7 @@
     <mkdir dir="${build.etc}"/>
     <mkdir dir="${build.lib}"/>
     <mkdir dir="${build.results}"/>
-    <mkdir dir="${build.plugins}"/>
+    <mkdir dir="${build.lib.broker.plugins}"/>
     <mkdir dir="${module.classes}"/>
     <mkdir dir="${module.precompiled}"/>
     <mkdir dir="${module.api}"/>
@@ -392,8 +384,6 @@
   <property name="QPID_HOME" value="${qpid.home}"/>
   <property name="QPID_WORK" value="${qpid.work}"/>
   <property name="broker.existing.qpid.work" value=""/>
-  <!-- Used by PluginTest -->
-  <property name="example.plugin.target" 
value="${project.root}/build/lib/plugins"/>
 
   <propertyset id="all.test.systemproperties">
     <propertyref prefix="test"/>
@@ -504,7 +494,7 @@
 
   <target name="postbuild" description="run after a build"/>
 
-  <target name="build" 
depends="jar,jar-tests,jar-sources,libs,copy-bin,copy-etc,postbuild" 
description="compile and copy resources into build tree"/>
+  <target name="build" 
depends="jar,jar-tests,jar-sources,libs,copy-bin,copy-etc,postbuild,copy-broker-plugin-jars"
 description="compile and copy resources into build tree"/>
 
   <target name="jar.manifest" depends="compile, copy-module-metainf" 
if="module.manifest">
     <jar destfile="${module.jar}" basedir="${module.classes}" 
manifest="${module.manifest}">
@@ -518,6 +508,10 @@
     </jar>
   </target>
 
+  <target name="copy-broker-plugin-jars" if="${broker.plugin}" 
description="copy broker plugins for use in release packaging">
+    <copy file="${module.jar}" todir="${build.lib.broker.plugins}" 
failonerror="true"/>
+  </target>
+
   <target name="copy-module-metainf" depends="copy-metainf-resources" 
if="module.src.resources.metainf.exists">
     <copy todir="${module.metainf}" failonerror="true">
         <fileset dir="${module.src.resources.metainf}"/>



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to