Bug 1285258 - [PATCH] Add python3-openbabel subpackage
Summary: [PATCH] Add python3-openbabel subpackage
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: openbabel
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Dominik 'Rathann' Mierzejewski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-11-25 10:07 UTC by Tomáš Trnka
Modified: 2016-02-22 09:48 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-02-22 09:28:59 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to openbabel.spec (3.63 KB, patch)
2015-11-25 10:07 UTC, Tomáš Trnka
no flags Details | Diff
Patch to openbabel.spec (fixed a typo) (3.62 KB, patch)
2015-11-25 10:09 UTC, Tomáš Trnka
no flags Details | Diff

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.


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