Skip to content

Commit 5f81c17

Browse files
authored
Merge pull request #606 from rh-openjdk/OPENJDK-4029-remove-overrides-ubi10
[OPENJDK-4029] Remove unused maven_init_var_MAVEN_SETTINGS_XML
2 parents a1da2dc + 52004b8 commit 5f81c17

File tree

3 files changed

+4
-29
lines changed
  • modules

3 files changed

+4
-29
lines changed

modules/maven/default/artifacts/opt/jboss/container/maven/default/maven.sh

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ function maven_init() {
1414
# initialize maven variables
1515
function maven_init_vars() {
1616
maven_init_var_MAVEN_LOCAL_REPO
17-
maven_init_var_MAVEN_SETTINGS_XML
17+
maven_init_var_MAVEN_SETTINGS_XML # must be defined by an accompanying cekit module
1818
maven_init_var_MAVEN_OPTS
1919
maven_init_var_MAVEN_ARGS
2020
}
@@ -23,16 +23,6 @@ function maven_init_var_MAVEN_LOCAL_REPO() {
2323
MAVEN_LOCAL_REPO="${MAVEN_LOCAL_REPO:-${HOME}/.m2/repository}"
2424
}
2525

26-
function maven_init_var_MAVEN_SETTINGS_XML() {
27-
if [ -f "${MAVEN_SETTINGS_XML}" ]; then
28-
:
29-
elif [ -f "${HOME}/.m2/settings.xml" ]; then
30-
MAVEN_SETTINGS_XML="${HOME}/.m2/settings.xml"
31-
else
32-
MAVEN_SETTINGS_XML="${JBOSS_CONTAINER_MAVEN_DEFAULT_MODULE}/jboss-settings.xml"
33-
fi
34-
}
35-
3626
function maven_init_var_MAVEN_OPTS() {
3727
export MAVEN_OPTS="${MAVEN_OPTS:-$(${JBOSS_CONTAINER_JAVA_JVM_MODULE}/java-default-options) -XX:MaxRAMPercentage=25.0}"
3828
}

modules/maven/s2i/artifacts/opt/jboss/container/maven/s2i/maven-overrides

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,18 +3,15 @@ function maven_init_var_MAVEN_LOCAL_REPO() {
33
MAVEN_LOCAL_REPO="${MAVEN_LOCAL_REPO:-${_MAVEN_S2I_ARCHIVED_REPO}}"
44
}
55

6-
# Overrides basic functionality by looking fore settings.xml in <source>/configuration
6+
# if not already defined, look for settings.xml in <source>/configuration
77
function maven_init_var_MAVEN_SETTINGS_XML() {
88
if [ -f "${MAVEN_SETTINGS_XML}" ]; then
99
:
1010
elif [ -f "${S2I_SOURCE_DIR}/configuration/settings.xml" ]; then
11-
MAVEN_SETTINGS_XML="${HOME}/.m2/settings.xml"
12-
mkdir -p $(dirname "${MAVEN_SETTINGS_XML}")
13-
cp "${S2I_SOURCE_DIR}/configuration/settings.xml" "${MAVEN_SETTINGS_XML}"
14-
elif [ -f "${HOME}/.m2/settings.xml" ]; then
15-
MAVEN_SETTINGS_XML="${HOME}/.m2/settings.xml"
11+
MAVEN_SETTINGS_XML="${S2I_SOURCE_DIR}/configuration/settings.xml"
1612
else
1713
MAVEN_SETTINGS_XML="${_MAVEN_S2I_SETTINGS_XML}"
14+
mkdir -p $(dirname "${MAVEN_SETTINGS_XML}")
1815
cp "${JBOSS_CONTAINER_MAVEN_DEFAULT_MODULE}/jboss-settings.xml" "${MAVEN_SETTINGS_XML}"
1916
fi
2017
}

modules/s2i/bash/artifacts/opt/jboss/container/java/s2i/maven-overrides

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

0 commit comments

Comments
 (0)