Skip to content

Commit 2a0c7b1

Browse files
jhuttanajmtd
authored andcommitted
Tweak run-java.sh script for some env varibales and fold run-env.sh into run-java.sh
- This tweak is to avoid defining JBOSS_CONTAINER_UTIL_LOGGING_MODULE as ENV in templates/jlink/Dokcerfile - Deleted run-env.sh and folded its content to run-java.sh as its not doing anything much other than setting value of JAVA_APP_DIR Signed-off-by: Jayashree Huttanagoudar <[email protected]>
1 parent d9035d2 commit 2a0c7b1

File tree

2 files changed

+8
-12
lines changed

2 files changed

+8
-12
lines changed

modules/run/artifacts/opt/jboss/container/java/run/run-env.sh

Lines changed: 0 additions & 6 deletions
This file was deleted.

modules/run/artifacts/opt/jboss/container/java/run/run-java.sh

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,13 @@
33
# Fail on a single failed command
44
set -eo pipefail
55

6+
#These are defined explicitly here to avoid defining them in templates/jlink/Dockerfile
7+
export JBOSS_CONTAINER_UTIL_LOGGING_MODULE="/opt/jboss/container/util/logging"
8+
export JBOSS_CONTAINER_JAVA_RUN_MODULE="/opt/jboss/container/java/run"
9+
10+
#This is moved here after deleting run-env.sh
11+
export JAVA_APP_DIR=/deployments
12+
613
source "$JBOSS_CONTAINER_UTIL_LOGGING_MODULE/logging.sh"
714

815
# ==========================================================
@@ -96,13 +103,8 @@ load_env() {
96103
if [ -z "${JAVA_APP_DIR}" ]; then
97104
# XXX: is this correct? This is defaulted above to /deployments. Should we
98105
# define a default to the old /opt/java-run?
99-
JAVA_APP_DIR="${JBOSS_CONTAINER_JAVA_RUN_MODULE}"
100-
else
101-
if [ -f "${JAVA_APP_DIR}/${run_env_sh}" ]; then
102-
source "${JAVA_APP_DIR}/${run_env_sh}"
103-
fi
106+
export JAVA_APP_DIR="${JBOSS_CONTAINER_JAVA_RUN_MODULE}"
104107
fi
105-
export JAVA_APP_DIR
106108

107109
# JAVA_LIB_DIR defaults to JAVA_APP_DIR
108110
export JAVA_LIB_DIR="${JAVA_LIB_DIR:-${JAVA_APP_DIR}}"

0 commit comments

Comments
 (0)