Hello,
<br/>
<br/>
I patched configure.py to prevent process leaks. I had this king of warning for each and every PyQt component:
<pre>
Generating the C++ source for the QtPrintSupport module...
.../sip OPTIONS ../PyQt5_gpl-5.12.dev1812131446/sip/QtPrintSupport/QtPrintSupportmod.sip
/usr/lib/python3.6/subprocess.py:766: ResourceWarning: subprocess 17168 is still running
ResourceWarning, source=self)
</pre>
If you are interested, this is the patch:
<pre>
--- configure.py 2018-12-17 11:19:15.184585200 +0100
+++ configure-patched.py 2018-12-17 11:18:54.340842077 +0100
@@ -2085,7 +2085,7 @@ def run_command(cmd, verbose):
if verbose:
sys.stdout.write(cmd + "\n")
- fout = get_command_output(cmd, and_stderr=True)
+ p, fout = get_command_output(cmd, and_stderr=True)
# Read stdout and stderr until there is no more output.
lout = fout.readline()
@@ -2099,6 +2099,7 @@ def run_command(cmd, verbose):
lout = fout.readline()
fout.close()
+ p.wait()
try:
os.wait()
@@ -2171,7 +2172,7 @@ def get_command_output(cmd, and_stderr=F
p = subprocess.Popen(cmd, shell=True, stdin=subprocess.PIPE,
stdout=subprocess.PIPE, stderr=stderr)
- return p.stdout
+ return p, p.stdout
def source_path(*names):
@@ -2200,8 +2201,9 @@ def check_dbus(target_config, verbose):
if verbose:
sys.stdout.write(cmd + "\n")
- sout = get_command_output(cmd)
+ p, sout = get_command_output(cmd)
iflags = sout.read().strip()
+ p.wait()
if not iflags:
inform("DBus v1 does not seem to be installed.")
</pre>
Regards,
<div class="signature" style="margin-top:1em;color:#666666;font-size:11px;">
Mickaƫl Schoentgen.<br/>
[FR] Personal blog: <a href="http://www.tiger-222.fr" target="_top" rel="nofollow" link="external">http://www.tiger-222.fr</a>
</div>
<br/><hr align="left" width="300" />
Sent from the <a href="http://python.6.x6.nabble.com/PyQt-f1792048.html">PyQt mailing list archive</a> at Nabble.com.<br/>