add possibility to call eConsoleAppContainer execute with unlimited count of arguments

when its called with single argument, then /bin/sh is started else not

diff --git a/lib/base/console.cpp b/lib/base/console.cpp
index a12cb5e29bc7f408442e79c2cc303a97c1d2182d..25318cc9e388550a7fb715e20dc329a7ea08799d 100644 (file)
--- a/lib/base/console.cpp
+++ b/lib/base/console.cpp
@@ -476,11 +476,37 @@ eConsolePy_running(eConsolePy* self)
 static PyObject *
 eConsolePy_execute(eConsolePy* self, PyObject *argt)
 {
-       const char *str;
-       if (PyArg_ParseTuple(argt, "s", &str))
-               return PyInt_FromLong(self->cont->execute(str));
-       PyErr_SetString(PyExc_TypeError,
-               "argument is not a string");
+       Py_ssize_t argc = PyTuple_Size(argt);
+       if (argc > 1)
+       {
+               const char *argv[argc + 1];
+               int argpos=0;
+               while(argpos < argc)
+               {
+                       PyObject *arg = PyTuple_GET_ITEM(argt, argpos);
+                       if (!PyString_Check(arg))
+                       {
+                               char err[255];
+                               if (argpos)
+                                       snprintf(err, 255, "arg %d is not a string", argpos);
+                               else
+                                       snprintf(err, 255, "cmd is not a string!");
+                               PyErr_SetString(PyExc_TypeError, err);
+                               return NULL;
+                       }
+                       argv[argpos++] = PyString_AsString(arg);
+               }
+               argv[argpos] = 0;
+               return PyInt_FromLong(self->cont->execute(argv[0], argv));
+       }
+       else
+       {
+               const char *str;
+               if (PyArg_ParseTuple(argt, "s", &str))
+                       return PyInt_FromLong(self->cont->execute(str));
+               PyErr_SetString(PyExc_TypeError,
+                       "cmd is not a string!");
+       }
        return NULL;
 }
 

diff --git a/lib/python/Components/Task.py b/lib/python/Components/Task.py
index 47acc87eb741ac2a1d465462384a2ac72d864ac7..ab85c66718b66e7c7f090ef87965d210452093b7 100644 (file)
--- a/lib/python/Components/Task.py
+++ b/lib/python/Components/Task.py
@@ -136,6 +136,7 @@ class Task(object):
 
        def setTool(self, tool):
                self.cmd = tool
+               self.args = [tool]
                self.global_preconditions.append(ToolExistsPrecondition())
                self.postconditions.append(ReturncodePostcondition())
 
@@ -171,8 +172,7 @@ class Task(object):
                if self.cwd is not None:
                        self.container.setCWD(self.cwd)
 
-               execstr = " ".join([self.cmd]+self.args)
-               print "execute:", self.container.execute(execstr), execstr
+               print "execute:", self.container.execute(self.cmd, *self.args), self.cmd, *self.args
                if self.initial_input:
                        self.writeInput(self.initial_input)