Bug 1285258

Summary: [PATCH] Add python3-openbabel subpackage
Product: [Fedora] Fedora Reporter: Tomáš Trnka <tomastrnka>
Component: openbabelAssignee: Dominik 'Rathann' Mierzejewski <dominik>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dominik, itamar, paul, susi.lehtola
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-22 09:28:59 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:
Attachments:
Description Flags
Patch to openbabel.spec
none
Patch to openbabel.spec (fixed a typo) none

Description Tomáš Trnka 2015-11-25 10:07:05 UTC
Created attachment 1098650 [details]
Patch to openbabel.spec

Hello,

attached is a (draft) patch that builds the Python 3 bindings to OpenBabel in addition to the Python 2 version. Python 3 is fully supported upstream. I've built the packages in my COPR and they seem to work just fine: 
http://copr.fedoraproject.org/coprs/tootea/earlybird/build/142165/

The patch needs a slight tuning to work on non-x86_64 architectures: currently the  path to the lib64 subdir created by CMake is hardcoded in the setup of BABEL_SYSTEM_LIBRARY, but it should be trivial to come up with an universal solution (I'm just not experienced enough with Fedora spec macros to pick the best approach).

Best regards,

Tomáš Trnka

Comment 1 Tomáš Trnka 2015-11-25 10:09:42 UTC
Created attachment 1098653 [details]
Patch to openbabel.spec (fixed a typo)

Comment 2 Dominik 'Rathann' Mierzejewski 2016-02-18 15:14:23 UTC
Thank you for the patch. I'll include this in the next package revision.

Comment 3 Dominik 'Rathann' Mierzejewski 2016-02-22 09:28:59 UTC
Done in rawhide. Do you want a backport to F23?

Comment 4 Tomáš Trnka 2016-02-22 09:48:28 UTC
Thanks a lot. I don't personally need it for F23 (I'm using the COPR build), so it's up to you.