Skip to content

Commit 4a71df6

Browse files
committed
[OPENJDK-3655] remove or adjust chmod calls for installed scripts
Many of the shell scripts we copy into the image are shell "libraries" which are intended to be sourced by other shell scripts and not directly executed themselves. It's confusing to install them with +x permissions. The scripts which are intended to be executed were set with owner or group execute permission but "other" without. This was a problem for some users of the containers in a non-OpenShift context, and there was no good reason for it, so set those +x for other as well.
1 parent 4e45394 commit 4a71df6

File tree

15 files changed

+2
-15
lines changed

15 files changed

+2
-15
lines changed

modules/jdk/11/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jdk/*
1110

1211
pushd ${ARTIFACTS_DIR}
1312
cp -pr * /

modules/jdk/17/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jdk/*
1110

1211
pushd ${ARTIFACTS_DIR}
1312
cp -pr * /

modules/jdk/21/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jdk/*
1110

1211
pushd ${ARTIFACTS_DIR}
1312
cp -pr * /

modules/jdk/8/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jdk/*
1110

1211
pushd ${ARTIFACTS_DIR}
1312
cp -pr * /

modules/jre/11/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ echo $ARTIFACTS_DIR
99

1010
chown -R $USER:root $SCRIPT_DIR
1111
chmod -R ug+rwX $SCRIPT_DIR
12-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jre/*
1312

1413
pushd ${ARTIFACTS_DIR}
1514
cp -pr * /

modules/jre/17/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ echo $ARTIFACTS_DIR
99

1010
chown -R $USER:root $SCRIPT_DIR
1111
chmod -R ug+rwX $SCRIPT_DIR
12-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jre/*
1312

1413
pushd ${ARTIFACTS_DIR}
1514
cp -pr * /

modules/jre/21/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ echo $ARTIFACTS_DIR
99

1010
chown -R $USER:root $SCRIPT_DIR
1111
chmod -R ug+rwX $SCRIPT_DIR
12-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jre/*
1312

1413
pushd ${ARTIFACTS_DIR}
1514
cp -pr * /

modules/jre/8/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/openjdk/jre/*
1110

1211
pushd ${ARTIFACTS_DIR}
1312
cp -pr * /

modules/jvm/configure.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
77

88
chown -R $USER:root $SCRIPT_DIR
99
chmod -R ug+rwX $SCRIPT_DIR
10-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/java/jvm/*
10+
chmod +x ${ARTIFACTS_DIR}/opt/jboss/container/java/jvm/*
1111

1212
pushd ${ARTIFACTS_DIR}
1313
cp -pr * /

modules/maven/default/configure.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
88
# configure artifact permissions
99
chown -R $USER:root $ARTIFACTS_DIR
1010
chmod -R ug+rwX $ARTIFACTS_DIR
11-
chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/maven/default/maven.sh
1211

1312
# install artifacts
1413
pushd ${ARTIFACTS_DIR}

0 commit comments

Comments
 (0)