Trouble building rpms on Centos 7.3

[jnaulty]

I am trying to build the rpm for this project for python 2.7
when running command:
python setup.py bdist_rpm --python /usr/bin/python

I receive the error

+ cd tenacity-4.4.1.dev1
+ /usr/bin/python setup.py install --single-version-externally-managed -O1 --root=/home/jnaulty/workspace/tenacity/build/bdist.linux-x86_64/rpm/BUILDROOT/tenacity-4.4.1.dev1-1.x86_64 --record=INSTALLED_FILES
  File "/usr/lib/python2.7/site-packages/tenacity/tests/test_async.py", line 39
    yield from asyncio.sleep(0.00001)
             ^
SyntaxError: invalid syntax

  File "/usr/lib/python2.7/site-packages/tenacity/async.py", line 44
    result = yield from fn(*args, **kwargs)
                      ^
SyntaxError: invalid syntax

  File "/usr/lib/python2.7/site-packages/tenacity/tests/test_async.py", line 39
    yield from asyncio.sleep(0.00001)
             ^
SyntaxError: invalid syntax

  File "/usr/lib/python2.7/site-packages/tenacity/async.py", line 44
    result = yield from fn(*args, **kwargs)
                      ^
SyntaxError: invalid syntax

+ /usr/lib/rpm/find-debuginfo.sh --strict-build-id -m --run-dwz --dwz-low-mem-die-limit 10000000 --dwz-max-die-limit 110000000 /home/jnaulty/workspace/tenacity/build/bdist.linux-x86_64/rpm/BUILD/tenacity-4.4.1.dev1
find: 'debug': No such file or directory
+ /usr/lib/rpm/check-buildroot
+ /usr/lib/rpm/redhat/brp-compress
+ /usr/lib/rpm/redhat/brp-strip-static-archive /usr/bin/strip
+ /usr/lib/rpm/brp-python-bytecompile /usr/bin/python 1
error: Bad exit status from /var/tmp/rpm-tmp.8cn8w2 (%install)
    Bad exit status from /var/tmp/rpm-tmp.8cn8w2 (%install)
error: command 'rpmbuild' failed with exit status 1

What is the workaround for this?

[vicyap]

yield from syntax is a >= Python 3.3 syntax, which will not work for Python 2.7. I'm not sure what the usual solution to this problem would be. Maybe something in setup.py can fix this?

[jnaulty]

Taking a look at other projects and their solutions: https://github.com/kennethreitz/tablib/blob/master/tablib/compat.py

[jd]

Well this is code compatibility, that's easy enough. Tenacity uses https://pypi.python.org/pypi/six for that.

Problem is that it does not cover syntax changes like this is the problem here.

[jd]

@hguemar Do you have any idea on how to fix that?

[RonnyPfannschmidt]

easiest way would be to just delete the file in the rpm spec for the python2 build

[coreycb]

I'm also hitting this when trying to package a new release for Ubuntu. It looks like there are a few areas where this syntax is used, making the code incompatible for python 2.

tenacity/tests/test_async.py:    yield from asyncio.sleep(0.00001)
tenacity/tests/test_async.py:        yield from _retryable_coroutine(thing)
tenacity/async.py:                    result = yield from fn(*args, **kwargs)
tenacity/async.py:                yield from self.sleep(do)

[mrunge]

Fedora now switched to python 3.7, tenacity 4.9 uses async.

[jnaulty]

opened a PR here: #133
This is the solution I'm using internally...not sure if it's the best way forward, but it works :)

[asqui]

Should Python 2 versions be removed from the specifiers if the library is not Py2 compatible (as of v4.10, according to kragniz/python-etcd3#429 )?

Actually, I see that the library is attempting to maintain Py2-compatibility by conditionally importing the _asyncio.py module, but the problem occurs if you try to proactively compile it. How about making _asyncio.py more robust to this?

[jd]

Tenacity is compatible with Python 2.

Don't trust everything you read on the internet.