Backed out changeset c73b90b6dadd
diff --git a/Lib/platform.py b/Lib/platform.py
index e94a32a..d20c39d 100755
--- 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
index 39bd450..b6d375c 100644
--- 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
index 36033ea..a02c073 100644
--- 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