Skip to content

Commit

Permalink
moved the tools.jar propertes into the root pom so we can share ${too…
Browse files Browse the repository at this point in the history
…lsJar} across maven modules - and added it as a system dependency on hawtio-embedded so that the local JVM mbean works
  • Loading branch information
jstrachan committed Jul 12, 2013
1 parent e544476 commit ba8dffa
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 24 deletions.
8 changes: 8 additions & 0 deletions hawtio-embedded/pom.xml
Expand Up @@ -22,6 +22,14 @@
<version>${jetty-version}</version>
</dependency>

<dependency>
<groupId>com.sun</groupId>
<artifactId>tools</artifactId>
<version>1.6</version>
<scope>system</scope>
<systemPath>${toolsjar}</systemPath>
</dependency>

<!-- logging -->
<dependency>
<groupId>org.slf4j</groupId>
Expand Down
24 changes: 0 additions & 24 deletions hawtio-local-jvm-mbean/pom.xml
Expand Up @@ -101,29 +101,5 @@
</build>

<profiles>
<profile>
<id>default-profile</id>
<activation>
<activeByDefault>true</activeByDefault>
<file>
<exists>${java.home}/../lib/tools.jar</exists>
</file>
</activation>
<properties>
<toolsjar>${java.home}/../lib/tools.jar</toolsjar>
</properties>
</profile>
<profile>
<id>mac-profile</id>
<activation>
<activeByDefault>false</activeByDefault>
<file>
<exists>${java.home}/../Classes/classes.jar</exists>
</file>
</activation>
<properties>
<toolsjar>${java.home}/../Classes/classes.jar</toolsjar>
</properties>
</profile>
</profiles>
</project>
27 changes: 27 additions & 0 deletions pom.xml
Expand Up @@ -465,5 +465,32 @@
...
-->
</profile>


<!-- dealing with tools.jar for hawtio-local-jvm-mbean -->
<profile>
<id>default-profile</id>
<activation>
<activeByDefault>true</activeByDefault>
<file>
<exists>${java.home}/../lib/tools.jar</exists>
</file>
</activation>
<properties>
<toolsjar>${java.home}/../lib/tools.jar</toolsjar>
</properties>
</profile>
<profile>
<id>mac-profile</id>
<activation>
<activeByDefault>false</activeByDefault>
<file>
<exists>${java.home}/../Classes/classes.jar</exists>
</file>
</activation>
<properties>
<toolsjar>${java.home}/../Classes/classes.jar</toolsjar>
</properties>
</profile>
</profiles>
</project>

0 comments on commit ba8dffa

Please sign in to comment.