Bug 472303 - Review Request: python-ropevim - Python module to add rope refactoring to vim
Summary: Review Request: python-ropevim - Python module to add rope refactoring to vim
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Brian Pepple
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 472269
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-11-19 22:31 UTC by Toshio Ernie Kuratomi
Modified: 2010-03-10 18:08 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-03-10 18:08:12 UTC
Type: ---
Embargoed:
bdpepple: fedora-review+


Attachments (Terms of Use)

Description Toshio Ernie Kuratomi 2008-11-19 22:31:30 UTC
Spec URL: http://toshio.fedorapeople.org/packages/python-ropevim.spec
SRPM URL: http://toshio.fedorapeople.org/packages/python-ropevim-0.2-1.fc9.src.rpm
Description:

Ropevim provides a python module to integrate the rope python refactoring
library with vim.

Comment 1 Brian Pepple 2008-11-22 19:33:40 UTC
MD5Sum:
42390b31187a6ac5f06b5148af661f97  ropevim-0.2.tar.gz

Good:
* Source URL is canonical
* Upstream source tarball verified
* Package name conforms to the Fedora Naming Guidelines
* Group Tag is from the official list
* Valid license tag
* Buildroot has all required elements
* All paths begin with macros
* All necessary BuildRequires listed.
* Package builds in Mock.
* rpmlint produces no errors.
* Package installs and uninstalls cleanly.

Minor:
* could look at making summary a bit less verbose.

+1 APPROVED

Comment 2 Toshio Ernie Kuratomi 2010-03-10 18:07:52 UTC
I wasn't able to get the integration to work bug-free enough that I was comfortable supporting this so I've never imported it.  It's been quite a while and I realize I don't have the time to work on making it work so i'm closing this review request -- someone else can take my work and update it if they feel they need to.


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