Change in docker-playground[master]: make/Makefile: allow overriding OSMO_TTCN3_BRANCH

osmith gerrit-no-reply at
Mon Mar 4 12:10:38 UTC 2019

osmith has uploaded this change for review. (

Change subject: make/Makefile: allow overriding OSMO_TTCN3_BRANCH

make/Makefile: allow overriding OSMO_TTCN3_BRANCH

Expose the OSMO_TTCN3_BRANCH build argument from ttcn3-*/Dockerfile as
environment variable, so it can be changed when building the images
with 'make' or './'.

Dockerfiles that do not have a OSMO_TCCN3_BRANCH build arg produce a
warning. But this is also the case with Dockerfiles that don't have an
USER build arg, and avoiding the warning would only be possible with a
much more complex patch.

Change-Id: If2c312ff9206d1613fc19f41bd088c9aad523684
M make/Makefile
2 files changed, 6 insertions(+), 1 deletion(-)

  git pull ssh:// refs/changes/10/13110/1

diff --git a/ b/
index adb6420..4977256 100644
--- a/
+++ b/
@@ -18,6 +18,9 @@
 * `IMAGE_SUFFIX`: the version of the Osmocom stack to run the testsuite
   against. Default is `master`, set this to `latest` to test the last
   stable releases.
+* `OSMO_TTCN3_BRANCH`: [osmo-ttcn3-hacks.git](
+  branch, which will be used when building a `ttcn3-*` docker image.
+  Defaults to `master`.
 * `NO_DOCKER_IMAGE_BUILD`: when set to `1`, it won't try to update the
   containers (see "caching" below)
diff --git a/make/Makefile b/make/Makefile
index 5bbdbb7..2321fcf 100644
--- a/make/Makefile
+++ b/make/Makefile
@@ -16,6 +16,7 @@
 NAME?=$(shell basename $(CURDIR))
 RELEASE_SUPPORT := $(shell dirname $(abspath $(lastword $(MAKEFILE_LIST))))/.make-release-support
@@ -39,7 +40,8 @@
 docker-build: .release
-	docker build --build-arg USER=$(USERNAME) -t $(IMAGE):latest .
+	docker build --build-arg USER=$(USERNAME) --build-arg OSMO_TTCN3_BRANCH=$(OSMO_TTCN3_BRANCH) \
+		-t $(IMAGE):latest .
 	@DOCKER_MAJOR=$(shell docker -v | sed -e 's/.*version //' -e 's/,.*//' | cut -d\. -f1) ; \
 	DOCKER_MINOR=$(shell docker -v | sed -e 's/.*version //' -e 's/,.*//' | cut -d\. -f2) ; \

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If2c312ff9206d1613fc19f41bd088c9aad523684
Gerrit-Change-Number: 13110
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list