Bug 481725

Summary: Review Request: python-setupdocs - Setuptools plugin
Product: [Fedora] Fedora Reporter: Rakesh Pandit <rpandit>
Component: Package ReviewAssignee: José Matos <jamatos>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, jamatos, jochen, notting, orion
Target Milestone: ---Flags: jamatos: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.0.1-3.fc10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-02-02 15:05:07 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 481727, 481734, 481738, 481759, 482216    

Description Rakesh Pandit 2009-01-27 11:48:58 UTC
Description:
Setuptools plugin that automates building of docs from ReST source.

SPEC: http://rakesh.fedorapeople.org/spec/python-setupdocs.spec
SRPM: http://rakesh.fedorapeople.org/srpm/python-setupdocs-1.0.1-1.fc10.src.rpm

Required for packaging Mayavi

Comment 1 José Matos 2009-01-27 16:37:56 UTC
The package is simple (as it should :-) ).

Some quirks,

In the the spec file the URL should be just
http://pypi.python.org/pypi/setupdocs

In the next line the comment there should be also removed:
#md5...

My suggestion regarding the description is to change it to

%description
Python setuptools plugin...

That is just to add Python to the begin of the sentence. I am aware that this is redundant but it does not hurt. :-) Anyway this is a suggestion and you are free to ignore it.

Other than that the package is fine.

Comment 3 José Matos 2009-01-27 18:30:46 UTC
OK.

The package follows the guidelines.
The package name is OK.
The license (BSD) is correct and acceptable.

The source tar ball is the same as shipped upstream.

The package is APPROVED.

Comment 4 Jochen Schmitt 2009-01-28 16:56:44 UTC
Unfortunately I have found a mistake in the package

The packaging guidlines says, theat the egg-info files should be build from source. So it may be helpful to remove the setupdocs.egg-info directory from source in the %setup stanza.

Comment 5 José Matos 2009-01-28 17:14:01 UTC
Argh... good catch.

I have examined the source ball egg-info dir and I have forgot to do the same in the generated rpm as I was expecting for setuptools to do the right thing. My mistake. :-(

Comment 7 Rakesh Pandit 2009-01-29 08:35:07 UTC
It seems even if I donn't remove eggs folder setuptools takes care of it by creating it from source:

log snippet when it was not removed:
=====================
running egg_info
writing setupdocs.egg-info/PKG-INFO
writing top-level names to setupdocs.egg-info/top_level.txt
writing dependency_links to setupdocs.egg-info/dependency_links.txt
writing entry points to setupdocs.egg-info/entry_points.txt
reading manifest file 'setupdocs.egg-info/SOURCES.txt'
writing manifest file 'setupdocs.egg-info/SOURCES.txt'
Copying setupdocs.egg-info to /home/redhat/rpm/BUILDROOT/python-setupdocs-1.0.1-3.fc10.i386/usr/lib/python2.5/site-packages/setupdocs-1.0.1-py2.5.egg-info
running install_scripts
+ /usr/lib/rpm/check-rpaths /usr/lib/rpm/check-buildroot
=========================

Comment 8 Rakesh Pandit 2009-01-29 08:39:08 UTC
New Package CVS Request
=======================
Package Name:  python-setupdocs
Short Description: Setuptools plugin
Owners: rakesh
Branches: F-9 F-10
InitialCC:
Cvsextras Commits: yes

Comment 9 Kevin Fenzi 2009-01-30 06:35:53 UTC
cvs done.

Comment 10 Fedora Update System 2009-03-09 22:55:42 UTC
python-setupdocs-1.0.1-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-03-09 23:04:02 UTC
python-setupdocs-1.0.1-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Orion Poplawski 2013-05-01 02:42:46 UTC
I would like to see this in EPEL6.  Rakesh - would you like to maintain it or shall I?

Comment 13 Orion Poplawski 2013-05-01 14:41:27 UTC
Package Change Request
======================
Package Name: python-setupdocs
New Branches: el6
Owners: orion rakesh
InitialCC: 

On 1 May 2013 05:44, Orion Poplawski wrote:
> I'd like to see setupdocs in EPEL6.  Commented here:
>
> https://bugzilla.redhat.com/show_bug.cgi?id=481725
>
> Would you like to maintain or shall I?
>

It would be great if you can maintain it. I would help later once I am
free with my thesis work.

Thank you.

--
Rakesh Pandit
https://fedoraproject.org/wiki/User:Rakesh
freedom, friends, features, first

Comment 14 Gwyn Ciesla 2013-05-01 14:47:34 UTC
Git done (by process-git-requests).