Skip to content

Commit 5f04691

Browse files
committed
Fix return code handling
1 parent b74f064 commit 5f04691

File tree

2 files changed

+9
-12
lines changed
  • modules
    • s2i/bash/artifacts/usr/local/s2i

2 files changed

+9
-12
lines changed

modules/jlink/artifacts/opt/jboss/container/java/jlink/mkdeps.sh

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,7 @@ function generate_deps() {
2121
--module-path dependencies \
2222
"$JAVA_APP_JAR" \
2323
"$JAVA_LIB_DIR"/**/*.jar \
24-
> deps.txt || {
25-
echo "jdeps failed: return code $?"
26-
exit $?
27-
}
24+
> deps.txt
2825
else
2926
$JAVA_HOME/bin/jdeps --multi-release $JAVA_VERSION -R -s \
3027
--module-path dependencies \

modules/s2i/bash/artifacts/usr/local/s2i/assemble

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,29 +28,29 @@ if [ "$S2I_ENABLE_JLINK" = "true" ]; then
2828
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkdeps.sh"
2929
echo "Invoking mkdeps"
3030
generate_deps || {
31-
log_error "mkdeps failed, return code: $1"
32-
exit $1
31+
log_error "mkdeps failed, return code: $?"
32+
exit 1
3333
}
3434

3535
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkstrippeddeps.sh"
3636
echo "Stripping dependencies"
3737
mkstrippeddeps || {
38-
log_error "stripping dependencies failed, return code: $1"
39-
exit $1
38+
log_error "stripping dependencies failed, return code: $?"
39+
exit 1
4040
}
4141

4242
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/generatejdkdeps.sh"
4343
echo "Generating JDK dependencies"
4444
generatejdkdeps || {
45-
log_error "generating JDK dependencies failed, return code: $1"
46-
exit $1
45+
log_error "generating JDK dependencies failed, return code: $?"
46+
exit 1
4747
}
4848

4949
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkjreimage.sh"
5050
echo "Linking jre"
5151
generate_jre_image || {
52-
log_error "Linking JRE failed, return code: $1"
53-
exit $1
52+
log_error "Linking JRE failed, return code: $?"
53+
exit 1
5454
}
5555

5656
fi

0 commit comments

Comments
 (0)