Request for Review - PyRTF

[Michael A. Peters]

On Wed, 2005-04-13 at 14:15 -0400, Ignacio Vazquez-Abrams wrote:
> On Wed, 2005-04-13 at 10:35 -0700, Michael Peters wrote:
> > On Wed, 2005-04-13 at 05:33 -0400, Ignacio Vazquez-Abrams wrote:
> > > On Wed, 2005-04-13 at 02:14 -0700, Michael Peters wrote:
> > > > My spec file - http://mpeters.us/fc_extras/PyRTF.spec
> > > 
> > > ? Source0 should have a complete URL if possible (e.g.,
> > > http://prdownloads.sourceforge.net/pyrtf/PyRTF-0.43.tar.gz)
> > > - BR should be python, not python-devel
> > > - Dir %{python_sitelib}/PyRTF not owned by package
> > > - Should have [E:]V-R on every changelog entry
> > 
> > Thanks - fixed
> 
> You missed the BR. noarch Python packages only need distutils, not the
> whole includes/libs set. And distutils comes in the base python package.

My apologies - my last reply was with respect to a different package.

Current PyRTF spec file and package:

http://mpeters.us/fc_extras/PyRTF.spec
http://mpeters.us/fc_extras/PyRTF-0.43-1.src.rpm