Bug 783064 - Review Request: python-omniORB - A robust high performance CORBA ORB for Python
Summary: Review Request: python-omniORB - A robust high performance CORBA ORB for Python
Status: CLOSED DUPLICATE of bug 1065758
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Michael S.
QA Contact: Fedora Extras Quality Assurance
Depends On: 783061
TreeView+ depends on / blocked
Reported: 2012-01-19 08:57 UTC by Haïkel Guémar
Modified: 2014-02-16 17:46 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-02-16 17:46:10 UTC
Type: ---
misc: fedora-review?

Attachments (Terms of Use)

Description Haïkel Guémar 2012-01-19 08:57:09 UTC
Spec URL: http://hguemar.fedorapeople.org/diet/python-omniORB.spec
SRPM URL: http://hguemar.fedorapeople.org/diet/python-omniORB-3.6-1.fc16.src.rpm
Description: omniORB is a CORBA 2.6 compliant open source implementation for python (requires omniORB to be reviewed first)

Comment 1 Michael S. 2012-03-24 14:46:21 UTC
Is this package planned to be sent to EPEL ?

If not, can you :
- remove BuildRoot
- remove %defattr, %clean and the rm in %install

For python, you need to say what version of python you need ( ie python2 or 3 ) for the buildRequires.

You can also drop the snippet for python_sitelib, as they are now already set.

The patch should have a mention of being sent upstream ( url, mail ).

Comment 2 Haïkel Guémar 2012-03-24 19:09:21 UTC
Yes I plan to push this into EPEL5+ (since omniORB is already there). I fixed the BR (didn't know that EL5 python already provided python2), python_sitelib is only defined for EL5. A mail has been sent to the maintainer for both omniORB and python-omniORB about the FSF address issue, a comment was added.

new spec and src.rpm: 
Spec URL: http://hguemar.fedorapeople.org/diet/python-omniORB.spec

Comment 3 Michael S. 2012-03-24 19:45:19 UTC
You should add the url to download the tarball :

And rpmlint complain :
python-omniORB-examples.noarch: W: doc-file-dependency /usr/share/doc/python-omniORB-examples-3.6/examples/poa/mainthread.py /usr/bin/env

python-omniORB-debuginfo.i686: E: debuginfo-without-sources

python-omniORB.i686: W: private-shared-object-provides /usr/lib/python2.7/site-packages/_omnicodesetsmodule.so.3.6 _omnicodesetsmodule.so.3

Comment 4 Haïkel Guémar 2012-03-24 21:07:45 UTC
1. download url fixed
2. for the examples, i'm unsure of where i should put them (especially since they're noarch). I'm open to any suggestions.
3. the provides is now filtered and should not appear anymore

spec and src.rpm updated. (still the same url)

Comment 5 Michael S. 2012-03-25 09:04:12 UTC
For the example, wouldn't removing the permission to execute be sufficient ?

The debuginfo issue is maybe listed on https://fedoraproject.org/wiki/Packaging:Debuginfo , and is a MUST, so should be fixed.

Comment 6 Eberhard Leba 2012-04-02 09:23:46 UTC
Seems the omniORBpy-devel package is missing.

# -*- python -*-
#                           Package   : omniidl
# python.py                 Created on: 1999/10/29
#                           Author    : Duncan Grisby (dpg1)
# Description:
#   Back-end for Python

Comment 7 Jaromír Talíř 2012-05-08 07:59:36 UTC
Is there any progress in having python-omniorb in Fedora together with the rest of omniorb packages? Any help needed?

Comment 8 Richard Shaw 2014-02-16 02:55:42 UTC
I'm not sure how successful I'll be just yet, but I'm attempting to package salome which requires this package. Let me know if you need any help.

Comment 9 Sandro Mani 2014-02-16 15:50:31 UTC
Haïkel, if you are not actively working on this anymore, I'm happy to continue the effort. I have an updated SPEC and SRPM here:
Spec URL: http://smani.fedorapeople.org/review/python-omniORB.spec

Comment 10 Haïkel Guémar 2014-02-16 17:35:35 UTC
Thank you Sandro, feel free to take over (may i offer you we exchange acls, since i already maintain omniORB ?).
I'm on the verge of burnout and won't cling to a package that is very low in my priority list.

Comment 11 Sandro Mani 2014-02-16 17:43:54 UTC
No problem with comaintaining omniORB (is this is what you meant by "exchanging acls"?). As for this package, I'll start a new review-request and close this as duplicate. Take care.

Comment 12 Sandro Mani 2014-02-16 17:46:10 UTC

*** This bug has been marked as a duplicate of bug 1065758 ***

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