[Bug 233522] Review Request: nginx - http and reverse proxy server

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: nginx -   http and reverse proxy server


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233522





------- Additional Comments From jeremy at hinegardner.org  2007-03-23 13:16 EST -------
(In reply to comment #7)
> Hi Jeremy,
> 
> - zlib-devel is already required by openssl-devel
> - You can even drop the whole Requires line. rpm is smart enough to calculate
that by itself.

Unless someone objects, I'll leave these as they are.

> - Replace /usr/sbin/nginx in your %install section with %{_sbindir}/nginx
> - You're missing a forward slash in
--lock-path=%{_localstatedir}lock/subsys/%{name}

Both of these are now fixed

> Nothing that should block this review, but I hope it helps.

Thanks for the feedback, it always helps.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.