Removing the permgen JVM options for JDK 8 and higher (Linux part). T…

…he warning will still be there on Windows.
rhq-project · Apr 28, 2015 · b1708d2 · b1708d2
1 parent 1d6201f
commit b1708d2
Show file tree

Hide file tree

Showing 7 changed files with 24 additions and 7 deletions.
diff --git a/modules/enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-installer.sh b/modules/enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-installer.sh
@@ -179,7 +179,10 @@ fi
 # ----------------------------------------------------------------------
 
 if [ -z "$RHQ_SERVER_INSTALLER_JAVA_OPTS" ]; then
-   RHQ_SERVER_INSTALLER_JAVA_OPTS="-Xms512M -Xmx512M -XX:PermSize=128M -XX:MaxPermSize=128M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   RHQ_SERVER_INSTALLER_JAVA_OPTS="-Xms512M -Xmx512M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || {
+      RHQ_SERVER_INSTALLER_JAVA_OPTS="$RHQ_SERVER_INSTALLER_JAVA_OPTS -XX:PermSize=128M -XX:MaxPermSize=128M"
+   }
 fi
 
 # Add the JVM opts that we always want to specify, whether or not the user set RHQ_SERVER_INSTALLER_JAVA_OPTS.

diff --git a/modules/enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-server.sh b/modules/enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-server.sh
@@ -349,7 +349,10 @@ export JAVA
 # ----------------------------------------------------------------------
 
 if [ -z "$RHQ_SERVER_JAVA_OPTS" ]; then
-   RHQ_SERVER_JAVA_OPTS="-Xms1024M -Xmx1024M -XX:PermSize=256M -XX:MaxPermSize=256M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true -Dsun.rmi.dgc.client.gcInterval=3600000 -Dsun.rmi.dgc.server.gcInterval=3600000"
+   RHQ_SERVER_JAVA_OPTS="-Xms1024M -Xmx1024M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true -Dsun.rmi.dgc.client.gcInterval=3600000 -Dsun.rmi.dgc.server.gcInterval=3600000"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || {
+      RHQ_SERVER_JAVA_OPTS="$RHQ_SERVER_JAVA_OPTS -XX:PermSize=256M -XX:MaxPermSize=256M"
+   }
 fi
 
 if [ -n "$RHQ_SERVER_DEBUG" ] && [ "$RHQ_SERVER_DEBUG" != "false" ]; then

diff --git a/.../enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-storage-installer.sh b/.../enterprise/server/appserver/src/main/bin-resources/bin/internal/rhq-storage-installer.sh
@@ -179,7 +179,10 @@ fi
 # ----------------------------------------------------------------------
 
 if [ -z "$RHQ_STORAGE_INSTALLER_JAVA_OPTS" ]; then
-   RHQ_STORAGE_INSTALLER_JAVA_OPTS="-Xms512M -Xmx512M -XX:PermSize=128M -XX:MaxPermSize=128M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   RHQ_STORAGE_INSTALLER_JAVA_OPTS="-Xms512M -Xmx512M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || {
+      RHQ_STORAGE_INSTALLER_JAVA_OPTS="$RHQ_STORAGE_INSTALLER_JAVA_OPTS -XX:PermSize=128M -XX:MaxPermSize=128M"
+   }
 fi
 
 # Add the JVM opts that we always want to specify, whether or not the user set RHQ_CCM_JAVA_OPTS.

diff --git a/modules/enterprise/server/appserver/src/main/bin-resources/bin/rhq-data-migration.sh b/modules/enterprise/server/appserver/src/main/bin-resources/bin/rhq-data-migration.sh
@@ -181,7 +181,10 @@ fi
 # ----------------------------------------------------------------------
 
 if [ -z "$RHQ_DATA_MIGRATION_JAVA_OPTS" ]; then
-   RHQ_DATA_MIGRATION_JAVA_OPTS="-Xms512M -Xmx512M -XX:PermSize=128M -XX:MaxPermSize=128M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   RHQ_DATA_MIGRATION_JAVA_OPTS="-Xms512M -Xmx512M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || {
+      RHQ_DATA_MIGRATION_JAVA_OPTS="$RHQ_DATA_MIGRATION_JAVA_OPTS -XX:PermSize=128M -XX:MaxPermSize=128M"
+   }
 fi
 
 # Add the JVM opts that we always want to specify, whether or not the user set RHQ_CCM_JAVA_OPTS.

diff --git a/modules/enterprise/server/appserver/src/main/bin-resources/bin/rhqctl b/modules/enterprise/server/appserver/src/main/bin-resources/bin/rhqctl
@@ -181,7 +181,10 @@ fi
 # ----------------------------------------------------------------------
 
 if [ -z "$RHQ_CONTROL_JAVA_OPTS" ]; then
-   RHQ_CONTROL_JAVA_OPTS=" -Xmx512M -XX:MaxPermSize=128M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   RHQ_CONTROL_JAVA_OPTS=" -Xmx512M -Djava.net.preferIPv4Stack=true -Dorg.jboss.resolver.warning=true"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || {
+      RHQ_CONTROL_JAVA_OPTS="$RHQ_CONTROL_JAVA_OPTS -XX:MaxPermSize=128M"
+   }
 fi
 
 # Add the JVM opts that we always want to specify, whether or not the user set RHQ_CCM_JAVA_OPTS.

diff --git a/publish.sh b/publish.sh
@@ -206,7 +206,8 @@ set_local_and_environment_variables()
    print_function_information $FUNCNAME
 
    # Set various environment variables.
-   MAVEN_OPTS="-Xms512M -Xmx1024M -XX:PermSize=128M -XX:MaxPermSize=256M"
+   MAVEN_OPTS="-Xms512M -Xmx1024M"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || MAVEN_OPTS="$MAVEN_OPTS -XX:PermSize=128M -XX:MaxPermSize=256M"
    export MAVEN_OPTS
 
    # Set various local variables

diff --git a/release.sh b/release.sh
@@ -269,7 +269,8 @@ set_local_and_environment_variables()
    print_function_information $FUNCNAME
 
    # Set environment variables
-   MAVEN_OPTS="-Xms512M -Xmx1024M -XX:PermSize=128M -XX:MaxPermSize=256M"
+   MAVEN_OPTS="-Xms512M -Xmx1024M"
+   [[ $("$JAVA_HOME/bin/java" -version 2>&1 | awk -F '"' '/version/ {print $2}') > "1.8" ]] || MAVEN_OPTS="$MAVEN_OPTS -XX:PermSize=128M -XX:MaxPermSize=256M"
    export MAVEN_OPTS