Bug 1634995

Summary: liblouis: Remove (sub)packages from Fedora 30+: python2-louis
Product: [Fedora] Fedora Reporter: Miro Hrončok <mhroncok>
Component: liblouisAssignee: Martin Gieseking <martin.gieseking>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: cstratak, igor.raits, martin.gieseking, mhroncok, ngompa13, pviktori, zbyszek
Target Milestone: ---   
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: 2018-10-13 08:19:59 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: 1625773    

Description Miro Hrončok 2018-10-01 19:48:44 UTC
In line with the Mass Python 2 Package Removal [0], the following (sub)packages of liblouis were marked for removal:

 * python2-louis

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 Miro Hrončok 2018-10-11 14:16:30 UTC
python2-louis is needed by liblouis-utils. Can liblouis-utils switch to python3-louis?

Comment 2 Martin Gieseking 2018-10-11 15:10:09 UTC
Yes, that shouldn't be a problem since the script checks whether Python 2 or 3 is available. I just roughly tested lou_harnessGenerator (the only Python script in the utils package) with Python 3 and it works. If you'd like to adapt the spec file, feel free to do so. Otherwise, I'll fix it next weekend.

Comment 3 Miro Hrončok 2018-10-11 15:14:19 UTC
Next weekend is perfectly fine, I'd be happier if you do it (to do it "your way"). Feel free to ask for a review if you feel like you'd need one.

Changing to ASSIGNED so recognize more easily that this will be handled.

Thank you so much!

Comment 4 Martin Gieseking 2018-10-13 08:19:59 UTC
I've removed python2-louis and also dropped the Python dependency from the utils package because the 3.x release series no longer provides any utilities written in Python.