Sorry, I forgot the revision 1075685.

On 02/28/2011 11:40 PM, Marcel Offermans wrote:
On Feb 28, 2011, at 22:41 ,<[email protected]>  wrote:

Author: jbonofre
Date: Mon Feb 28 21:41:41 2011
New Revision: 1075548

Modified: incubator/ace/trunk/ace-discovery-property/pom.xml
URL: 
http://svn.apache.org/viewvc/incubator/ace/trunk/ace-discovery-property/pom.xml?rev=1075548&r1=1075547&r2=1075548&view=diff
==============================================================================
--- incubator/ace/trunk/ace-discovery-property/pom.xml (original)
+++ incubator/ace/trunk/ace-discovery-property/pom.xml Mon Feb 28 21:41:41 2011
@@ -40,7 +40,8 @@
             *
         </import.package>
         <private.package>
-            org.apache.ace.discovery.property
+            org.apache.ace.discovery.property,
+            org.apache.ace.discovery.property.constants
         </private.package>
         
<bundle.activator>org.apache.ace.discovery.property.Activator</bundle.activator>
     </properties>

Why are you adding the .constants package? It only contains constants that are 
applied by the Java compiler. They are not needed at all at runtime, which is 
why we did not include them in the first place.

Also, I regret to have to say the the build is still broken. That means I 
basically could not take a look at the problematic integration test that kept 
hanging. I would really appreciate it if you would fix the build on the next 
commit, or roll everything back to a compiling state and prepare an update that 
works completely.

Greetings, Marcel

Reply via email to