<div dir="ltr">Thanks!<div><br></div><div>Would anyone be so kind to commit that?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 4, 2017 at 11:47 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sorry, a combination of national holidays and extended vacations happened and this fell off my radar.  lgtm</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 4, 2017 at 11:46 AM Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="m_8292166423998933535gmail_msg">Zachary, <div class="m_8292166423998933535gmail_msg">Can you please take a look at that change? (<a href="https://reviews.llvm.org/D27476" class="m_8292166423998933535gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D27476</a>)</div><div class="m_8292166423998933535gmail_msg"><br class="m_8292166423998933535gmail_msg"></div><div class="m_8292166423998933535gmail_msg">It'll be sad if another snapshot build gets published with broken lldb. :(</div><div class="m_8292166423998933535gmail_msg"><br class="m_8292166423998933535gmail_msg"></div><div class="m_8292166423998933535gmail_msg"><br class="m_8292166423998933535gmail_msg"></div><div class="gmail_extra m_8292166423998933535gmail_msg"><div class="gmail_quote m_8292166423998933535gmail_msg">On Tue, Dec 6, 2016 at 11:54 AM, Vadim Chugunov <span dir="ltr" class="m_8292166423998933535gmail_msg"><<a href="mailto:vadimcn@gmail.com" class="m_8292166423998933535gmail_msg" target="_blank">vadimcn@gmail.com</a>></span> wrote:<br class="m_8292166423998933535gmail_msg"><blockquote class="gmail_quote m_8292166423998933535gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class="m_8292166423998933535gmail_msg">This seems to work: <a href="https://reviews.llvm.org/D27476" class="m_8292166423998933535gmail_msg" target="_blank">https://reviews.llvm.<wbr>org/D27476</a><br class="m_8292166423998933535gmail_msg"></div><div class="gmail_extra m_8292166423998933535gmail_msg"><br class="m_8292166423998933535gmail_msg"><div class="gmail_quote m_8292166423998933535gmail_msg">On Mon, Dec 5, 2016 at 12:04 PM, Hans Wennborg <span dir="ltr" class="m_8292166423998933535gmail_msg"><<a href="mailto:hans@chromium.org" class="m_8292166423998933535gmail_msg" target="_blank">hans@chromium.org</a>></span> wrote:<br class="m_8292166423998933535gmail_msg"><blockquote class="gmail_quote m_8292166423998933535gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The only thing needed to build the installer should be having NSIS<br class="m_8292166423998933535gmail_msg">
installed and building the "package" target generated by CMake. The<br class="m_8292166423998933535gmail_msg">
other prerequisites are mostly for building the visual studio<br class="m_8292166423998933535gmail_msg">
clang-format plugin.<br class="m_8292166423998933535gmail_msg">
<br class="m_8292166423998933535gmail_msg">
Having said that, you don't even have to build the installer to see<br class="m_8292166423998933535gmail_msg">
what goes in it. Just building the "install" target generated by CMake<br class="m_8292166423998933535gmail_msg">
will install the same set of files.<br class="m_8292166423998933535gmail_msg">
<br class="m_8292166423998933535gmail_msg">
I'm not sure how LLDB's cmake files are organized, but in the end<br class="m_8292166423998933535gmail_msg">
what's required is invoking the install() command:<br class="m_8292166423998933535gmail_msg">
<a href="https://cmake.org/cmake/help/v3.0/command/install.html" rel="noreferrer" class="m_8292166423998933535gmail_msg" target="_blank">https://cmake.org/cmake/help/<wbr>v3.0/command/install.html</a>  In LLVM, this<br class="m_8292166423998933535gmail_msg">
is done automatically by macros such as add_llvm_executale, etc.<br class="m_8292166423998933535gmail_msg">
<br class="m_8292166423998933535gmail_msg">
On Mon, Dec 5, 2016 at 11:56 AM, Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" class="m_8292166423998933535gmail_msg" target="_blank">vadimcn@gmail.com</a>> wrote:<br class="m_8292166423998933535gmail_msg">
> Hi Hans,<br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
> I'd love to help, but I don't have half the tools that<br class="m_8292166423998933535gmail_msg">
> build_llvm_package.bat requires installed on my machine.  My setup is to<br class="m_8292166423998933535gmail_msg">
> build llvm with msbuild.   Is it possible to build the installer this way<br class="m_8292166423998933535gmail_msg">
> too?<br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
> Can you point me to the specific CMake source that determines what's<br class="m_8292166423998933535gmail_msg">
> included in the package?   At a glance, everything from<br class="m_8292166423998933535gmail_msg">
> %LLVM%/lib/site-packages is missing.<br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
> Vadim<br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
> On Mon, Dec 5, 2016 at 10:41 AM, Hans Wennborg <<a href="mailto:hans@chromium.org" class="m_8292166423998933535gmail_msg" target="_blank">hans@chromium.org</a>> wrote:<br class="m_8292166423998933535gmail_msg">
>><br class="m_8292166423998933535gmail_msg">
>> Is anyone working on this?<br class="m_8292166423998933535gmail_msg">
>><br class="m_8292166423998933535gmail_msg">
>> I'm happy to include LLDB in the installer, but I'm really not the<br class="m_8292166423998933535gmail_msg">
>> best person to be debugging it.<br class="m_8292166423998933535gmail_msg">
>><br class="m_8292166423998933535gmail_msg">
>> If more files need to be included in the install, that's configured in<br class="m_8292166423998933535gmail_msg">
>> the CMake files (what's installed by the 'install' build target is<br class="m_8292166423998933535gmail_msg">
>> also what ends up going into the installer). If it needs more build<br class="m_8292166423998933535gmail_msg">
>> flags, patches to build_llvm_package.bat are welsome.<br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
><br class="m_8292166423998933535gmail_msg">
</blockquote></div><br class="m_8292166423998933535gmail_msg"></div>
</blockquote></div><br class="m_8292166423998933535gmail_msg"></div></div>
</blockquote></div>
</blockquote></div><br></div>