Skip to content

Commit 49712fa

Browse files
pkubatrhhhorak
authored andcommitted
Dockerfile: put variables inside curly braces (sclorg#130)
Dockerfile: put variables inside curly braces since OSBS had trouble expanding some during build
1 parent 035ac68 commit 49712fa

File tree

4 files changed

+20
-20
lines changed

4 files changed

+20
-20
lines changed

5.20/Dockerfile

+5-5
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ LABEL summary="$SUMMARY" \
2323
io.k8s.description="$DESCRIPTION" \
2424
io.k8s.display-name="Apache 2.4 with mod_perl/$PERL_VERSION" \
2525
io.openshift.expose-services="8080:http" \
26-
io.openshift.tags="builder,$NAME,$NAME$PERL_SHORT_VER" \
26+
io.openshift.tags="builder,${NAME},${NAME}${PERL_SHORT_VER}" \
2727
io.openshift.s2i.scripts-url="image:///usr/libexec/s2i" \
2828
io.s2i.scripts-url="image:///usr/libexec/s2i" \
29-
name="centos/$NAME-$PERL_SHORT_VER-centos7" \
30-
com.redhat.component="rh-$NAME$PERL_SHORT_VER-docker" \
29+
name="centos/${NAME}-${PERL_SHORT_VER}-centos7" \
30+
com.redhat.component="rh-${NAME}${PERL_SHORT_VER}-docker" \
3131
version="$PERL_VERSION" \
3232
maintainer="SoftwareCollections.org <[email protected]>" \
33-
help="For more information visit https://github.com/sclorg/s2i-$NAME-container" \
34-
usage="s2i build <SOURCE-REPOSITORY> centos/$NAME-$PERL_SHORT_VER-centos7:latest <APP-NAME>"
33+
help="For more information visit https://github.com/sclorg/s2i-${NAME}-container" \
34+
usage="s2i build <SOURCE-REPOSITORY> centos/${NAME}-${PERL_SHORT_VER}-centos7:latest <APP-NAME>"
3535

3636
# TODO: Cleanup cpanp cache after cpanminus is installed?
3737
RUN yum install -y centos-release-scl && \

5.20/Dockerfile.rhel7

+5-5
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ LABEL summary="$SUMMARY" \
2323
io.k8s.description="$DESCRIPTION" \
2424
io.k8s.display-name="Apache 2.4 with mod_perl/$PERL_VERSION" \
2525
io.openshift.expose-services="8080:http" \
26-
io.openshift.tags="builder,$NAME,$NAME$PERL_SHORT_VER" \
26+
io.openshift.tags="builder,${NAME},${NAME}${PERL_SHORT_VER}" \
2727
io.openshift.s2i.scripts-url="image:///usr/libexec/s2i" \
2828
io.s2i.scripts-url="image:///usr/libexec/s2i" \
29-
name="rhscl/$NAME-$PERL_SHORT_VER-rhel7" \
30-
com.redhat.component="rh-$NAME$PERL_SHORT_VER-docker" \
29+
name="rhscl/${NAME}-${PERL_SHORT_VER}-rhel7" \
30+
com.redhat.component="rh-${NAME}${PERL_SHORT_VER}-docker" \
3131
version="$PERL_VERSION" \
3232
maintainer="SoftwareCollections.org <[email protected]>" \
33-
help="For more information visit https://github.com/sclorg/s2i-$NAME-container" \
34-
usage="s2i build <SOURCE-REPOSITORY> rhscl/$NAME-$PERL_SHORT_VER-rhel7:latest <APP-NAME>"
33+
help="For more information visit https://github.com/sclorg/s2i-${NAME}-container" \
34+
usage="s2i build <SOURCE-REPOSITORY> rhscl/${NAME}-${PERL_SHORT_VER}-rhel7:latest <APP-NAME>"
3535

3636
# TODO: Cleanup cpanp cache after cpanminus is installed?
3737
RUN yum install -y yum-utils && \

5.24/Dockerfile

+5-5
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ LABEL summary="$SUMMARY" \
2323
io.k8s.description="$DESCRIPTION" \
2424
io.k8s.display-name="Apache 2.4 with mod_perl/$PERL_VERSION" \
2525
io.openshift.expose-services="8080:http" \
26-
io.openshift.tags="builder,$NAME,$NAME$PERL_SHORT_VER" \
26+
io.openshift.tags="builder,${NAME},${NAME}${PERL_SHORT_VER}" \
2727
io.openshift.s2i.scripts-url="image:///usr/libexec/s2i" \
2828
io.s2i.scripts-url="image:///usr/libexec/s2i" \
29-
name="centos/$NAME-$PERL_SHORT_VER-centos7" \
30-
com.redhat.component="rh-$NAME$PERL_SHORT_VER-docker" \
29+
name="centos/${NAME}-${PERL_SHORT_VER}-centos7" \
30+
com.redhat.component="rh-${NAME}${PERL_SHORT_VER}-docker" \
3131
version="$PERL_VERSION" \
3232
maintainer="SoftwareCollections.org <[email protected]>" \
33-
help="For more information visit https://github.com/sclorg/s2i-$NAME-container" \
34-
usage="s2i build <SOURCE-REPOSITORY> centos/$NAME-$PERL_SHORT_VER-centos7:latest <APP-NAME>"
33+
help="For more information visit https://github.com/sclorg/s2i-${NAME}-container" \
34+
usage="s2i build <SOURCE-REPOSITORY> centos/${NAME}-${PERL_SHORT_VER}-centos7:latest <APP-NAME>"
3535

3636
RUN yum install -y centos-release-scl && \
3737
yum-config-manager --enable centos-sclo-rh-testing && \

5.24/Dockerfile.rhel7

+5-5
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ LABEL summary="$SUMMARY" \
2323
io.k8s.description="$DESCRIPTION" \
2424
io.k8s.display-name="Apache 2.4 with mod_perl/$PERL_VERSION" \
2525
io.openshift.expose-services="8080:http" \
26-
io.openshift.tags="builder,$NAME,$NAME$PERL_SHORT_VER" \
26+
io.openshift.tags="builder,${NAME},${NAME}${PERL_SHORT_VER}" \
2727
io.openshift.s2i.scripts-url="image:///usr/libexec/s2i" \
2828
io.s2i.scripts-url="image:///usr/libexec/s2i" \
29-
name="rhscl/$NAME-$PERL_SHORT_VER-rhel7" \
30-
com.redhat.component="rh-$NAME$PERL_SHORT_VER-docker" \
29+
name="rhscl/${NAME}-${PERL_SHORT_VER}-rhel7" \
30+
com.redhat.component="rh-${NAME}${PERL_SHORT_VER}-docker" \
3131
version="$PERL_VERSION" \
3232
maintainer="SoftwareCollections.org <[email protected]>" \
33-
help="For more information visit https://github.com/sclorg/s2i-$NAME-container" \
34-
usage="s2i build <SOURCE-REPOSITORY> rhscl/$NAME-$PERL_SHORT_VER-rhel7:latest <APP-NAME>"
33+
help="For more information visit https://github.com/sclorg/s2i-${NAME}-container" \
34+
usage="s2i build <SOURCE-REPOSITORY> rhscl/${NAME}-${PERL_SHORT_VER}-rhel7:latest <APP-NAME>"
3535

3636
RUN yum install -y yum-utils && \
3737
yum-config-manager --disable \* &> /dev/null && \

0 commit comments

Comments
 (0)