Bug 490028 - Review Request: pytc - Python bindings for Tokyo Cabinet
Summary: Review Request: pytc - Python bindings for Tokyo Cabinet
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-03-12 21:20 UTC by Silas Sewell
Modified: 2009-03-31 07:39 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-03-31 07:39:36 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Silas Sewell 2009-03-12 21:20:38 UTC
Spec URL:
http://silassewell.googlecode.com/svn/trunk/projects/packages/rpms/python-tokyocabinet/python-tokyocabinet.spec
SRPM URL:
http://silassewell.googlecode.com/files/python-tokyocabinet-0.7-1.src.rpm
Description: 
Python bindings for Tokyo Cabinet.

I'll need a sponsor.

Comment 1 Mamoru TASAKA 2009-03-23 15:44:05 UTC
Some notes:

* %define -> %global
  - Now Fedora suggests to use %global instead of %define, especially
    for the nested macro definition. ref:
    https://fedoraproject.org/wiki/Packaging/Python#System_Architecture
    https://fedoraproject.org/wiki/PackagingDrafts/global_preferred_over_define
    (The latter one says "Draft", however it is already accepted by
    FESCo).

* Naming
  - Fedora prefers to use the tarball name as rpm name as
    much as possible. For this package I recommend to use
    "pytc" as rpm name.

* Requires
----------------------------------------------------------
Requires:       python >= 2.3
Requires:       tokyocabinet
----------------------------------------------------------
  - Both of these are unnecessary. These dependencies are
    automatically pulled in by libraries' dependencies
    caught by rpmbuild itself.

Comment 3 Mamoru TASAKA 2009-03-27 15:22:58 UTC
This one can be accepted.

---------------------------------------------------
  This package (pytc) is APPROVED by mtasaka
---------------------------------------------------

Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from "Get a Fedora Account".
After you request for sponsorship a mail will be sent to sponsor 
members automatically (which is invisible for you) which notifies 
that you need a sponsor. After that, please also write on
this bug for confirmation that you requested for sponsorship and
your FAS (Fedora Account System) name. Then I will sponsor you.

If you want to import this package into Fedora 9/10, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on koji Fedora rebuilding system).

If you have questions, please ask me.

Comment 4 Silas Sewell 2009-03-27 15:45:00 UTC
Thanks Mamoru, I have requested sponsorship and my FAS username is "silas".

Comment 5 Mamoru TASAKA 2009-03-27 16:01:14 UTC
Okay, now I am sponsoring you. Please follow "Join" wiki
again.

Comment 6 Silas Sewell 2009-03-28 06:11:00 UTC
New Package CVS Request
=======================
Package Name: pytc
Short Description: Tokyo Cabinet Python bindings
Owners: silas
Branches: 
InitialCC:

Comment 7 Kevin Fenzi 2009-03-30 21:50:03 UTC
cvs done.

Comment 8 Silas Sewell 2009-03-31 07:39:36 UTC
Build for F-11, closing.

Thanks Mamoru and Kevin.


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