Bug 1716464
| Summary: | graphviz fails to build with Python 3.8 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Miro Hrončok <mhroncok> | ||||
| Component: | graphviz | Assignee: | Jaroslav Škarvada <jskarvad> | ||||
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | high | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | rawhide | CC: | alex, jima, jskarvad, tremble | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| URL: | https://copr.fedorainfracloud.org/coprs/g/python/python3.8/package/graphviz/ | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | graphviz-2.40.1-50.fc31 | Doc Type: | If docs needed, set a value | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2019-06-03 21:14:39 UTC | Type: | --- | ||||
| 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: | 1686977 | ||||||
| Attachments: |
|
||||||
|
Description
Miro Hrončok
2019-06-03 14:35:28 UTC
It failed in python2. Well we could probably improve the dual python building, but the following is probably the source of the problem: checking for python3... python3 checking Python.h usability... no checking Python.h presence... no checking for Python.h... no Is there python.h? The package requires python3-devel. If yes, I will probably need access to the build root to resolve this. I probably got it, it's because previous python had /usr/include/python3.7m and it seems the path is now /usr/include/python3.8 (i.e. without the 'm'). Created attachment 1576733 [details]
Proposed fix
Please try with the following patch and let me know.
Will try. Why was that in there in the first place? python3-config --includes gives you the proper flags. Hmm, it seems it hardcodes. I will try fixing it later. The patch ends with the same error.
checking for python3... python3
checking Python.h usability... yes
checking Python.h presence... yes
checking for Python.h... yes
...
/bin/sh ../../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -I. -I../.. -I../.. -I../../lib/gvc -I../../lib/common -I../../lib/cgraph -I../../lib/cdt -I../../lib/pathplan -DDEMAND_LOADING=1 -I/usr/include/python3.8m -I/usr/include/x86_64-linux -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-strict-aliasing -fno-strict-overflow -c -o libgv_python_la-gv_python.lo `test -f 'gv_python.cpp' || echo './'`gv_python.cpp
libtool: compile: g++ -DHAVE_CONFIG_H -I. -I../.. -I../.. -I../../lib/gvc -I../../lib/common -I../../lib/cgraph -I../../lib/cdt -I../../lib/pathplan -DDEMAND_LOADING=1 -I/usr/include/python3.8m -I/usr/include/x86_64-linux -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-strict-aliasing -fno-strict-overflow -c gv_python.cpp -fPIC -DPIC -o .libs/libgv_python_la-gv_python.o
gv_python.cpp:178:11: fatal error: Python.h: No such file or directory
178 | # include <Python.h>
| ^~~~~~~~~~
compilation terminated.
If you need access to the buildroot, use the mock config from https://copr.fedorainfracloud.org/coprs/g/python/python3.8/
(In reply to Miro Hrončok from comment #7) > If you need access to the buildroot, use the mock config from > https://copr.fedorainfracloud.org/coprs/g/python/python3.8/ Thanks, I will check in the mock. Fixed in graphviz-2.40.1-50.fc31 (tested in mock). |