[Lldb-commits] [PATCH] D79659: [lldb/Commands] Add ability to run shell command on the host.

Med Ismail Bennani via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon May 11 09:07:33 PDT 2020


mib marked an inline comment as done.
mib added inline comments.


================
Comment at: lldb/source/Commands/CommandObjectPlatform.cpp:1625
 
+    // When called from the `shell` alias with no argument, exit.
+    if (args.GetRawPart().empty())
----------------
JDevlieghere wrote:
> Is this only true for the alias?
I added this check because when using `shell` with no command, instead of showing the command usage, lldb tries running an empty command (if that makes sense) and showed the following error message:

```
(lldb) shell
error: executable doesn't exist: '(empty)'
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79659/new/

https://reviews.llvm.org/D79659





More information about the lldb-commits mailing list