Skip to content

Commit e0fdb89

Browse files
committed
Address review comments
- add newline at end of rabbitmq.conf - drop rhs filename in Dockerfile COPY - drop $conf variable from shell
1 parent e5cd7f7 commit e0fdb89

File tree

4 files changed

+4
-5
lines changed

4 files changed

+4
-5
lines changed

Dockerfile-alpine.template

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ VOLUME $RABBITMQ_DATA_DIR
270270
# https://docs.docker.com/samples/library/ubuntu/#locales
271271
ENV LANG=C.UTF-8 LANGUAGE=C.UTF-8 LC_ALL=C.UTF-8
272272

273-
COPY rabbitmq.conf /etc/rabbitmq/rabbitmq.conf
273+
COPY rabbitmq.conf /etc/rabbitmq/
274274

275275
COPY docker-entrypoint.sh /usr/local/bin/
276276
ENTRYPOINT ["docker-entrypoint.sh"]

Dockerfile-ubuntu.template

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -288,7 +288,7 @@ VOLUME $RABBITMQ_DATA_DIR
288288
# https://docs.docker.com/samples/library/ubuntu/#locales
289289
ENV LANG=C.UTF-8 LANGUAGE=C.UTF-8 LC_ALL=C.UTF-8
290290

291-
COPY rabbitmq.conf /etc/rabbitmq/rabbitmq.conf
291+
COPY rabbitmq.conf /etc/rabbitmq/
292292

293293
COPY docker-entrypoint.sh /usr/local/bin/
294294
ENTRYPOINT ["docker-entrypoint.sh"]

apply-templates.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ for version; do
4141
gawk -f "$jqt" "Dockerfile-$variant.template"
4242
} > "$version/$variant/Dockerfile"
4343

44-
conf='rabbitmq.conf'
45-
cp -a "$conf" "$version/$variant/rabbitmq.conf"
44+
cp -a "rabbitmq.conf" "$version/$variant/rabbitmq.conf"
4645

4746
entrypoint='docker-entrypoint.sh'
4847
rcVersion="${version%-rc}"

rabbitmq.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@ loopback_users.guest = false
55

66
## DEFAULT SETTINGS ARE NOT MEANT TO BE TAKEN STRAIGHT INTO PRODUCTION
77
## see https://www.rabbitmq.com/configure.html for further information
8-
## on configuring RabbitMQ
8+
## on configuring RabbitMQ

0 commit comments

Comments
 (0)