<p style="white-space: pre-wrap; word-wrap: break-word;">posted my nitpicks but you can also just fix the typo and merge it.<br>don't spend too much time on it.</p><p><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/README">File README:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/README@43">Patch Set #2, Line 43:</a> <code style="font-family:monospace,monospace">Run ./gen_makefile.py with a choice of choice of configure options.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">meta choice</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/gen_makefile.py">File gen_makefile.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/gen_makefile.py@248">Patch Set #2, Line 248:</a> <code style="font-family:monospace,monospace">    continue</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">i kind of like to keep that explicit .deps argument; now you did all the work to remove it. "explicit is better than implicit"</p><p style="white-space: pre-wrap; word-wrap: break-word;">well it doesn't really matter either way, will have zero impact on my daily work.<br>it's just a personal preference of mine to allow a .deps file arg</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/gen_makefile.py@255">Patch Set #2, Line 255:</a> <code style="font-family:monospace,monospace">  make_dir = 'make-3G+2G-%s' % opts_names</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">let's just drop '-3G+2G'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233/2/gen_makefile.py@277">Patch Set #2, Line 277:</a> <code style="font-family:monospace,monospace">default: usrp</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe as default just print a listing of existing meaningful targets?<br>because if you like usrp and i like cn, we can't agree.<br>that was the advantage of the custom .deps file, 'make all' would build your choice.</p><p style="white-space: pre-wrap; word-wrap: break-word;">we could pass a favorite default target to gen_makefile.py but IMO not worth the trouble</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-dev/+/25233">change 25233</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/osmo-dev/+/25233"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-dev </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6394882f67eecda3d2c03a97d3119657f7d3873f </div>
<div style="display:none"> Gerrit-Change-Number: 25233 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 21 Sep 2021 10:28:02 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>