<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">wouldnt it have made sense to specify a default value for the variable, so that building withou tany explicit build-arg still works?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The default value is in Makefile.am, so if you build with make it works out of the box. Default value in Dockerfile is only needed if one attempts to build directly from it. I can add it to each Dockerfile if you prefer, but I thought it was going to be confusing since anyway that value is not used because a default value is pssed through Makefile.</p><p><a href="https://gerrit.osmocom.org/c/docker-playground/+/21239">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/docker-playground/+/21239">change 21239</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/docker-playground/+/21239"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: docker-playground </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8010b08f3dabacfb3c13a44eece6c7a490e0742e </div>
<div style="display:none"> Gerrit-Change-Number: 21239 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 18 Nov 2020 21:02:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>