[Lldb-commits] [PATCH] Move argument globbing to Target::Launch from Platform::LaunchProcess.

Zachary Turner zturner at google.com
Thu Feb 19 17:55:21 PST 2015


On Thu Feb 19 2015 at 5:43:55 PM Greg Clayton <gclayton at apple.com> wrote:

> >
> > That which will be part of testing the platform-specific bits of your
> platform
> >
> >  There's inherent value in having as few platform specific bits as
> possible.  When the same rules apply to every platform,
> > a) everyone gets better test coverage
> > b) everyone gets to benefit from everyone else's changes and bugfixes
> > c) as a user, it makes it easier and more comfortable to switch
> platforms since you don't have to do a mental switch.
> > d) It makes the code easier to maintain
>
> With the detriment that glowing might not behave like the shell does on
> your system. It would be nice to also support other shells (tcsh, zsh, ksh,
> etc) so that things would expand as the user expects them to.
>
We already have SBPlatformShellCommand for that right?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150220/88513259/attachment.html>


More information about the lldb-commits mailing list