Polishing run_cli_process calls

Author: PrzemekWirkus

mbed_lstools/lstools_base.py

@@ -577,6 +577,7 @@ def scan_html_line_for_target_id(self, line):
                 return result
         return None
 
+    @staticmethod
     def run_cli_process(cmd, shell=True):
         """! Runs command as a process and return stdout, stderr and ret code
         @param cmd Command to execute

mbed_lstools/lstools_linux_generic.py

@@ -127,7 +127,7 @@ def get_dev_by_id_cmd(self, subdir):
         """
         cmd = 'ls -oA /dev/' + subdir + '/by-id/'
         _stdout, _, retval = self.run_cli_process(cmd)
-        return (_stdout, retval)
+        return (_stdout.splitlines(), retval)
 
     def get_dev_by_id_process(self, lines, retval):
         """! Remove unnecessary lines from command line output
@@ -163,7 +163,7 @@ def get_mounts(self):
         _stdout, _, retval = self.run_cli_process(cmd)
 
         if not retval:
-            for line in _stdout.readlines():
+            for line in _stdout.splitlines():
                 line = line.rstrip()
                 result.append(line)
                 if self.DEBUG_FLAG: