Request for additional review: denyhosts

[Ignacio Vazquez-Abrams]

On Thu, 2005-07-21 at 11:59 -0500, Jason L Tibbitts III wrote:
> The denyhosts package has been in extras for a while now, but a
> recently released version changed from "put the executable in
> /usr/bin" to the standard Python setup system ("python ./setup.py").
> Since I'm not completely familiar with the latter, I figure I'd ask
> for input.
> 
> It's in CVS devel branch, and a spec and src.rpm are in
> http://www.math.uh.edu/~tibbs/rpms/denyhosts
> 
> The only thing I really have a question about is the fact that the
> Python setup process seems to install documentation in
> /usr/share/denyhosts, while the %doc directive places them in
> /usr/share/doc/denyhosts-0.8.0 and the result is two copies of the
> documentation.
> 
> What's the accepted way of handling this?

*boggles*

Who the heck puts setup.py in data_files? Anyways, the attached patch
should fix this.

-- 
Ignacio Vazquez-Abrams <ivazquez at ivazquez.net>
http://fedora.ivazquez.net/

gpg --keyserver hkp://subkeys.pgp.net --recv-key 38028b72
-------------- next part --------------
A non-text attachment was scrubbed...
Name: denyhosts-0.8.0-docs.patch
Type: text/x-patch
Size: 674 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/fedora-extras-list/attachments/20050721/0ea64897/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/fedora-extras-list/attachments/20050721/0ea64897/attachment.sig>