Bug 783064

Summary:	Review Request: python-omniORB - A robust high performance CORBA ORB for Python
Product:	[Fedora] Fedora	Reporter:	Haïkel Guémar <karlthered>
Component:	Package Review	Assignee:	Michael S. <misc>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	eberhard.leba, hobbes1069, jaromir.talir, manisandro, misc, package-review, sergio.pasra
Target Milestone:	---	Flags:	misc: fedora-review?
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-02-16 17:46:10 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:	783061
Bug Blocks:

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
SRPM URL:
http://hguemar.fedorapeople.org/diet/python-omniORB-3.6-1.fc16.src.rpm

Comment 3 Michael S. 2012-03-24 19:45:19 UTC

You should add the url to download the tarball :
http://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net

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.
files:
/usr/lib/python2.6/site-packages/omniidl_be/python.py
/usr/lib/python2.6/site-packages/omniidl_be/python.pyc
/usr/lib/python2.6/site-packages/omniidl_be/python.pyo

# -*- 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
SRPM URL:
http://smani.fedorapeople.org/review/omniORBpy-3.7-1.fc21.src.rpm

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 ***