<div dir="ltr">Mishal,<div><br></div><div>Reverted here:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div>commit 02a493b8380bbcb5c4bf2e123db0b6d79c95dc8f (HEAD -&gt; master, origin/master, origin/HEAD)</div></div><div><div>Author: Todd Fiala &lt;<a href="mailto:tfiala@apple.com">tfiala@apple.com</a>&gt;</div></div><div><div>Date:   Thu Dec 3 22:57:16 2015</div></div></blockquote><div><br></div><div>You should be back in business.</div><div><br></div><div>I&#39;ll re-introduce this change a more appropriate way in the morning.</div><div><br></div><div>-Todd</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 3, 2015 at 10:56 PM, Todd Fiala <span dir="ltr">&lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Okay I think I see the issue.  It is that we can still pass backdoor combinations of build modes after the &#39;--&#39; in build-script that will control the final build variant used by lldb.<div><br></div><div>The logic for this check doesn&#39;t belong in build-script, but rather build-script-impl.  build-script-impl is the only place to find the final build mode after considering both the values passed to build-script, and other bits that can be passed along to build-script-impl.</div><div><br></div><div>I&#39;m going to revert my change and add a proper fix in the morning.  I don&#39;t have enough brain power to do it right and test it now.</div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Thu, Dec 3, 2015 at 10:51 PM, Todd Fiala <span dir="ltr">&lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yep I finally added enough gunk to the command line to reproduce.<div><br></div><div>I&#39;m looking at how it&#39;s getting the Release value now.  That&#39;s the issue (I&#39;m not seeing it come in as Release).</div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Thu, Dec 3, 2015 at 10:44 PM, Dmitri Gribenko <span dir="ltr">&lt;<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Thu, Dec 3, 2015 at 10:41 PM, Todd Fiala &lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt; wrote:<br>
&gt; I see.<br>
&gt;<br>
&gt; And I think I see the issue.  It&#39;s trying to do a release-debuginfo build<br>
&gt; across the board.  LLDB chokes on those.  So we&#39;ll need to explicitly set<br>
&gt; the lldb build variant to something different (either release or debug,<br>
&gt; likely release).<br>
<br>
</span>The preset is trying to do that (lldb-build-type=Release), but<br>
apparently is getting ignored.<br>
<div><div><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j&lt;i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf(&quot;%d\n&quot;,i);}}} /*Dmitri Gribenko &lt;<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>&gt;*/<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>