Is anyone working on this? I'm happy to include LLDB in the installer, but I'm really not the best person to be debugging it.
If more files need to be included in the install, that's configured in the CMake files (what's installed by the 'install' build target is also what ends up going into the installer). If it needs more build flags, patches to build_llvm_package.bat are welsome. Thanks, Hans On Mon, Nov 28, 2016 at 10:17 AM, Zachary Turner <ztur...@google.com> wrote: > I overlooked that part of it, but yes that is another separate issue. (BTW, > _lldb.pyd is simply a symlink to liblldb.dll). > > In any case, yea I think the entire lib/site-packages folder needs to be > included. > > On Mon, Nov 28, 2016 at 10:15 AM Vadim Chugunov <vadi...@gmail.com> wrote: >> >> Please correct me if I'm wrong, but isn't the issue here that LLDB's >> Python support files don't get packaged into the Windows installer? Does >> packaging them somehow depend on knowing what the Python installation path >> is? >> >> On Mon, Nov 28, 2016 at 10:09 AM, Hans Wennborg <h...@chromium.org> wrote: >>> >>> The snapshots are built with the script in >>> utils/release/build_llvm_package.bat. It's currently passing >>> -DLLDB_RELOCATABLE_PYTHON=1 and -DPYTHON_HOME=<path>. >>> >>> I was planning on trying to build a new snapshot today and can add >>> -DLLDB_DEFAULT_PYTHON_HOME if you think that will help. >>> >>> On Mon, Nov 28, 2016 at 9:51 AM, Zachary Turner <ztur...@google.com> >>> wrote: >>> > So it sounds like you're saying that in order for Python support to >>> > work as >>> > part of an LLDB shipped in the installer, we need to do set 3 variables >>> > at >>> > CMake time. >>> > >>> > 1) -DLLDB_RELOCATABLE_PYTHON=TRUE >>> > 2) -DPYTHON_HOME = <path-to-python-3-5-on-build-machine> >>> > 3) -DLLDB_DEFAULT_PYTHON_HOME=TRUE >>> > >>> > Now because of #3, the lldb shipped in the installer will use the >>> > PYTHONHOME >>> > system environment variable to locate python, which must point to a >>> > valid >>> > Python 3.5 installation. Is this correct? >>> > >>> > On Mon, Nov 28, 2016 at 9:35 AM Ted Woodward >>> > <ted.woodw...@codeaurora.org> >>> > wrote: >>> >> >>> >> Windows has no concept of a default python installation, and I can’t >>> >> be >>> >> sure what version of python my users have, if any, so I need to solve >>> >> 2 >>> >> problems: >>> >> >>> >> 1) Where is python when I’m building? >>> >> >>> >> 2) Where is python when I’m running? >>> >> >>> >> >>> >> >>> >> To solve #1, I set LLDB_RELOCATABLE_PYTHON to TRUE, and PYTHON_HOME to >>> >> my >>> >> python installation (on our buildbots, c:/python351). >>> >> >>> >> >>> >> >>> >> #2 only needs to be solved if the machine you’re running on doesn’t >>> >> have >>> >> the same python installation, in PYTHON_HOME above. To do that, I’ve >>> >> added >>> >> code to set a cmake path LLDB_DEFAULT_PYTHONHOME, which I pass as a >>> >> macro >>> >> down to InitializePythonHome in >>> >> source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp, >>> >> and >>> >> call Py_SetPythonHome with it. My installations have the python dll >>> >> and >>> >> python library directory. We put the library in <install>/lib/python35 >>> >> and >>> >> the dll in <install>/bin. >>> >> >>> >> -- >>> >> >>> >> Qualcomm Innovation Center, Inc. >>> >> >>> >> The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, >>> >> a >>> >> Linux Foundation Collaborative Project >>> >> >>> >> >>> >> >>> >> From: Zachary Turner [mailto:ztur...@google.com] >>> >> Sent: Wednesday, November 23, 2016 12:40 PM >>> >> To: Vadim Chugunov <vadi...@gmail.com> >>> >> Cc: Reid Kleckner <r...@google.com>; Hans Wennborg <h...@chromium.org>; >>> >> LLDB <lldb-dev@lists.llvm.org>; Ted Woodward >>> >> <ted.woodw...@codeaurora.org> >>> >> Subject: Re: [lldb-dev] lldb-3.8.1 prebuilt binary for windows7 >>> >> >>> >> >>> >> >>> >> I believe the way to fix this is going to be building LLDB for the >>> >> installer with LLDB_RELOCATABLE_PYTHON=1 at CMake time >>> >> >>> >> >>> >> >>> >> +Ted, since I believe he is one of the few people currently using this >>> >> flag. >>> >> >>> >> On Wed, Nov 23, 2016 at 10:36 AM Vadim Chugunov <vadi...@gmail.com> >>> >> wrote: >>> >> >>> >> This is still broken in the October snapshot. Do you know which >>> >> script >>> >> is used to build the Windows installer? >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 6:24 PM, Zachary Turner <ztur...@google.com> >>> >> wrote: >>> >> >>> >> I think it is a problem with the way we built lldb. I will look into >>> >> what >>> >> additional steps we need to take when making the prebuilt binary so >>> >> that it >>> >> works next time. >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 6:20 PM Vadim Chugunov <vadi...@gmail.com> >>> >> wrote: >>> >> >>> >> Nope, that didn't help. >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 5:16 PM, Zachary Turner <ztur...@google.com> >>> >> wrote: >>> >> >>> >> I may know what this is. Can you try setting PYTHONPATH though to >>> >> point >>> >> to your Python 3.5 installation though and see if it fixes it? (I >>> >> don't >>> >> think it will, but let's try anyway) >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 4:59 PM Vadim Chugunov <vadi...@gmail.com> >>> >> wrote: >>> >> >>> >> It outputs 'c:\Program Files (x86)\LLVM\lib\site-packages', however >>> >> the >>> >> 'site-packages' directory does not exist. Nor do I see '_lldb.pyd' >>> >> anywhere >>> >> else. >>> >> >>> >> 'script import lldb' also fails, of course. >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 4:01 PM, Zachary Turner <ztur...@google.com> >>> >> wrote: >>> >> >>> >> He said he did, so I don't know. Vadim, can you elaborate? When you >>> >> run >>> >> `lldb -P` from the command line, what do you see? >>> >> >>> >> >>> >> >>> >> On Tue, Oct 11, 2016 at 4:00 PM Reid Kleckner via lldb-dev >>> >> <lldb-dev@lists.llvm.org> wrote: >>> >> >>> >> I imagine that Hans doesn't have Python 3 installed on his system, so >>> >> LLDB >>> >> didn't autoconfigure with Python support. >>> >> >>> >> >>> >> >>> >> On Sun, Oct 9, 2016 at 1:07 PM, Vadim Chugunov via lldb-dev >>> >> <lldb-dev@lists.llvm.org> wrote: >>> >> >>> >> > Does the 4.0 binary not work for you? It is the first release that >>> >> > contains prebuilt lldb binary. >>> >> >>> >> >>> >> >>> >> Looks like the Python API is not included though. Do you know why it >>> >> was >>> >> left out? >>> >> >>> >> >>> >> >>> >> >>> >> _______________________________________________ >>> >> lldb-dev mailing list >>> >> lldb-dev@lists.llvm.org >>> >> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev >>> >> >>> >> >>> >> >>> >> _______________________________________________ >>> >> lldb-dev mailing list >>> >> lldb-dev@lists.llvm.org >>> >> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev >>> >> >>> >> >>> >> >>> >> >>> >> >>> >> >> >> > _______________________________________________ lldb-dev mailing list lldb-dev@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev