- User Since
- Jun 7 2018, 12:06 PM (19 w, 1 d)
Tue, Sep 25
Changed the test to use %python variable instead of python
Changed python version required to use 'pass_fds' argument to 3.2.
I tested this patch with python 2.7 and 3.5.
Removed the shebang line since we call the script via python ... so that line doesn't matter.
Thanks Pavel, I fixed it here https://reviews.llvm.org/D52498.
Mon, Sep 24
Reduced timer from 120 to 30 seconds.
If so, we can try to run the script with python2.x. @teemperor can you try to modify target-select-so-path.test this way:
change # RUN: python %p/inputs/target-select-so-path.py "%debugserver" "%lldbmi %t" %s to
# RUN: python2 %p/inputs/target-select-so-path.py "%debugserver" "%lldbmi %t" %s
Sat, Sep 22
AFAIR, adding an exit(...) to ConnectToRemote won't solve this problem. The test will still be failing on Arch.
Fri, Sep 21
Do you mean ConnectToRemote method from lldb/tools/lldb-server/lldb-gdbserver.cpp?
Thu, Sep 20
Thanks to @tatyana-krasnukha for the idea about a timer. Added a timer to target-select-so-path test.
Can you provide some logs? For example, you might be able to run this test (llvm-lit -a -vv .../target-select-so-path.test) and kill lldb-mi and filecheck processes if they hang. Also, information about which processes hang would be useful too (lldb-mi or filecheck or maybe both).
I found out that the reason of hanging of the test suite was in incorrect usage of Filecheck and lldb-mi processes in target-select-so-path.test. Current patch has been tested on CentOS 7.
Sep 17 2018
It seems reasonable, I'll look at this.
Sep 15 2018
Sep 14 2018
I've created this revision to be sure that you don't think that this approach is some kind of hack.
Sep 7 2018
Sep 3 2018
Aug 9 2018
I think those options don't fit. I'm looking for behavior like this:
~/workspace/gsoc/build/bin/lldb-server gdbserver --pipe 1 localhost:0 36251
Here lldb-server prints out the port number it is listening on and continues running.
It seems that target-select-so-path.test hangs on macOS. Thanks to @t.p.northover for noting this.
Aug 7 2018
Sure, will do it.
Splitted the patch into two parts: this part with the new API and another one with re-implementing of target-select command.
The second part will be committed separately.
Aug 1 2018
@clayborg can you explain what are you worry about because I don't completely understand you? As I said, all lldb-mi commands print their result to STDOUT, AFAIK, an IDE should parse it and then show in a GUI.
Jul 31 2018
Changed the order of if statements to follow llvm coding standards.
Made error handling more meaningful: added different error messages for cases - empty <from> path and empty <to> path.
Added converting from const char * to ConstString, documented new API.
So, do you have any thoughts about this approach letting the debugserver choose a tcp port and patch overall?
Jul 30 2018
Jul 26 2018
It seems that it's impossible to get HandleProcessEventStateSuspended called on Linux, so I've copied code of this patch into
HandleProcessEventStateStopped to check new output.
Now tcp port is choosing by debugserver.
Jul 25 2018
Moved test from bash to python, removed unnecessary Target::AppendImageSearchPath.
Thanks, I used this lldb-server gdbserver --pipe 0 localhost:0 and got a port chosen by debugserver. But to let lldb-mi know about this port we need an additional script, is python a good choice for that?
packages/Python/lldbsuite/test/tools/lldb-mi/signal/TestMiSignal.py test contains port = 12000 + random.randint(0, 3999).
What do you think about running tests with a hardcoded port number(as it's done in a web-services). Doing this, we get rid of additional scripts and os-specific things. AFAIK, debugserver even has its own default port.
Jul 24 2018
Jul 23 2018
You mean that it's unreasonable to provide such an output to stdout since MI clients are text redactors, IDE and not people?
Another approach is to implement SBTargetSettings' functionality such as serialization inside lldb_private TargetProperties class and then just add an API to the SBTarget. For example:
Jul 21 2018
Jul 19 2018
Converted data-info-line python test to a lit one.
Jul 13 2018
@stella.stamenova Done in r337064.
It would be good to convert to LIT as more tests as we can. I'll do this in one of a next patches. Let's continue with this one?
Jul 9 2018
Could you please provide an output of this command on Windows: E:\_work\4\b\LLVMBuild\Release\bin\lldb-mi.EXE --synchronous E:\_work\4\b\LLVMBuild\tools\lldb\lit\tools\lldb-mi\symbol\Output\symbol-list-lines.test.tmp < E:\_work\4\s\llvm\tools\lldb\lit\tools\lldb-mi\symbol\symbol-list-lines.test? I'll try to find out what causes a failure.
Returned accidentally removed backslashes in self.expect pattern, removed comment from:
#include <string> // For std::to_string.
Jul 8 2018
Jul 7 2018
Jul 6 2018
Unfortunately, I don't have an opportunity to check this out on windows, but if this works, I'll be glad that someone found out how to fix compilation failure on windows.