<div dir="ltr">Thanks! <div><br></div><div>Complete command: </div><div><br></div><div><pre class="" style="white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(0,0,0);font-size:11px">swift/utils/build-script --preset=buildbot_osx_package install_destdir=/tmp/swift-install installable_package=/tmp/swift.tar.gz install_toolchain_dir=/Applications/Xcode.app/Contents/Developer/Toolchains/swift.xctoolchain install_symroot=/tmp/swift-symroot symbols_package=/tmp/swift-symbols.tar.gz darwin_toolchain_bundle_identifier=com.test.x &#39;darwin_toolchain_display_name=test&#39; darwin_toolchain_xctoolchain_name=swift darwin_toolchain_version=swift</pre><pre class="" style="white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(0,0,0);font-size:11px"><br></pre></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Mishal Shah</div></div>
<br><div class="gmail_quote">On Thu, Dec 3, 2015 at 11:01 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">Sure thing, sorry for the bother.</div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Thu, Dec 3, 2015 at 11:00 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">Thanks, Todd!<br>
<div><div><br>
On Thu, Dec 3, 2015 at 11:00 PM, Todd Fiala &lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt; wrote:<br>
&gt; Mishal,<br>
&gt;<br>
&gt; Reverted here:<br>
&gt;<br>
&gt; commit 02a493b8380bbcb5c4bf2e123db0b6d79c95dc8f (HEAD -&gt; master,<br>
&gt; origin/master, origin/HEAD)<br>
&gt; Author: Todd Fiala &lt;<a href="mailto:tfiala@apple.com" target="_blank">tfiala@apple.com</a>&gt;<br>
&gt; Date:   Thu Dec 3 22:57:16 2015<br>
&gt;<br>
&gt;<br>
&gt; You should be back in business.<br>
&gt;<br>
&gt; I&#39;ll re-introduce this change a more appropriate way in the morning.<br>
&gt;<br>
&gt; -Todd<br>
&gt;<br>
&gt;<br>
&gt; On Thu, Dec 3, 2015 at 10:56 PM, Todd Fiala &lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Okay I think I see the issue.  It is that we can still pass backdoor<br>
&gt;&gt; combinations of build modes after the &#39;--&#39; in build-script that will control<br>
&gt;&gt; the final build variant used by lldb.<br>
&gt;&gt;<br>
&gt;&gt; The logic for this check doesn&#39;t belong in build-script, but rather<br>
&gt;&gt; build-script-impl.  build-script-impl is the only place to find the final<br>
&gt;&gt; build mode after considering both the values passed to build-script, and<br>
&gt;&gt; other bits that can be passed along to build-script-impl.<br>
&gt;&gt;<br>
&gt;&gt; I&#39;m going to revert my change and add a proper fix in the morning.  I<br>
&gt;&gt; don&#39;t have enough brain power to do it right and test it now.<br>
&gt;&gt;<br>
&gt;&gt; On Thu, Dec 3, 2015 at 10:51 PM, Todd Fiala &lt;<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Yep I finally added enough gunk to the command line to reproduce.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I&#39;m looking at how it&#39;s getting the Release value now.  That&#39;s the issue<br>
&gt;&gt;&gt; (I&#39;m not seeing it come in as Release).<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Thu, Dec 3, 2015 at 10:44 PM, Dmitri Gribenko &lt;<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>&gt;<br>
&gt;&gt;&gt; wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; 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;<br>
&gt;&gt;&gt;&gt; wrote:<br>
&gt;&gt;&gt;&gt; &gt; I see.<br>
&gt;&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt;&gt; &gt; And I think I see the issue.  It&#39;s trying to do a release-debuginfo<br>
&gt;&gt;&gt;&gt; &gt; build<br>
&gt;&gt;&gt;&gt; &gt; across the board.  LLDB chokes on those.  So we&#39;ll need to explicitly<br>
&gt;&gt;&gt;&gt; &gt; set<br>
&gt;&gt;&gt;&gt; &gt; the lldb build variant to something different (either release or<br>
&gt;&gt;&gt;&gt; &gt; debug,<br>
&gt;&gt;&gt;&gt; &gt; likely release).<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; The preset is trying to do that (lldb-build-type=Release), but<br>
&gt;&gt;&gt;&gt; apparently is getting ignored.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Dmitri<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; --<br>
&gt;&gt;&gt;&gt; main(i,j){for(i=2;;i++){for(j=2;j&lt;i;j++){if(!(i%j)){j=0;break;}}if<br>
&gt;&gt;&gt;&gt; (j){printf(&quot;%d\n&quot;,i);}}} /*Dmitri Gribenko &lt;<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>&gt;*/<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; --<br>
&gt;&gt;&gt; -Todd<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; -Todd<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; -Todd<br>
<br>
<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 class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br></div>