Skip to content

Commit 065f7ac

Browse files
committed
Source jvm-options from JRE module path, use in cmd
Source jvm-options (if it exists) from the Cekit "JRE" module path, rather than JDK. Move the evaluation of 'jvm_specific_options' out of the GC options function (gc_config) and into the general command line assembly. At present, none of the Cekit modules (JDK or JRE) provide the jvm-options file. Signed-off-by: Jonathan Dowland <[email protected]>
1 parent 645a631 commit 065f7ac

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

modules/jvm/artifacts/opt/jboss/container/java/jvm/java-default-options

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ jvm_specific_options() {
1111
}
1212

1313
# Include overridden jvm_specific_*() functions
14-
if [ -f "${JBOSS_CONTAINER_OPENJDK_JDK_MODULE}/jvm-options" ]; then
15-
source "${JBOSS_CONTAINER_OPENJDK_JDK_MODULE}/jvm-options"
14+
if [ -f "${JBOSS_CONTAINER_OPENJDK_JRE_MODULE}/jvm-options" ]; then
15+
source "${JBOSS_CONTAINER_OPENJDK_JRE_MODULE}/jvm-options"
1616
fi
1717

1818
# Check for memory options and calculate a sane default if not given
@@ -60,7 +60,7 @@ gc_config() {
6060
fi
6161
fi
6262

63-
local allOptions="$(jvm_specific_options) "
63+
local allOptions=""
6464
allOptions+="${gcOptions} "
6565
allOptions+="-XX:MinHeapFreeRatio=${minHeapFreeRatio} "
6666
allOptions+="-XX:MaxHeapFreeRatio=${maxHeapFreeRatio} "
@@ -81,5 +81,7 @@ error_handling() {
8181
echo "-XX:+ExitOnOutOfMemoryError"
8282
}
8383

84-
## Echo options, trimming trailing and multiple spaces
85-
echo "$(max_memory) $(gc_config) $(diagnostics) $(error_handling)" | awk '$1=$1'
84+
echo "$(max_memory) $(gc_config) $(diagnostics) $(error_handling)\
85+
$(jvm_specific_options)
86+
87+
" | awk '$1=$1' # trim leading/trailing spaces and squash multi

0 commit comments

Comments
 (0)