[Python-checkins] cpython (2.7): Backed out changeset c73b90b6dadd

[jesus.cea]

http://hg.python.org/cpython/rev/5abacebec9d2
changeset:   79449:5abacebec9d2
branch:      2.7
parent:      79445:c73b90b6dadd
user:        Jesus Cea <jcea at jcea.es>
date:        Thu Oct 04 14:22:16 2012 +0200
summary:
  Backed out changeset c73b90b6dadd

files:
  Lib/platform.py |  13 +++++--------
  Misc/ACKS       |   1 -
  Misc/NEWS       |   3 ---
  3 files changed, 5 insertions(+), 12 deletions(-)


diff --git a/Lib/platform.py b/Lib/platform.py
--- a/Lib/platform.py
+++ b/Lib/platform.py
@@ -113,7 +113,7 @@
 
 __version__ = '1.0.7'
 
-import sys,string,os,re,subprocess
+import sys,string,os,re
 
 ### Globals & Constants
 
@@ -1023,16 +1023,13 @@
     if sys.platform in ('dos','win32','win16','os2'):
         # XXX Others too ?
         return default
-    target = _follow_symlinks(target)
+    target = _follow_symlinks(target).replace('"', '\\"')
     try:
-        with open(DEV_NULL) as dev_null:
-            proc = subprocess.Popen(['file', '-b', '--', target],
-                    stdout=subprocess.PIPE, stderr=dev_null)
-
+        f = os.popen('file "%s" 2> %s' % (target, DEV_NULL))
     except (AttributeError,os.error):
         return default
-    output = proc.stdout.read()
-    rc = proc.wait()
+    output = string.strip(f.read())
+    rc = f.close()
     if not output or rc:
         return default
     else:
diff --git a/Misc/ACKS b/Misc/ACKS
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -77,7 +77,6 @@
 Ben Bell
 Thomas Bellman
 Alexander “Саша” Belopolsky
-David Benjamin
 Andrew Bennetts
 Andy Bensky
 Bennett Benson
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -143,9 +143,6 @@
   io.BytesIO and io.StringIO objects now raise ValueError when the object has
   been closed.  Patch by Alessandro Moura.
 
-- Issue #16112: platform.architecture does not correctly escape argument to
-  /usr/bin/file.  Patch by David Benjamin.
-
 - Issue #12776,#11839: call argparse type function (specified by add_argument)
   only once. Before, the type function was called twice in the case where the
   default was specified and the argument was given as well.  This was

-- 
Repository URL: http://hg.python.org/cpython