Bug 1632330 - OpenImageIO: Remove (sub)packages from Fedora 30+: python2-openimageio
Summary: OpenImageIO: Remove (sub)packages from Fedora 30+: python2-openimageio
Alias: None
Product: Fedora
Classification: Fedora
Component: OpenImageIO
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Richard Shaw
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2018-09-24 15:33 UTC by Miro Hrončok
Modified: 2018-10-02 16:53 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2018-10-02 16:53:07 UTC

Attachments (Terms of Use)

Description Miro Hrončok 2018-09-24 15:33:29 UTC
In line with the Mass Python 2 Package Removal [0], the following (sub)packages of OpenImageIO were marked for removal:

 * python2-openimageio

According to our query, those (sub)packages only provide a Python 2 importable module. If this is not true, please tell us why, so we can fix our query.

Please remove them from your package.

As said in the change document, if there is no objection in a week, we will remove the package(s) as soon as we get to it. This change might not match your packaging style, so we'd prefer if you did the change. If you need more time, please let us know here.

We hope this doesn't come to you as a surprise. If you want to know our motivation for this, please read the change document [0].

[0] https://fedoraproject.org/wiki/Changes/Mass_Python_2_Package_Removal

Comment 1 Richard Shaw 2018-09-24 15:43:02 UTC
I need to verify with upstream first but I may be able to switch it to Python3 but I'm also not sure what all the "consumers" are of the Python2 package...

Comment 2 Miro Hrončok 2018-09-24 16:06:54 UTC
repoquery yields nothing:

$ dnf repoquery --repo=rawhide-source --whatrequires python2-openimageio
$ dnf repoquery --repo=rawhide --whatrequires python2-openimageio

Comment 3 Richard Shaw 2018-09-24 16:11:03 UTC
I've already completed a local x86_64 mock build successfully and just kicked of a scratch build for the other arches.

Comment 4 Miro Hrončok 2018-09-24 17:05:16 UTC
I wonder whether we can do the same for OpenColorIO? Only blender depends on it, but that runs on Python 3 so I doubt it uses the Python 2 bits.

Comment 5 Richard Shaw 2018-09-24 18:43:08 UTC
Checking Koschi I don't see any direct dependency.


I'll try a scratch build of it next.

Comment 6 Richard Shaw 2018-09-24 20:26:55 UTC
OpenColorIO built fine but I just realized that the python library is with the main library... Not sure why I did that but not worth splitting off at this point. 

# dnf repoquery --whatrequires /usr/lib64/python2.7/site-packages/PyOpenColorIO.so
Last metadata expiration check: 0:01:09 ago on Mon 24 Sep 2018 03:18:40 PM CDT.

It looks like there are 4 consumers of the OCIO library:
- Itself
- OpenImageIO
- Blender
- krita

I'm doing local mock builds of OIIO and OCIO and will pull down SRPMS of blender and krita if that succeeds.

Comment 7 Miro Hrončok 2018-09-24 20:31:46 UTC
Not sure what results is this supposed to show:

dnf repoquery --whatrequires /usr/lib64/python2.7/site-packages/PyOpenColorIO.so

But blender requires just libOpenColorIO.so.1()(64bit) and so does krita.

Comment 8 Richard Shaw 2018-09-24 20:39:07 UTC
So theoretically I wouldn't have to rebuilt those two... I can't do a repoquery search on "PyOpenColorIO.so()(64bit)" because the python library is filtered from the provides of the main OpenColorIO package.

Comment 9 Richard Shaw 2018-09-24 20:49:51 UTC
Ok, it looks like the python modules for both OCIO and OIIO are unused in Fedora. Both rebuild fine so I'm going to kick off official builds.

Comment 10 Richard Shaw 2018-09-24 20:51:47 UTC
Since OCIO put it in the main package I don't need to worry about it but in the case of OIIO, do I need to add an Obsoletes: for the python2 package?

Comment 11 Miro Hrončok 2018-09-24 21:14:29 UTC
Into fedora-obsolete-packages. But don't worry about it, I do it in batches.

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