subprocess.Popen().wait() at the end of stdout collection... Fix #64 (#66)

* subprocess.Popen().wait() at the end of stdout collection... Fix #64

Limit supported python to < 3.5 (v2.0+ for 3.5+)
Reduce CI jobs and packages amount on PyPI
Requirements ad PEP0508 only (standard is not young)

* enforce bump

Author: penguinolog

.travis.yml

@@ -2,13 +2,9 @@ sudo: false
 language: python
 os: linux
 python:
-- 2.7
+- &main_python 2.7
 - 3.4
-- 3.5
-- 3.6
-- &mainstream_python 3.7-dev
 - &pypy pypy
-- pypy3.5
 install:
 - &upgrade_python_toolset pip install --upgrade pip setuptools wheel
 - pip install tox-travis
@@ -21,7 +17,7 @@ jobs:
   fast_finish: true
   include:
   - stage: Static analisys
-    python: 3.6
+    python: 3.4
     services: []
     install:
     - *upgrade_python_toolset
@@ -31,7 +27,7 @@ jobs:
     after_success: skip
 
   - stage: Code style check
-    python: *mainstream_python
+    python: 3.4
     install:
     - *upgrade_python_toolset
     - pip install tox

CI_REQUIREMENTS.txt

@@ -1,4 +1,2 @@
 mock  # no assert_called_once in py35
-futures>=3.1; python_version == "2.7"
-enum34>=1.1; python_version == "2.7"
 -r requirements.txt

README.rst

@@ -44,11 +44,9 @@ Pros:
 
     Python 2.7
     Python 3.4
-    Python 3.5
-    Python 3.6
-    Python 3.7
     PyPy
-    PyPy3 3.5+
+
+.. note:: Update to version 2.0+ for usage with python 3.5+. This version is for legacy python and no new features are planned.
 
 This package includes:
 

exec_helpers/__init__.py

@@ -48,7 +48,7 @@
     'ExecResult',
 )
 
-__version__ = '1.3.6'
+__version__ = '1.3.7'
 __author__ = "Alexey Stepanov"
 __author_email__ = 'penguinolog@gmail.com'
 __maintainers__ = {

exec_helpers/subprocess_runner.py

@@ -135,6 +135,7 @@ def poll_stdout():
                 log=logger,
                 verbose=verbose
             )
+            interface.wait()  # wait for the end of execution
 
         @threaded.threadpooled
         def poll_stderr():

requirements.txt

@@ -4,4 +4,6 @@ six>=1.10.0  # MIT
 threaded>=1.0  # Apache-2.0
 PyYAML>=3.12  # MIT
 advanced-descriptors>=1.0  # Apache-2.0
-typing >= 3.6 ; python_version < "3.8"
+typing >= 3.6  # PSF
+futures>=3.1; python_version == "2.7"
+enum34>=1.1; python_version == "2.7"

setup.py

@@ -238,9 +238,6 @@ def get_simple_vars_from_src(src):
     'Programming Language :: Python :: 2.7',
     'Programming Language :: Python :: 3',
     'Programming Language :: Python :: 3.4',
-    'Programming Language :: Python :: 3.5',
-    'Programming Language :: Python :: 3.6',
-    'Programming Language :: Python :: 3.7',
 
     'Programming Language :: Python :: Implementation :: CPython',
     'Programming Language :: Python :: Implementation :: PyPy',
@@ -267,7 +264,7 @@ def get_simple_vars_from_src(src):
     long_description=long_description,
     classifiers=classifiers,
     keywords=keywords,
-    python_requires='>=2.7.5,!=3.0.*,!=3.1.*,!=3.2.*,!=3.3.*',
+    python_requires='>=2.7.5,!=3.0.*,!=3.1.*,!=3.2.*,!=3.3.*,<3.5',
     # While setuptools cannot deal with pre-installed incompatible versions,
     # setting a lower bound is not harmful - it makes error messages cleaner. DO
     # NOT set an upper bound on setuptools, as that will lead to uninstallable
@@ -277,12 +274,6 @@ def get_simple_vars_from_src(src):
     setup_requires="setuptools >= 21.0.0,!=24.0.0,"
                    "!=34.0.0,!=34.0.1,!=34.0.2,!=34.0.3,!=34.1.0,!=34.1.1,!=34.2.0,!=34.3.0,!=34.3.1,!=34.3.2,"
                    "!=36.2.0",
-    extras_require={
-        ':python_version == "2.7"': [
-            'futures>=1.0',
-            'enum34>=1.1'
-        ],
-    },
     install_requires=required,
     package_data={
         str('exec_helpers'): [

tools/build-wheels.sh

@@ -1,5 +1,5 @@
 #!/bin/bash
-PYTHON_VERSIONS="cp34-cp34m cp35-cp35m cp36-cp36m cp37-cp37m"
+PYTHON_VERSIONS="cp34-cp34m"
 
 # Avoid creation of __pycache__/*.py[c|o]
 export PYTHONDONTWRITEBYTECODE=1

tox.ini

@@ -5,7 +5,7 @@
 
 [tox]
 minversion = 2.0
-envlist = pep8, pep257, py{27,34,35,36,37,py,py3}, pylint, bandit, py{34,35,36,37}-nocov, mypy, docs
+envlist = pep8, pep257, py{27,34,py}, pylint, bandit, py34-nocov, mypy, docs
 skipsdist = True
 skip_missing_interpreters = True
 
@@ -20,7 +20,7 @@ deps =
     pytest-cov
     pytest-html
     pytest-sugar
-    py{27,34,35,36}-nocov: Cython
+    py34-nocov: Cython
     -r{toxinidir}/CI_REQUIREMENTS.txt
     py{27,py}: mock
 
@@ -36,38 +36,13 @@ commands =
     pip install exec_helpers --no-index -f dist
     py.test -vvv {posargs:test}
 
-[testenv:py35-nocov]
-usedevelop = False
-commands =
-    python setup.py bdist_wheel
-    pip install exec_helpers --no-index -f dist
-    py.test -vvv {posargs:test}
-
-[testenv:py36-nocov]
-usedevelop = False
-commands =
-    python setup.py bdist_wheel
-    pip install exec_helpers --no-index -f dist
-    py.test -vvv {posargs:test}
-
-[testenv:py37-nocov]
-usedevelop = False
-commands =
-    python setup.py bdist_wheel
-    pip install exec_helpers --no-index -f dist
-    py.test -vvv {posargs:test}
-
 [testenv:venv]
 commands = {posargs:}
 
 [tox:travis]
 2.7 = install, py27,
 3.4 = py34,
-3.5 = py35,
-3.6 = py36,
-3.7 = py37,
 pypy = install, pypy,
-pypy3 = install, pypy3,
 
 [testenv:pep8]
 deps =