<p>osmith <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-dev/+/22596">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved; Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gen_makefile.py: regen: put args in own lines<br><br>Make it easier to adjust opts before running 'make regen'.<br><br>Change-Id: I3c537ca9cd876e62547efca1ea40f31e90d8761e<br>---<br>M gen_makefile.py<br>1 file changed, 14 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/gen_makefile.py b/gen_makefile.py</span><br><span>index f76ebb0..ad42978 100755</span><br><span>--- a/gen_makefile.py</span><br><span>+++ b/gen_makefile.py</span><br><span>@@ -276,22 +276,29 @@</span><br><span> # regenerate this Makefile, in case the deps or opts changed</span><br><span> .PHONY: regen</span><br><span> regen:</span><br><span style="color: hsl(0, 100%, 40%);">-      {script} {projects_and_deps} {configure_opts} -m {make_dir} -o {makefile} -s {src_dir} -b {build_dir} -u "{url}"{push_url}{sudo_make_install}{no_ldconfig}{ldconfig_without_sudo}{make_check}</span><br><span style="color: hsl(120, 100%, 40%);">+       {script} \</span><br><span style="color: hsl(120, 100%, 40%);">+            {projects_and_deps} \</span><br><span style="color: hsl(120, 100%, 40%);">+         {configure_opts} \</span><br><span style="color: hsl(120, 100%, 40%);">+            -m {make_dir} \</span><br><span style="color: hsl(120, 100%, 40%);">+               -o {makefile} \</span><br><span style="color: hsl(120, 100%, 40%);">+               -s {src_dir} \</span><br><span style="color: hsl(120, 100%, 40%);">+                -b {build_dir} \</span><br><span style="color: hsl(120, 100%, 40%);">+              -u "{url}"{push_url}{sudo_make_install}{no_ldconfig}{ldconfig_without_sudo}{make_check}</span><br><span> </span><br><span> '''.format(</span><br><span>     script=os.path.relpath(sys.argv[0], make_dir),</span><br><span>     projects_and_deps=os.path.relpath(args.projects_and_deps_file, make_dir),</span><br><span style="color: hsl(0, 100%, 40%);">-    configure_opts=' '.join([os.path.relpath(f, make_dir) for f in configure_opts_files]),</span><br><span style="color: hsl(120, 100%, 40%);">+    configure_opts=' \\\n\t\t'.join([os.path.relpath(f, make_dir) for f in configure_opts_files]),</span><br><span>     make_dir='.',</span><br><span>     makefile=args.output,</span><br><span>     src_dir=os.path.relpath(args.src_dir, make_dir),</span><br><span>     build_dir=os.path.relpath(build_dir, make_dir),</span><br><span>     url=args.url,</span><br><span style="color: hsl(0, 100%, 40%);">-    push_url=(" -p '%s'"%args.push_url) if args.push_url else '',</span><br><span style="color: hsl(0, 100%, 40%);">-    sudo_make_install=' -I' if args.sudo_make_install else '',</span><br><span style="color: hsl(0, 100%, 40%);">-    no_ldconfig=' -L' if args.no_ldconfig else '',</span><br><span style="color: hsl(0, 100%, 40%);">-    ldconfig_without_sudo=' --ldconfig-without-sudo' if args.ldconfig_without_sudo else '',</span><br><span style="color: hsl(0, 100%, 40%);">-    make_check='' if args.make_check else " --no-make-check",</span><br><span style="color: hsl(120, 100%, 40%);">+    push_url=(" \\\n\t\t-p '%s'"%args.push_url) if args.push_url else '',</span><br><span style="color: hsl(120, 100%, 40%);">+    sudo_make_install=' \\\n\t\t-I' if args.sudo_make_install else '',</span><br><span style="color: hsl(120, 100%, 40%);">+    no_ldconfig=' \\\n\t\t-L' if args.no_ldconfig else '',</span><br><span style="color: hsl(120, 100%, 40%);">+    ldconfig_without_sudo=' \\\n\t\t--ldconfig-without-sudo' if args.ldconfig_without_sudo else '',</span><br><span style="color: hsl(120, 100%, 40%);">+    make_check='' if args.make_check else " \\\n\t\t--no-make-check",</span><br><span>     ))</span><br><span> </span><br><span>   # convenience target: clone all repositories first</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-dev/+/22596">change 22596</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/+/22596"/><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: I3c537ca9cd876e62547efca1ea40f31e90d8761e </div>
<div style="display:none"> Gerrit-Change-Number: 22596 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>