<div dir="ltr">Sorry, just found another problem: the installed lldb crashes when given a script via the command line.  For example, `lldb -O "p 42"` dies with exception 0xC0000409.<div><div><div><br><div>It didn't happen with the one I've build locally, so I started digging... The difference seems to be that build_llvm_build_package.bat links CRT statically (-DLLVM_USE_CRT_RELEASE=MT), whereas default is the dynamic CRT.  </div><div>The crash kinda makes sense, because python35.dll links CRT dynamically, so LLDB ends up with two copies of it in the process, which is known to cause all sorts of trouble.</div><div><br></div><div>Not sure what to do about this one, - you probably wanted static CRT to avoid having to install MSVC redistributable?</div><div><br></div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 10, 2017 at 7:00 PM, Vadim Chugunov <span dir="ltr"><<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Yes, the new build works!</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 10, 2017 at 6:20 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I've downgraded my swig to 3.0.8 and built a new snapshot (r291454).<br>
Please let me know if that works.<br>
<br>
On Tue, Jan 10, 2017 at 10:14 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> It sounds like the solution to the problem is to downgrade SWIG on the build<br>
> machine.  If it's using version 3.0.9 or higher, just use whatever the last<br>
> version before that is.  3.0.8, for example.<br>
><br>
> At least there's a good workaround in the interim (i.e. setting PYTHONPATH)<br>
><br>
> On Tue, Jan 10, 2017 at 10:06 AM Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>><br>
>> I'll do another snapshot maybe next week or the week after. You can<br>
>> also ping me if you want it sooner or later.<br>
>><br>
>> We're kicking off the release process for 4.0.0 on Thursday. I don't<br>
>> fully understand the problem here, but if there's some way to work<br>
>> around it and get lldb into good shape for the 4.0.0 installer, that<br>
>> would be great.<br>
>><br>
>> Thanks,<br>
>> Hans<br>
>><br>
>> On Mon, Jan 9, 2017 at 10:40 PM, Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>> wrote:<br>
>> > This appears to be a SWIG bug: <a href="https://github.com/swig/swig/issues/769" rel="noreferrer" target="_blank">https://github.com/swig/swig/i<wbr>ssues/769</a><br>
>> ><br>
>> > On Mon, Jan 9, 2017 at 9:14 PM, Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> It worked!<br>
>> >><br>
>> >> ...but not before I set PYTHONPATH=C:\Program Files<br>
>> >> (x86)\LLVM\lib\site-packages\l<wbr>ldb<br>
>> >> Without that, it couldn't find the _lldb module, so we are not quite<br>
>> >> out<br>
>> >> of the woods yet.<br>
>> >><br>
>> >> When are you planning to make the next snapshot build?<br>
>> >> Thanks!<br>
>> >><br>
>> >><br>
>> >> On Mon, Jan 9, 2017 at 3:48 PM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Vadim, it looks like your change was committed in r291291, and I've<br>
>> >>> built a new snapshot today which includes it. Can you give it a try<br>
>> >>> and see if everything works?<br>
>> >>><br>
>> >>> Cheers,<br>
>> >>> Hans<br>
>> >>><br>
>> >>> On Thu, Jan 5, 2017 at 10:46 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>><br>
>> >>> wrote:<br>
>> >>> > I will commit it, in the meantime can you request commit access so<br>
>> >>> > that<br>
>> >>> > any<br>
>> >>> > future patches you can commit?<br>
>> >>> ><br>
>> >>> > On Wed, Jan 4, 2017 at 1:54 PM Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>><br>
>> >>> > wrote:<br>
>> >>> >><br>
>> >>> >> Thanks!<br>
>> >>> >><br>
>> >>> >> Would anyone be so kind to commit that?<br>
>> >>> >><br>
>> >>> >> On Wed, Jan 4, 2017 at 11:47 AM, Zachary Turner<br>
>> >>> >> <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>><br>
>> >>> >> wrote:<br>
>> >>> >>><br>
>> >>> >>> Sorry, a combination of national holidays and extended vacations<br>
>> >>> >>> happened<br>
>> >>> >>> and this fell off my radar.  lgtm<br>
>> >>> >>><br>
>> >>> >>> On Wed, Jan 4, 2017 at 11:46 AM Vadim Chugunov <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>><br>
>> >>> >>> wrote:<br>
>> >>> >>>><br>
>> >>> >>>> Zachary,<br>
>> >>> >>>> Can you please take a look at that change?<br>
>> >>> >>>> (<a href="https://reviews.llvm.org/D27476" rel="noreferrer" target="_blank">https://reviews.llvm.org/D274<wbr>76</a>)<br>
>> >>> >>>><br>
>> >>> >>>> It'll be sad if another snapshot build gets published with broken<br>
>> >>> >>>> lldb.<br>
>> >>> >>>> :(<br>
>> >>> >>>><br>
>> >>> >>>><br>
>> >>> >>>> On Tue, Dec 6, 2016 at 11:54 AM, Vadim Chugunov<br>
>> >>> >>>> <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>><br>
>> >>> >>>> wrote:<br>
>> >>> >>>>><br>
>> >>> >>>>> This seems to work: <a href="https://reviews.llvm.org/D27476" rel="noreferrer" target="_blank">https://reviews.llvm.org/D2747<wbr>6</a><br>
>> >>> >>>>><br>
>> >>> >>>>> On Mon, Dec 5, 2016 at 12:04 PM, Hans Wennborg<br>
>> >>> >>>>> <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>><br>
>> >>> >>>>> wrote:<br>
>> >>> >>>>>><br>
>> >>> >>>>>> The only thing needed to build the installer should be having<br>
>> >>> >>>>>> NSIS<br>
>> >>> >>>>>> installed and building the "package" target generated by CMake.<br>
>> >>> >>>>>> The<br>
>> >>> >>>>>> other prerequisites are mostly for building the visual studio<br>
>> >>> >>>>>> clang-format plugin.<br>
>> >>> >>>>>><br>
>> >>> >>>>>> Having said that, you don't even have to build the installer to<br>
>> >>> >>>>>> see<br>
>> >>> >>>>>> what goes in it. Just building the "install" target generated<br>
>> >>> >>>>>> by<br>
>> >>> >>>>>> CMake<br>
>> >>> >>>>>> will install the same set of files.<br>
>> >>> >>>>>><br>
>> >>> >>>>>> I'm not sure how LLDB's cmake files are organized, but in the<br>
>> >>> >>>>>> end<br>
>> >>> >>>>>> what's required is invoking the install() command:<br>
>> >>> >>>>>> <a href="https://cmake.org/cmake/help/v3.0/command/install.html" rel="noreferrer" target="_blank">https://cmake.org/cmake/help/v<wbr>3.0/command/install.html</a>  In<br>
>> >>> >>>>>> LLVM,<br>
>> >>> >>>>>> this<br>
>> >>> >>>>>> is done automatically by macros such as add_llvm_executale,<br>
>> >>> >>>>>> etc.<br>
>> >>> >>>>>><br>
>> >>> >>>>>> On Mon, Dec 5, 2016 at 11:56 AM, Vadim Chugunov<br>
>> >>> >>>>>> <<a href="mailto:vadimcn@gmail.com" target="_blank">vadimcn@gmail.com</a>><br>
>> >>> >>>>>> wrote:<br>
>> >>> >>>>>> > Hi Hans,<br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> > I'd love to help, but I don't have half the tools that<br>
>> >>> >>>>>> > build_llvm_package.bat requires installed on my machine.  My<br>
>> >>> >>>>>> > setup<br>
>> >>> >>>>>> > is to<br>
>> >>> >>>>>> > build llvm with msbuild.   Is it possible to build the<br>
>> >>> >>>>>> > installer<br>
>> >>> >>>>>> > this way<br>
>> >>> >>>>>> > too?<br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> > Can you point me to the specific CMake source that determines<br>
>> >>> >>>>>> > what's<br>
>> >>> >>>>>> > included in the package?   At a glance, everything from<br>
>> >>> >>>>>> > %LLVM%/lib/site-packages is missing.<br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> > Vadim<br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> > On Mon, Dec 5, 2016 at 10:41 AM, Hans Wennborg<br>
>> >>> >>>>>> > <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>><br>
>> >>> >>>>>> > wrote:<br>
>> >>> >>>>>> >><br>
>> >>> >>>>>> >> Is anyone working on this?<br>
>> >>> >>>>>> >><br>
>> >>> >>>>>> >> I'm happy to include LLDB in the installer, but I'm really<br>
>> >>> >>>>>> >> not<br>
>> >>> >>>>>> >> the<br>
>> >>> >>>>>> >> best person to be debugging it.<br>
>> >>> >>>>>> >><br>
>> >>> >>>>>> >> If more files need to be included in the install, that's<br>
>> >>> >>>>>> >> configured<br>
>> >>> >>>>>> >> in<br>
>> >>> >>>>>> >> the CMake files (what's installed by the 'install' build<br>
>> >>> >>>>>> >> target<br>
>> >>> >>>>>> >> is<br>
>> >>> >>>>>> >> also what ends up going into the installer). If it needs<br>
>> >>> >>>>>> >> more<br>
>> >>> >>>>>> >> build<br>
>> >>> >>>>>> >> flags, patches to build_llvm_package.bat are welsome.<br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> ><br>
>> >>> >>>>>> ><br>
>> >>> >>>>><br>
>> >>> >>>>><br>
>> >>> >>>><br>
>> >>> >><br>
>> >>> ><br>
>> >><br>
>> >><br>
>> ><br>
</blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></div></div>