Bug 494853 - Review Request: python-achoo - A fluent interface for testing Python objects
Summary: Review Request: python-achoo - A fluent interface for testing Python objects
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-04-08 11:49 UTC by Ramakrishna Reddy
Modified: 2009-04-29 08:46 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-29 08:46:01 UTC
Type: ---
Embargoed:
panemade: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Ramakrishna Reddy 2009-04-08 11:49:46 UTC
Spec URL: http://ramakrishnareddy.info/fedora/python-achoo.spec
SRPM URL: http://ramakrishnareddy.info/fedora/python-achoo-1.0-1.fc11.src.rpm
Description: 
Achoo is a fluent interface for unit testing Python objects.
Achoo makes it easy to make assertions about the properties 
and behaviours of Python objects. For example, is this object
equal to this other one? Does calling this function with these
arguments raise an error? More Infomation on fluent from here
http://www.martinfowler.com/bliki/FluentInterface.html

Comment 1 Parag AN(पराग) 2009-04-09 09:23:46 UTC
1)unable to verify sources.
2030a24b9daa0232aa776c08461e8061  achoo-1.0.tar.gz <= from upstream
00bcb086f715c41821d22748d123308c  achoo-1.0.tar.gz <= from SRPM

Please add correct upstream sources while building SRPM.

submit new SRPM with adding Changelog and increasing release to -2

Changelog should add like "Add correct upstream source"

Comment 2 Ramakrishna Reddy 2009-04-09 10:16:57 UTC
1. Added the correct Source file.  
* http://ramakrishnareddy.info/fedora/python-achoo-1.0-2.fc10.src.rpm

2. Updated the SPEC file with the changelog entries 
* http://ramakrishnareddy.info/fedora/python-achoo.spec

Comment 3 Parag AN(पराग) 2009-04-15 05:47:54 UTC
Review:
+ package builds in mock.
 Koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1299561
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
2030a24b9daa0232aa776c08461e8061  achoo-1.0.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Not a GUI app.
  
APPROVED.

Comment 4 Ramakrishna Reddy 2009-04-17 07:44:47 UTC
New Package CVS Request
=======================
Package Name: python-achoo
Short Description: A fluent interface for testing Python Object
Owners: ramkrsna
Branches: F-10 F-11 
InitialCC:

Comment 5 Kevin Fenzi 2009-04-17 21:36:25 UTC
cvs done.


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