Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
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.