Remove unnecessary double-shell.

client/common_lib/utils.py

 #
 # Copyright 2008 Google Inc. Released under the GPL v2
 
 import os, pickle, random, re, resource, select, shutil, signal, StringIO
 import socket, struct, subprocess, sys, time, textwrap, urlparse
 import warnings, smtplib, logging, urllib2
 from threading import Thread, Event
 try:
     import hashlib
 except ImportError:
@@ skipping 57 matching lines @@
             self.string_stdin = stdin
             stdin = subprocess.PIPE
         else:
             self.string_stdin = None
 
         if verbose:
             logging.debug("Running '%s'" % command)
         self.sp = subprocess.Popen(command, stdout=subprocess.PIPE,
                                    stderr=subprocess.PIPE,
                                    preexec_fn=self._reset_sigpipe, shell=True,
-                                   executable="/bin/bash",
+
+                                   # Default shell in ChromeOS test image is
+                                   # already bash. We're seeing shell-init
+                                   # errors if this value is set.
+
+                                   #executable="/bin/bash",
                                    stdin=stdin)
 
 
     def output_prepare(self, stdout_file=None, stderr_file=None):
         self.stdout_file = stdout_file
         self.stderr_file = stderr_file
 
 
     def process_output(self, stdout=True, final_read=False):
         """output_prepare must be called prior to calling this"""
@@ skipping 1612 matching lines @@
             s.close()
 
     # On the 2.6 kernel, calling try_bind() on UDP socket returns the
     # same port over and over. So always try TCP first.
     while True:
         # Ask the OS for an unused port.
         port = try_bind(0, socket.SOCK_STREAM, socket.IPPROTO_TCP)
         # Check if this port is unused on the other protocol.
         if port and try_bind(port, socket.SOCK_DGRAM, socket.IPPROTO_UDP):
             return port