[Bug 181974] Review Request: mod_geoip - Apache module for GeoIP lookups

[bugzilla 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: mod_geoip - Apache module for GeoIP lookups


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





------- Additional Comments From mfleming+rpm enlartenment com  2006-02-20 03:53 EST -------
I'll answer both questions in one reply -  Both are quite valid questions.

Jason: The httpd-mmn Requires: is to ensure that the module version matches that
of the Apache version you're going to install against. (Something one of the
more experienced packagers (Ville?) pointed out while I was working on mod_security)

Ralf: I called it mod_geoip to avoid potential confusion ("Where's/What is
mod_geoip1?) and ensure some level of clarity.

Upstream has different tarballs for Apache 1.3.x and 2.x, the latter is
mod_geoip2 but, given that FC doesn't package the older branch anyway, I went
for plain "mod_geoip" so even the most inebriated of users could guess what it
was for :-D.

As for a virtual property, I've never seen anyone package it elsewhere  - and if
they've called it mod_geoip they'll get cleanly upgraded anyway provided other
deps are met.

I'll import and build unless anyone has strong "blocker" objections to the name.


-- 
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.