<div dir="ltr">It's a little more difficult than that, because we have<div><br></div><div>#define LLDB_DEFAULT_SHELL "cmd.exe"</div><div><br></div><div>So we'd need to move this into a runtime function and fix up all assumptions about how it might be available at compile time.  It doesn't seem like that would be too bad though.   The biggest offender would be that we currently have a function like this:</div><div><br></div><div>void Foo(int x, int y, const char *shell = LLDB_DEFAULT_SHELL)</div><div><br></div><div>But I suppose we could just add a two-argument overload that calls the 3 argument version with GetDefaultShell(), to avoid having to fix up a ton of callsites.</div><div><br></div><div>This might be a better solution than fixing it in RunShellCommand because it removes the possibility of someone forgetting to resolve the path in the future.</div><div><br></div><div>Thoughts Greg?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 14, 2014 at 7:11 PM, Scott Graham <span dir="ltr"><<a href="mailto:scottmg@chromium.org" target="_blank">scottmg@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">getenv("COMSPEC") rather than "cmd.exe"? I've never seen that be non-absolute.</div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Oct 14, 2014 at 4:36 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><div dir="ltr">I have an issue on Windows when trying to run shell commands.  We specify the shell as "cmd.exe", create a FileSpec out of this, and call FileSpec::Resolve.  This ends up making an absolute path out of cmd.exe, but it does so by just sticking the working directory onto the front of it, which is obviously wrong.<div><br></div><div>My question is: For FileSpecs that are only filenames, nothing else, should we attempt to locate a matching file in PATH, and when we find one use the resulting absolute path?</div></div>
<br></span><span class="">_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br></span></blockquote></div><br></div>
</blockquote></div><br></div>