Bug 1748527

Summary: Consider moving sip-api provides from python2-sip to python3-sip
Product: [Fedora] Fedora Reporter: Kevin Fenzi <kevin>
Component: sipAssignee: Rex Dieter <rdieter>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: kde-sig, mhroncok, rdieter, than, vstinner
Target Milestone: ---Keywords: Reopened
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: 2019-09-16 16:00:37 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: 1729593    

Description Kevin Fenzi 2019-09-03 19:11:54 UTC
Right now python2-sip provides:

python2-sip = 4.19.18-5.fc32
python2-sip(x86-64) = 4.19.18-5.fc32
python2-sip-api(12) = 12.6
python2-sip-api(12)(x86-64) = 12.6
python2.7dist(sip) = 4.19.18
python2dist(sip) = 4.19.18
sip-api(12) = 12.6
sip-api(12)(x86-64) = 12.6

and python3-sip provides: 

python3-sip = 4.19.18-5.fc32
python3-sip(x86-64) = 4.19.18-5.fc32
python3-sip-api(12) = 12.6
python3-sip-api(12)(x86-64) = 12.6
python3.8dist(sip) = 4.19.18
python3dist(sip) = 4.19.18

This causes calibre to pull in python2-sip where all the rest of it is python3.
Looks like avogadro is also in this boat. 

Can we move the api provides over to the python3 package now?

Comment 1 Rex Dieter 2019-09-03 22:05:07 UTC
Consumers should probably move to the explicit python2-sip-api or python3-sip-api variants.  The sip-api one is kept for historical reasons only... I'd prefer to remove them rather than change it.

Comment 2 Rex Dieter 2019-09-03 22:12:29 UTC
I'll think on it some more... then work to implement that changes needed (I think it's a little more complicated)

Comment 3 Kevin Fenzi 2019-09-03 22:16:18 UTC
I can change calibre easily enough. 

diff --git a/calibre.spec b/calibre.spec                                                                     
index 1a5d7e2..9092935 100644                                                                                
--- a/calibre.spec                                                                                           
+++ b/calibre.spec                                                                                           
@@ -1,4 +1,4 @@                                                                                              
-%{?_sip_api:Requires: sip-api(%{_sip_api_major}) >= %{_sip_api}}                                            
+%{?_sip_api:Requires: python3-sip-api(%{_sip_api_major}) >= %{_sip_api}}                                    
                                                                                                             
 %global __provides_exclude_from ^%{_libdir}/calibre/calibre/plugins/.*\.so$                                                            
@@ -6,7 +6,7 @@                                                                                              
                                                                                                             
 Name:           calibre
 Version:        3.47.1
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        E-book converter and library manager
 License:        GPLv3
 URL:            https://calibre-ebook.com/
@@ -401,6 +401,9 @@ ln -s %{_jsdir}/mathjax %{_datadir}/calibre/viewer/
 %{_datadir}/metainfo/*.appdata.xml
  
 %changelog
+* Tue Sep 03 2019 Kevin Fenzi <kevin> - 3.47.1-2
+- Adjust sip requires to Require the python3-sip-api package.
+
 * Mon Sep 02 2019 Kevin Fenzi <kevin> - 3.47.1-1
 - Update 3.47.1. Fixes bug #1747848
  
I can push that for now, and we can adjust if you want to move the main sip-api?

Comment 4 Rex Dieter 2019-09-04 11:57:40 UTC
Thing is, I dont think that's the right fix.  You prob want sip-pyqt5 (or o e of it's provides)

Comment 5 Rex Dieter 2019-09-04 11:58:50 UTC
Arg, make that python3-sip-pyqt5

Comment 6 Kevin Fenzi 2019-09-15 23:04:11 UTC
Hum, I don't see anything providing that?

Comment 7 Rex Dieter 2019-09-16 13:13:24 UTC
https://src.fedoraproject.org/rpms/sip/blob/master/f/sip.spec#_157

is where it's at, committing change to calibre now

Comment 8 Rex Dieter 2019-09-16 13:24:30 UTC
calibre fixed:

https://src.fedoraproject.org/rpms/calibre/c/0a6632fccdbcc3949e3650d0beb2075e7d1c8ab6?branch=master


qgis fixed:

https://src.fedoraproject.org/rpms/qgis/c/858a2312dd642ba7511a11dceb1efca76ad59711?branch=master


Those are the last two (according to repoquery), so I think I can follow-through with the threat to remove the non-namespaced python-sip subpkgs (for f31+)

Comment 9 Rex Dieter 2019-09-16 14:04:43 UTC
 %changelog
+* Mon Sep 16 2019 Rex Dieter <rdieter> - 4.19.18-7
+- drop no_namespace variant for f31+

Comment 10 Miro HronĨok 2019-09-16 15:04:34 UTC
I just got this from koschei: nothing provides python3-sip needed by python3-arcus-4.1.0-3.fc32.x86_64

What should I require instead? I assume python3-pyqt5-sip.

Comment 11 Rex Dieter 2019-09-16 15:36:40 UTC
OK, I only repoquery'd for python3-sip-api , that's probably what python3-arcus should depend on.  I'll take a closer look.

Comment 12 Rex Dieter 2019-09-16 15:42:24 UTC
Arg, thouht this sounded familiar, seems upstream closed
https://github.com/Ultimaker/libArcus/issues/76
without actually changing/fixing anything. :(

So, our patch
libArcus-3.5.1-PyQt5.sip.patch

I *think* makes libarcus depend on namespaced python3-pyqt5-sip, I'll need to test that to confirm.

Comment 14 Victor Stinner 2019-09-17 08:25:56 UTC
> Those are the last two (according to repoquery), so I think I can follow-through with the threat to remove the non-namespaced python-sip subpkgs (for f31+)

enki package is also affected by this change: see rhbz#1752766.

Comment 15 Rex Dieter 2019-09-18 13:51:35 UTC
followup issues being tracked in bug #1753069

Comment 16 Fedora Update System 2019-10-25 18:38:08 UTC
FEDORA-2019-c919b2ffda has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2019-c919b2ffda

Comment 17 Fedora Update System 2019-10-27 04:02:23 UTC
libarcus-4.1.0-4.fc31 has been pushed to the Fedora 31 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-c919b2ffda

Comment 18 Fedora Update System 2019-11-01 17:01:21 UTC
FEDORA-2019-1dc2365f05 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2019-1dc2365f05

Comment 19 Fedora Update System 2019-11-03 06:01:40 UTC
libarcus-4.1.0-5.fc31, libsavitar-4.1.0-4.fc31 has been pushed to the Fedora 31 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-1dc2365f05

Comment 20 Fedora Update System 2019-11-06 13:45:10 UTC
libarcus-4.1.0-5.fc31, libsavitar-4.1.0-4.fc31 has been pushed to the Fedora 31 stable repository. If problems still persist, please make note of it in this bug report.