<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:0 0 0 .8ex;border-left:1px #ccc solid;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">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">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">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/<wbr>issues/769</a><br>
>> ><br>
>> > On Mon, Jan 9, 2017 at 9:14 PM, Vadim Chugunov <<a href="mailto:vadimcn@gmail.com">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\<wbr>lldb<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">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">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">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">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">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/<wbr>D27476</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">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/<wbr>D27476</a><br>
>> >>> >>>>><br>
>> >>> >>>>> On Mon, Dec 5, 2016 at 12:04 PM, Hans Wennborg<br>
>> >>> >>>>> <<a href="mailto:hans@chromium.org">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/<wbr>v3.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">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">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>