Bug 1252140 - twisted/plugins shipped by python-twisted
Summary: twisted/plugins shipped by python-twisted
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-twisted
Version: 24
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Thomas Vander Stichele
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1244522
TreeView+ depends on / blocked
 
Reported: 2015-08-10 19:48 UTC by Julien Enselme
Modified: 2016-06-26 20:38 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-06-26 20:38:10 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Julien Enselme 2015-08-10 19:48:04 UTC
Hi,

I am in the process of packaging python-autobahn (http://autobahn.ws/, https://bugzilla.redhat.com/show_bug.cgi?id=1244522). Today, I run fedora-review on my package to see what I could improve in my package and it reported:

[ ]: Package does not own files or directories owned by other packages.
     Note: Dirs in package are owned also by: /usr/lib/python2.7/site-
     packages/twisted(python-carbon), /usr/lib/python2.7/site-
     packages/twisted/plugins(python-carbon)

It seems that we both need to place files in /usr/lib/python2.7/site-packages/twisted/plugins (I also need /usr/lib/python3.4/site-packages/twisted/plugins).

I think that the easiest way to solve this problem is to create a pytho-twisted-plugin package that would own /usr/lib/python2.7/site-packages/twisted (same for python3 + __pycache__) and /usr/lib/python2.7/site-packages/twisted/plugins and then make our packages require it. I don't expect conflict within the directory since our packages have different name.

What do you think?

Comment 1 Piotr Popieluch 2015-08-11 20:40:22 UTC
Wouldn't it be easier to add the directory it to the python-twisted package. So we won't have to create empty packages and go through the review process.

Are you also planning to package autobahn in epel?

Jonathan, what do you, as maintainer of python-twisted, think?

Comment 2 Julien Enselme 2015-08-12 20:03:50 UTC
> Wouldn't it be easier to add the directory it to the python-twisted package. So we won't have to create empty packages and go through the review process.

It may. I admit I haven't thought of that.

> Are you also planning to package autobahn in epel?

Nope

Comment 3 Zbigniew Jędrzejewski-Szmek 2015-08-13 03:08:53 UTC
FWIW, I think that having this directory co-owned by various plugin packages is fine. Adding the directory to python-twisted is also OK.

Comment 4 Jonathan Steffan 2015-08-15 16:11:57 UTC
I think it makes the most sense to create and own this directory in python-twisted.

Comment 5 Jonathan Steffan 2015-08-15 16:21:00 UTC
I will add this directory when I fix https://bugzilla.redhat.com/show_bug.cgi?id=1222954

Comment 6 Julien Enselme 2015-08-15 17:15:13 UTC
@Jonathan: sounds good. Keep me posted so I can update my package.

Comment 7 Julien Enselme 2015-11-27 12:47:11 UTC
Hi,

Any progress on that issue?

Comment 8 Jan Kurik 2016-02-24 13:36:19 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 24 development cycle.
Changing version to '24'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora24#Rawhide_Rebase

Comment 9 Jonathan Steffan 2016-06-26 20:38:10 UTC
This directory is now owned by the package.


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