symposion_app Changeset - c8218bb9d0ea

Changeset - c8218bb9d0ea

Parent rev.

Child rev.

[Not reviewed]

0 1 0

James Polley - 6 years ago 2018-06-06 02:21:26
jp@jamezpolley.com

Correct variable name to reflect what it's naming

IT's not the container name, it's the image name. Fix the name to be
less confusing.

1 file changed with 3 insertions and 3 deletions:

make_dev_container.sh

0 comments (0 inline, 0 general)

make_dev_container.sh

➞

Show inline comments

 #!/bin/bash -x
-CONTAINER_NAME=${1:-symposion_app}
+IMAGE_NAME=${1:-symposion_app}
 if [ -e ./symposion-tools ]; then
     pushd ./symposion-tools
@@ ... / @@ -8,10 +8,10 @@ if [ -e ./symposion-tools ]; then @@
     popd
 fi
-docker image build -f docker/Dockerfile -t ${CONTAINER_NAME} --target symposion_dev .
+docker image build -f docker/Dockerfile -t ${IMAGE_NAME} --target symposion_dev .
 docker container stop symposion
 docker container rm symposion
-docker container create --env-file docker/laptop-mode-env -p 28000:8000 -v $(pwd):/app/symposion_app --name symposion ${CONTAINER_NAME}
+docker container create --env-file docker/laptop-mode-env -p 28000:8000 -v $(pwd):/app/symposion_app --name symposion ${IMAGE_NAME}
 docker container start symposion
 ## When we started the container and mounted . into /app/symposion_app, it hides the static/build directory
 ## As a kludge, re-run collectstatic to recreate it

0 comments (0 inline, 0 general)