Bug 1716480 - pidgin fails to build with Python 3.8
Summary: pidgin fails to build with Python 3.8
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pidgin
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
Assignee: Jaroslav Škarvada
QA Contact: Fedora Extras Quality Assurance
URL: https://copr.fedorainfracloud.org/cop...
Whiteboard:
Depends On:
Blocks: PYTHON38
TreeView+ depends on / blocked
 
Reported: 2019-06-03 14:36 UTC by Miro Hrončok
Modified: 2019-06-04 09:01 UTC (History)
5 users (show)

Fixed In Version: pidgin-2.13.0-11.fc31
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-06-04 09:01:26 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Miro Hrončok 2019-06-03 14:36:50 UTC
See https://copr.fedorainfracloud.org/coprs/g/python/python3.8/package/pidgin/ for actual logs. This report is automated and not very verbose, but feel free to ping me for help.

Comment 1 Jaroslav Škarvada 2019-06-03 23:26:13 UTC
It looks like python3.8 problem:
$ python3-config --libs
 -lcrypt -lpthread -ldl  -lutil -lm -lm 

With python3.7:
$ python3-config --libs
-lpython3.7m -lcrypt -lpthread -ldl  -lutil -lm 

I.e. -lpython3.8m is missing, not counting that -lm is there two times.

Comment 2 Miro Hrončok 2019-06-03 23:37:33 UTC
-lpython3.8 missing is a deliberate change in 3.8, see https://docs.python.org/dev/whatsnew/3.8.html#debug-build-uses-the-same-abi-as-release-build

-lm being there two times is because we have reverted a downstream only patch that was supposed to remove duplicates there but was too greedy and broke other things, should be harmless.

If I understand the use case in pidgin correctly, it should use:

$ python3-config --libs --embed

Comment 3 Jaroslav Škarvada 2019-06-04 07:33:31 UTC
(In reply to Miro Hrončok from comment #2)
> -lpython3.8 missing is a deliberate change in 3.8, see
> https://docs.python.org/dev/whatsnew/3.8.html#debug-build-uses-the-same-abi-
> as-release-build
> 
> -lm being there two times is because we have reverted a downstream only
> patch that was supposed to remove duplicates there but was too greedy and
> broke other things, should be harmless.
> 
> If I understand the use case in pidgin correctly, it should use:
> 
> $ python3-config --libs --embed

Thanks for info, I will send patch upstream.

Comment 4 Jaroslav Škarvada 2019-06-04 08:14:19 UTC
Upstream ticket:
https://developer.pidgin.im/ticket/17396

Comment 5 Jaroslav Škarvada 2019-06-04 09:01:26 UTC
Fixed in pidgin-2.13.0-11.fc31


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