Bug 213906 - Review Request: Pound - a reverse proxy and load balancer
Summary: Review Request: Pound - a reverse proxy and load balancer
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-11-03 18:14 UTC by Ruben Kerkhof
Modified: 2014-04-07 12:17 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-11-14 18:44:39 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Ruben Kerkhof 2006-11-03 18:14:44 UTC
Spec URL: http://rubenkerkhof.com/packages/pound/Pound.spec
SRPM URL: http://rubenkerkhof.com/packages/pound/Pound-2.1.5-1.src.rpm
Description:
The Pound program is a reverse proxy, load balancer and
HTTPS front-end for Web server(s). Pound was developed
to enable distributing the load among several Web-servers
and to allow for a convenient SSL wrapper for those Web
servers that do not offer it natively. Pound is distributed
under the GPL - no warranty, it's free to use, copy and
give away.

Comment 1 Ruben Kerkhof 2006-11-03 18:18:54 UTC
This is my first package, and I'm looking for a sponsor.

Comment 2 Aurelien Bompard 2006-11-04 17:01:56 UTC
Needswork :
 - Do not repeat the name in the Summary
 - Please create an init script for pound (you can base it on
/etc/init.d/transparent-proxying in the squidGuard package for example)
 - Please make a default config file for /etc/pound.cfg, with a very basic setup
(like the second example in the pound(8) man page), but with the control socket
and running as a different user
  -> This implies that you create a directory for Pound's control sockets, for
example /var/lib/pound, and that you own it in the rpm
  -> It also implies that you create a pound user in %pre, see the Zope package
for an example)

Possible improvements:
 - Generate a dummy SSL cert in /etc/pki/tls/certs/pound.pem, you can look at
the %post script in the openldap-servers package for a way to do it.
 - Add an SELinux policy, but that's a bit more complicated

Comment 3 Ruben Kerkhof 2006-11-07 21:50:37 UTC
Thanks for the review Aurelien.

I've added everything you asked, except for the SELinux policy.
Upstream changed to 2.1.6 in the meantime, so I did as well.

http://rubenkerkhof.com/packages/pound/Pound.spec
http://rubenkerkhof.com/packages/pound/Pound-2.1.6-1.src.rpm

Regards,

Ruben

Comment 4 Aurelien Bompard 2006-11-08 12:26:56 UTC
Review for release 1:
* RPM name is OK
* Source Pound-2.1.6.tgz is the same as upstream
* This is the latest version
* Builds fine in mock
* File list looks OK
* Config files of Pound looks OK
* Works fine

Minor:
* rpmlint says "macro-in-%changelog". Please replace %post with %%post to avoid
warning.
* Please replace /usr/sbin with %{_sbindir} in Requires(pre).

APPROVED

You can now go on with the sponsorship procedure :
http://fedoraproject.org/wiki/Extras/Contributors#GetAFedoraAccount

Comment 5 Ruben Kerkhof 2006-11-08 23:13:30 UTC
Ok, I'll do that.

Created a new package with the above errors fixed.
http://rubenkerkhof.com/packages/pound/Pound-2.1.6-1.src.rpm

Comment 6 Ruben Kerkhof 2006-11-08 23:17:02 UTC
Sorry, wrong link.
It's at http://rubenkerkhof.com/packages/pound/Pound-2.1.6-2.src.rpm
and the updated spec:
http://rubenkerkhof.com/packages/pound/Pound.spec

Comment 7 Lubomir Rintel 2014-04-07 11:55:05 UTC
Package Change Request
======================
Package Name: Pound
New Branches: epel7
Owners: lkundrak wolfy

The EL-6 branch maintainer (wolfy) did not respond to private mails, IRC, a public non-responsive maintainer call via fedora-devel list, nor does his sponsor know a way to contact him) :( Adding him a as a co-maint so that he can step in if he's active again.

Comment 8 Gwyn Ciesla 2014-04-07 12:17:49 UTC
Git done (by process-git-requests).


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