File System Workspace: executable arguments can now be placed on sepa…

…rate row, as well as on the same line
eranif · Mar 25, 2022 · 519683b · 519683b
1 parent 8f05303
commit 519683b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/CodeLite/clConsoleBase.cpp b/CodeLite/clConsoleBase.cpp
@@ -157,6 +157,8 @@ bool clConsoleBase::StartProcess(const wxString& command)
         callback = new ConsoleProcess(m_sink, m_callbackUID);
     }
 
+    clDEBUG() << "Console: running command: `" << command << "`" << endl;
+
     SetPid(::wxExecute(command, wxEXEC_ASYNC | wxEXEC_MAKE_GROUP_LEADER | GetExecExtraFlags(), callback));
     // reset the m_callback (it will auto-delete itself)
     m_callback = nullptr;

diff --git a/Plugin/clFileSystemWorkspace.cpp b/Plugin/clFileSystemWorkspace.cpp
@@ -1009,6 +1009,22 @@ void clFileSystemWorkspace::GetExecutable(wxString& exe, wxString& args, wxStrin
     args = GetConfig()->GetArgs();
     wd = GetConfig()->GetWorkingDirectory().IsEmpty() ? GetFileName().GetPath() : GetConfig()->GetWorkingDirectory();
 
+    // build the arguments
+    args.Replace("\r", wxEmptyString);
+    args.Replace("\n", " ");
+    auto args_arr = StringUtils::BuildArgv(args);
+
+    args.clear();
+    for(auto& arg : args_arr) {
+        if(!args.empty()) {
+            args << " ";
+        }
+        arg.Trim().Trim(false);
+        // wrap with quotes if required
+        ::WrapWithQuotes(arg);
+        args << arg;
+    }
+
     exe = MacroManager::Instance()->Expand(exe, nullptr, wxEmptyString);
     args = MacroManager::Instance()->Expand(args, nullptr, wxEmptyString);
     wd = MacroManager::Instance()->Expand(wd, nullptr, wxEmptyString);