Revert 32057, 32058, 32059, 32062 because they still have unwanted side-effects.
Revert "Group SCM-specific functions in classes to simplify generalization of the interface."
Revert "Fix 2 recently introduced errors."
Revert "Fix a variable aliasing error."
Revert "Fix typo."
TBR=dpranke
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/391075
git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@32065 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/gclient_utils.py b/gclient_utils.py
index 4a5989c..9016270 100644
--- a/gclient_utils.py
+++ b/gclient_utils.py
@@ -12,8 +12,6 @@
# See the License for the specific language governing permissions and
# limitations under the License.
-"""Generic utils."""
-
import errno
import os
import re
@@ -24,6 +22,8 @@
import xml.dom.minidom
import xml.parsers.expat
+## Generic utils
+
def SplitUrlRevision(url):
"""Splits url and returns a two-tuple: url, rev"""
@@ -76,9 +76,9 @@
return output
-def FileRead(filename, mode='rU'):
+def FileRead(filename):
content = None
- f = open(filename, mode)
+ f = open(filename, "rU")
try:
content = f.read()
finally:
@@ -86,8 +86,8 @@
return content
-def FileWrite(filename, content, mode='w'):
- f = open(filename, mode)
+def FileWrite(filename, content):
+ f = open(filename, "w")
try:
f.write(content)
finally:
@@ -201,9 +201,9 @@
only if we actually need to print something else as well, so you can
get the context of the output. If print_messages is false and print_stdout
is false, no output at all is generated.
-
- Also, if print_stdout is true, the command's stdout is also forwarded
- to stdout.
+
+ Also, if print_stdout is true, the command's stdout is also forwarded
+ to stdout.
If a filter function is specified, it is expected to take a single
string argument, and it will be called with each line of the
@@ -223,7 +223,7 @@
# executable, but shell=True makes subprocess on Linux fail when it's called
# with a list because it only tries to execute the first item in the list.
kid = subprocess.Popen(command, bufsize=0, cwd=in_directory,
- shell=(sys.platform == 'win32'), stdout=subprocess.PIPE,
+ shell=(sys.platform == 'win32'), stdout=subprocess.PIPE,
stderr=subprocess.STDOUT)
# Also, we need to forward stdout to prevent weird re-ordering of output.
@@ -238,7 +238,7 @@
if not print_messages:
print("\n________ running \'%s\' in \'%s\'"
% (' '.join(command), in_directory))
- print_messages = True
+ print_messages = True
sys.stdout.write(in_byte)
if in_byte != "\n":
in_line += in_byte