split shell.system into shell.system_raw/system_piped

system_raw calls os.system, system_piped uses pexpect/utils.platform magic

use system_raw in Terminal except on Windows and in tests
use system_piped elsewhere

closes gh-297
closes gh-457

IPython/core/interactiveshell.py

@@ -1872,22 +1872,40 b' class InteractiveShell(SingletonConfigurable, Magic):'
     # Things related to the running of system commands
     #-------------------------------------------------------------------------
 
-    def system(self, cmd):
-        """Call the given cmd in a subprocess.
+    def system_piped(self, cmd):
+        """Call the given cmd in a subprocess, piping stdout/err
 
         Parameters
         ----------
         cmd : str
-          Command to execute (can not end in '&', as bacground processes are
-          not supported.
+          Command to execute (can not end in '&', as background processes are
+          not supported.  Should not be a command that expects input
+          other than simple text.
         """
+        if cmd.rstrip().endswith('&'):
+            # this is *far* from a rigorous test
         # We do not support backgrounding processes because we either use
         # pexpect or pipes to read from.  Users can always just call
-        # os.system() if they really want a background process.
-        if cmd.endswith('&'):
+            # os.system() or use ip.system=ip.system_raw
+            # if they really want a background process.
             raise OSError("Background processes not supported.")
 
-        return system(self.var_expand(cmd, depth=2))
+        # don't return the result, always return None
+        system(self.var_expand(cmd, depth=2))
+    
+    def system_raw(self, cmd):
+        """Call the given cmd in a subprocess using os.system
+        
+        Parameters
+        ----------
+        cmd : str
+          Command to execute.
+        """
+        # don't return the result, always return None
+        os.system(self.var_expand(cmd, depth=2))
+    
+    # use piped system by default, because it is better behaved
+    system = system_piped
 
     def getoutput(self, cmd, split=True):
         """Get output (possibly including stderr) from a subprocess.
@@ -1905,7 +1923,8 b' class InteractiveShell(SingletonConfigurable, Magic):'
           manipulation of line-based output.  You can use '?' on them for
           details.
           """
-        if cmd.endswith('&'):
+        if cmd.rstrip().endswith('&'):
+            # this is *far* from a rigorous test
             raise OSError("Background processes not supported.")
         out = getoutput(self.var_expand(cmd, depth=2))
         if split:

IPython/frontend/terminal/interactiveshell.py

@@ -86,6 +86,12 b' class TerminalInteractiveShell(InteractiveShell):'
             config=config, ipython_dir=ipython_dir, user_ns=user_ns,
             user_global_ns=user_global_ns, custom_exceptions=custom_exceptions
         )
+        # use os.system instead of utils.process.system by default, except on Windows
+        if os.name == 'nt':
+            self.system = self.system_piped
+        else:
+            self.system = self.system_raw
+        
         self.init_term_title()
         self.init_usage(usage)
         self.init_banner(banner1, banner2, display_banner)