Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

Commit cde53a1

Browse files
author
Rebecca Graber
authored
fix: remove periods from project name when release version is present (#1252)
1 parent a43aa5e commit cde53a1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

options.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ DEVSTACK_WORKSPACE ?= $(shell pwd)/..
3030
# so changing it will give you a separate set of databases.
3131
# See https://docs.docker.com/compose/reference/envvars/#compose_project_name
3232
# If OPENEDX_RELAESE is defined, defaults to `devstack-${OPENEDX_RELEASE}`;
33-
# otherwise, it defaults to `devstack`.
33+
# otherwise, it defaults to `devstack`. Any periods will be replaced with hyphens to comply with docker project naming rules (eg `devstack-quince.master` will become `devstack-quince-master`).
3434
# Be sure to bring down services before changing the value of `COMPOSE_PROJECT_NAME`.
3535
ifdef OPENEDX_RELEASE
36-
COMPOSE_PROJECT_NAME ?= devstack-${OPENEDX_RELEASE}
36+
COMPOSE_PROJECT_NAME ?= devstack-$(echo ${OPENEDX_RELEASE} | tr . -)
3737
else
3838
COMPOSE_PROJECT_NAME ?= devstack
3939
endif

0 commit comments

Comments
 (0)