Bug 205031 - Review Request: python-telepathy - Python libraries for Telepathy
Summary: Review Request: python-telepathy - Python libraries for Telepathy
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Sander Hoentjen
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
: 205066 (view as bug list)
Depends On:
Blocks: FE-ACCEPT 205343
TreeView+ depends on / blocked
 
Reported: 2006-09-02 18:06 UTC by Brian Pepple
Modified: 2010-07-21 05:02 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-09-08 13:52:07 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Brian Pepple 2006-09-02 18:06:33 UTC
Spec URL: http://piedmont.homelinux.org/fedora/telepathy/python-telepathy.spec
SRPM URL: http://piedmont.homelinux.org/fedora/telepathy/python-telepathy-0.13.2-1.src.rpm
Description: Python libraries for use in Telepathy clients and connection managers.

Comment 1 Brian Pepple 2006-09-03 13:14:42 UTC
*** Bug 205066 has been marked as a duplicate of this bug. ***

Comment 2 Sander Hoentjen 2006-09-03 13:42:54 UTC
Spec file is basically the same as I had, only you used the correct naming for
Fedora. Also the URL *could* be changed to
http://telepathy.freedesktop.org/wiki/TelepathyPython but that doesn't really
matter.
I also had the examples packaged under docs, but I am not even sure if that is a
good thing.
I would approve, but I am still not in fedorabugs so I'll leave the review part
until i am, maybe someone else will pick up before then.

Comment 3 Sander Hoentjen 2006-09-08 11:03:15 UTC
GOOD
- package meets naming and packaging guidelines.
- specfile is properly named, is cleanly written and uses macros consistently.
- dist tag is present.
- build root is correct.
- license field matches the actual license.
- license is open source-compatible.  License text included in package.
- source files match upstream: 39a07295c78699da6efc1e4638c53688
- BuildRequires are proper.
- package builds in mock (x86_64).
- rpmlint is silent.
- final provides and requires are sane:
 python-telepathy-0.13.2-1.fc6.noarch.rpm
  python-telepathy = 0.13.2-1.fc6
 =
  python(abi) = 2.4
- no shared libraries are present.
- package is not relocatable.
- owns the directories it creates.
- doesn't own any directories it shouldn't.
- no duplicates in %files.
- file permissions are appropriate.
- %clean is present.
- no scriptlets present.
- code, not content.
- documentation is small, so no -docs subpackage is necessary.
- %docs are not necessary for the proper functioning of the package.
- no headers.
- no pkgconfig files.
- no libtool .la droppings.
- not a GUI app.
- not a web app.

APPROVED

Comment 4 Brian Pepple 2007-10-12 16:51:39 UTC
Package Change Request
======================
Package Name: python-telepathy
New Branches: F-8

Comment 5 Kevin Fenzi 2007-10-12 17:41:39 UTC
cvs done.

Comment 6 Brian Pepple 2008-11-10 14:35:56 UTC
New Package CVS Request
=======================
Package Name: python-telepathy 
Short Description: Python bindings for Telepathy
Owners: morgan.collett, tomeu 
Branches: OLPC-3

Comment 7 Kevin Fenzi 2008-11-10 17:02:01 UTC
cvs done.

Comment 8 Peter Robinson 2010-06-10 20:32:25 UTC
Package Change Request
======================
Package Name: python-telepathy
New Branches: EL-6
Owners: pbrobinson sdz

Comment 9 Sebastian Dziallas 2010-07-20 23:30:24 UTC
Package Change Request
======================
Package Name: python-telepathy
New Branches: EL-6
Owners: pbrobinson sdz bpepple

I emailed Brian and he said that it would probably be better if someone else
maintained the EPEL branches, since he's not currently running any systems that
use EPEL. If you need additional confirmation, can you comment here please,
Brian? For background, we're interested in maintaining these in EPEL since they
are dependencies of the Sugar Environment, which we're trying to push into
EPEL.

Comment 10 Kevin Fenzi 2010-07-21 05:02:11 UTC
CVS done (by process-cvs-requests.py).


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