Bug 1583626

Summary: python-twisted FTBFS with Python 3.7
Product: [Fedora] Fedora Reporter: Miro Hrončok <mhroncok>
Component: python-twistedAssignee: Thomas Vander Stichele <thomas>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: thomas
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-06-13 14:14:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1565020    
Attachments:
Description Flags
build.log from copr
none
root.log from copr none

Description Miro Hrončok 2018-05-29 11:46:40 UTC
Created attachment 1445368 [details]
build.log from copr

Description of problem: python-twisted FTBFS with Python 3.7 bacuse it uses async (now a keyword) as a function parameter name.

Bytecompiling .py files below /builddir/build/BUILDROOT/python-twisted-16.4.1-9.fc29.x86_64/usr/lib64/python3.7 using /usr/bin/python3.7
*** Error compiling '/builddir/build/BUILDROOT/python-twisted-16.4.1-9.fc29.x86_64/usr/lib64/python3.7/site-packages/twisted/conch/manhole.py'...
  File "/usr/lib64/python3.7/manhole.py", line 133
    def write(self, data, async=False):
                              ^
SyntaxError: invalid syntax


Version-Release number of selected component (if applicable): 16.4.1-9.fc29

This is blocking other packages.

Build logs attached.

Comment 2 Miro Hrončok 2018-05-29 11:51:31 UTC
Created attachment 1445370 [details]
root.log from copr

Comment 3 Miro Hrončok 2018-06-04 13:08:29 UTC
Thomas?

Comment 4 Miro Hrončok 2018-06-05 16:10:03 UTC
See proposed upstream fix: https://github.com/twisted/twisted/pull/966

Comment 5 Miro Hrončok 2018-06-08 10:41:39 UTC
FYI: 3.7.0c1 is expected in 3 days (2018-06-11, [1]). I'd like to start building stuff in a side tag soon after that.

This blocks the "packager stack", hence a bit critical.

Thomas, could you please respond?


[1] https://www.python.org/dev/peps/pep-0537/

Comment 6 Miro Hrončok 2018-06-11 13:42:04 UTC
I'm trying to backport https://github.com/twisted/twisted/pull/966 to see if that helps. This package does not run tests, so I will simply try to rebuild the dependent packages to verify if "works".

Comment 7 Miro Hrončok 2018-06-11 14:04:18 UTC
txzmq:

    + nosetests-3.7
    ...
    Ran 36 tests in 3.346s


python-moksha-hub:


    + /usr/bin/python3 setup.py test
    ...
    Ran 58 tests in 60.315s

Comment 8 Miro Hrončok 2018-06-11 14:07:55 UTC
Patch ready at https://src.fedoraproject.org/rpms/python-twisted/pull-request/5

Still waiting for "important" packages to build.

Comment 9 Miro Hrončok 2018-06-11 14:58:54 UTC
+ py.test-3 fedmsg
============= 222 passed, 102 skipped, 1 warnings in 14.80 seconds =============


I think we are good in here.

Comment 10 Miro Hrončok 2018-06-13 14:14:51 UTC
Pushed the dirty patch.

Comment 11 Red Hat Bugzilla 2023-09-14 04:29:06 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days