mirror of
https://github.com/python/cpython
synced 2024-09-22 17:23:46 +00:00
Explicitly close pipes so test_ctypes won't appear to randomly leak
+33 or -33 references. (See discussion in #1597.)
This commit is contained in:
parent
b8189f3b5a
commit
f4d4f8b97f
|
@ -50,8 +50,10 @@ def _findLib_gcc(name):
|
|||
'$CC -Wl,-t -o ' + ccout + ' 2>&1 -l' + name
|
||||
try:
|
||||
f = os.popen(cmd)
|
||||
trace = f.read()
|
||||
f.close()
|
||||
try:
|
||||
trace = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
finally:
|
||||
try:
|
||||
os.unlink(ccout)
|
||||
|
@ -70,7 +72,12 @@ def _get_soname(f):
|
|||
if not f:
|
||||
return None
|
||||
cmd = "/usr/ccs/bin/dump -Lpv 2>/dev/null " + f
|
||||
res = re.search(r'\[.*\]\sSONAME\s+([^\s]+)', os.popen(cmd).read())
|
||||
f = os.popen(cmd)
|
||||
try:
|
||||
data = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
res = re.search(r'\[.*\]\sSONAME\s+([^\s]+)', data)
|
||||
if not res:
|
||||
return None
|
||||
return res.group(1)
|
||||
|
@ -80,7 +87,12 @@ def _get_soname(f):
|
|||
if not f:
|
||||
return None
|
||||
cmd = "objdump -p -j .dynamic 2>/dev/null " + f
|
||||
res = re.search(r'\sSONAME\s+([^\s]+)', os.popen(cmd).read())
|
||||
f = os.popen(cmd)
|
||||
try:
|
||||
data = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
res = re.search(r'\sSONAME\s+([^\s]+)', data)
|
||||
if not res:
|
||||
return None
|
||||
return res.group(1)
|
||||
|
@ -103,8 +115,12 @@ def _num_version(libname):
|
|||
def find_library(name):
|
||||
ename = re.escape(name)
|
||||
expr = r':-l%s\.\S+ => \S*/(lib%s\.\S+)' % (ename, ename)
|
||||
res = re.findall(expr,
|
||||
os.popen('/sbin/ldconfig -r 2>/dev/null').read())
|
||||
f = os.popen('/sbin/ldconfig -r 2>/dev/null')
|
||||
try:
|
||||
data = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
res = re.findall(expr, data)
|
||||
if not res:
|
||||
return _get_soname(_findLib_gcc(name))
|
||||
res.sort(cmp= lambda x,y: cmp(_num_version(x), _num_version(y)))
|
||||
|
@ -115,12 +131,21 @@ def find_library(name):
|
|||
def _findLib_ldconfig(name):
|
||||
# XXX assuming GLIBC's ldconfig (with option -p)
|
||||
expr = r'/[^\(\)\s]*lib%s\.[^\(\)\s]*' % re.escape(name)
|
||||
res = re.search(expr,
|
||||
os.popen('/sbin/ldconfig -p 2>/dev/null').read())
|
||||
f = os.popen('/sbin/ldconfig -p 2>/dev/null')
|
||||
try:
|
||||
data = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
res = re.search(expr, data)
|
||||
if not res:
|
||||
# Hm, this works only for libs needed by the python executable.
|
||||
cmd = 'ldd %s 2>/dev/null' % sys.executable
|
||||
res = re.search(expr, os.popen(cmd).read())
|
||||
f = os.popen(cmd)
|
||||
try:
|
||||
data = f.read()
|
||||
finally:
|
||||
f.close()
|
||||
res = re.search(expr, data)
|
||||
if not res:
|
||||
return None
|
||||
return res.group(0)
|
||||
|
|
Loading…
Reference in a new issue