summaryrefslogtreecommitdiff
path: root/source/core/unix
diff options
context:
space:
mode:
Diffstat (limited to 'source/core/unix')
-rw-r--r--source/core/unix/slang-unix-process-util.cpp18
1 files changed, 15 insertions, 3 deletions
diff --git a/source/core/unix/slang-unix-process-util.cpp b/source/core/unix/slang-unix-process-util.cpp
index ee61cbc85..0447dd499 100644
--- a/source/core/unix/slang-unix-process-util.cpp
+++ b/source/core/unix/slang-unix-process-util.cpp
@@ -59,12 +59,21 @@ namespace Slang {
/* static */String ProcessUtil::getCommandLineString(const CommandLine& commandLine)
{
+ // When outputting the command line we potentially need to escape the path to the
+ // command and args - that aren't already explicitly marked as escaped.
StringBuilder cmd;
appendCommandLineEscaped(commandLine.m_executable.getUnownedSlice(), cmd);
for (const auto& arg : commandLine.m_args)
{
cmd << " ";
- appendCommandLineEscaped(arg.getUnownedSlice(), cmd);
+ if (arg.type == CommandLine::ArgType::Unescaped)
+ {
+ appendCommandLineEscaped(arg.value.getUnownedSlice(), cmd);
+ }
+ else
+ {
+ cmd << arg.value;
+ }
}
return cmd.ToString();
}
@@ -76,10 +85,13 @@ namespace Slang {
List<char const*> argPtrs;
// Add the command
argPtrs.add(commandLine.m_executable.getBuffer());
- // Add all the args
+
+ // Add all the args - they don't need any explicit escaping
for (auto arg : commandLine.m_args)
{
- argPtrs.add(arg.getBuffer());
+ // All args for this target must be unescaped
+ SLANG_ASSERT(arg.type == CommandLine::ArgType::Unescaped);
+ argPtrs.add(arg.value.getBuffer());
}
// Terminate with a null
argPtrs.add(nullptr);