Author: veithen
Date: Thu Nov 13 12:20:29 2008
New Revision: 713810

URL: http://svn.apache.org/viewvc?rev=713810&view=rev
Log:
tools.jar doesn't exist on Mac OS X. Using a profile as suggested in 
http://maven.apache.org/general.html#tools-jar-dependency to add it as a 
dependency only when building on Sun JDKs. This should fix the build for all 
platforms.

Modified:
    webservices/commons/trunk/modules/transport/modules/testkit/pom.xml

Modified: webservices/commons/trunk/modules/transport/modules/testkit/pom.xml
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/transport/modules/testkit/pom.xml?rev=713810&r1=713809&r2=713810&view=diff
==============================================================================
--- webservices/commons/trunk/modules/transport/modules/testkit/pom.xml 
(original)
+++ webservices/commons/trunk/modules/transport/modules/testkit/pom.xml Thu Nov 
13 12:20:29 2008
@@ -76,15 +76,29 @@
             <artifactId>jetty</artifactId>
             <version>5.1.10</version>
         </dependency>
-               <dependency>
-                       <groupId>com.sun</groupId>
-                       <artifactId>tools</artifactId>
-                       <version>1.5.0</version>
-                       <scope>system</scope>
-                       <systemPath>${java.home}/../lib/tools.jar</systemPath>
-               </dependency>
     </dependencies>
 
+    <profiles>
+        <profile>
+            <id>default-tools.jar</id>
+            <activation>
+                <property>
+                    <name>java.vendor</name>
+                    <value>Sun Microsystems Inc.</value>
+                </property>
+            </activation>
+            <dependencies>
+                <dependency>
+                    <groupId>com.sun</groupId>
+                    <artifactId>tools</artifactId>
+                    <version>1.5.0</version>
+                    <scope>system</scope>
+                    <systemPath>${java.home}/../lib/tools.jar</systemPath>
+                </dependency>
+            </dependencies>
+        </profile>
+    </profiles>
+
     <build>
         <resources>
             <resource>


Reply via email to