[Python-checkins] r80190 - python/branches/py3k/Lib/platform.py

[victor.stinner]

Author: victor.stinner
Date: Sun Apr 18 20:28:09 2010
New Revision: 80190

Log:
Revert r80167, it breaks build on many platforms


Modified:
   python/branches/py3k/Lib/platform.py

Modified: python/branches/py3k/Lib/platform.py
==============================================================================
--- python/branches/py3k/Lib/platform.py	(original)
+++ python/branches/py3k/Lib/platform.py	Sun Apr 18 20:28:09 2010
@@ -111,7 +111,7 @@
 
 __version__ = '1.0.7'
 
-import sys, os, re, subprocess
+import sys, os, re
 
 ### Globals & Constants
 
@@ -942,20 +942,13 @@
     if sys.platform in ('dos','win32','win16','os2'):
         # XXX Others too ?
         return default
-    target = _follow_symlinks(target)
+    target = _follow_symlinks(target).replace('"', '\\"')
     try:
-        proc = subprocess.Popen(
-            ['file', target],
-            stdout=subprocess.PIPE,
-            stderr=open(DEV_NULL, 'wb'))
+        f = os.popen('file "%s" 2> %s' % (target, DEV_NULL))
     except (AttributeError,os.error):
         return default
-    stdout, stderr = proc.communicate()
-    stdout = stdout.rstrip(b'\n\r')
-    # get output from "filename: output"
-    output = stdout.split(b': ', 1)[-1]
-    output = output.decode('ASCII')
-    rc = proc.wait()
+    output = f.read().strip()
+    rc = f.close()
     if not output or rc:
         return default
     else:
@@ -971,6 +964,8 @@
     'dos': ('','MSDOS'),
 }
 
+_architecture_split = re.compile(r'[\s,]').split
+
 def architecture(executable=sys.executable,bits='',linkage=''):
 
     """ Queries the given executable (defaults to the Python interpreter
@@ -1005,11 +1000,11 @@
 
     # Get data from the 'file' system command
     if executable:
-        fileout = _syscmd_file(executable, '')
+        output = _syscmd_file(executable, '')
     else:
-        fileout = ''
+        output = ''
 
-    if not fileout and \
+    if not output and \
        executable == sys.executable:
         # "file" command did not return anything; we'll try to provide
         # some sensible defaults then...
@@ -1021,6 +1016,9 @@
                 linkage = l
         return bits,linkage
 
+    # Split the output into a list of strings omitting the filename
+    fileout = _architecture_split(output)[1:]
+
     if 'executable' not in fileout:
         # Format not supported
         return bits,linkage