Bug 1638488 - Proposed python-pygraphviz update
Summary: Proposed python-pygraphviz update
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: python-pygraphviz
Version: rawhide
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-10-11 17:49 UTC by Jerry James
Modified: 2019-02-09 02:12 UTC (History)
2 users (show)

Fixed In Version: python-pygraphviz-1.5-1.fc29
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-09 02:12:30 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Proposed spec file modifications (6.34 KB, text/plain)
2018-10-11 17:49 UTC, Jerry James
no flags Details
Proposed patch to the swig interface (1.28 KB, patch)
2018-10-11 17:50 UTC, Jerry James
no flags Details | Diff

Description Jerry James 2018-10-11 17:49:32 UTC
Created attachment 1492990 [details]
Proposed spec file modifications

Description of problem:
I am attempting to update python-networkx to version 2.2 in Rawhide.  However, the test suite is failing with errors being thrown from pygraphviz when built with python 3.  Upstream seems to have fixed the problems.  I would like to propose that python-pygraphviz be updated to version 1.5.  I will attach a modified spec file and a patch to the swig interface to make that work.

Version-Release number of selected component (if applicable):
N/A

How reproducible:
N/A

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Jerry James 2018-10-11 17:50:08 UTC
Created attachment 1492991 [details]
Proposed patch to the swig interface

Comment 2 Jerry James 2018-10-11 18:06:25 UTC
I was too hasty.  Apparently the top-level __init__.py imports the tests.  That seems quite strange.  Still, I guess you have to ship the tests after all, which means the dependency on nose is also needed.  Ugh.

Comment 3 Jerry James 2018-10-11 19:43:00 UTC
And, after all that, the python-networkx tests are still failing, but now it is because multigraph support is broken in graphviz itself.  See https://github.com/pygraphviz/pygraphviz/issues/162.  I guess I'm just going to have to disable %check for python-networkx until that issue is fixed...

I still need this pygraphviz update, though, to fix the python 3 issues.  Thanks!

Comment 4 Casey Deccio 2019-01-24 18:23:07 UTC
Among the problems with running pygraphviz 1.3 with python 3.7 are StopIteration errors, which are fixed in later versions of pygraphviz.  Now that the python2-pygraphviz package has officially been removed, it is important that pygraphviz work with python 3.7.  The python3-pygraphviz package in (at least) Fedora 29 is version 1.3 and therefore has compatibility bugs--but at there's a python2-pygraphviz package to fall back on.  Because the version of python3-graphviz package in Rawhide is also 1.3 and therefore has issues with python 3.7, but there is no python2 fallback, it's problematic.  This is especially important to me, as I have software that depends on pygraphviz, and without an updated version python3-pygraphviz, my software won't work with Fedora Rawhide.

Comment 5 Zbigniew Jędrzejewski-Szmek 2019-01-26 13:10:31 UTC
I pushed the proposed patch to rawhide and F29 (in F29 both packages are built, and in F30 only python3).
I'm leaning towards pushing an update in F29, because it doesn't seem useful to have a broken package. What do you think?

Comment 6 Casey Deccio 2019-01-26 14:21:11 UTC
Sounds like the right thing to do.  Thanks!

Comment 7 Fedora Update System 2019-01-26 14:42:26 UTC
python-pygraphviz-1.5-1.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-ce1b2603fc

Comment 8 Fedora Update System 2019-01-27 02:28:45 UTC
python-pygraphviz-1.5-1.fc29 has been pushed to the Fedora 29 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-ce1b2603fc

Comment 9 Fedora Update System 2019-02-09 02:12:30 UTC
python-pygraphviz-1.5-1.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.