Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 156785 - app-admin/webalizer with USE="xtended" excludes GeoIP support
Summary: app-admin/webalizer with USE="xtended" excludes GeoIP support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Jonathan Smith (RETIRED)
URL:
Whiteboard:
Keywords:
: 162757 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-12-01 04:07 UTC by Hans Rakers
Modified: 2008-02-23 16:53 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
hacked ebuild (webalizer-2.01.10-r14.ebuild,4.21 KB, text/plain)
2007-01-19 10:21 UTC, Hans Rakers
Details
Updated webalizer-2.01.10-r15.ebuild to allow +xtended +geoip (webalizer-2.01.10-r15.ebuild,4.14 KB, text/plain)
2007-07-09 12:13 UTC, Hannes Wyss
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Hans Rakers 2006-12-01 04:07:57 UTC
Webalizer-xtended includes the geolizer patch (GeoIP support), but gentoo's ebuild for webalizer also includes support for just the geolizer patch through the use of the use flag 'geoip'

Since webalizer can only be compiled with either the use flag 'geoip' or 'xtended', it is impossible to use xtended with geoip support.

This could be solved by adding a extra useflag 'xtended-geoip' or something.


For the record, the xtended patch introduces the following extra configure options:

--with-geoip-inc=DIR      Alternate location for GeoIP headers
--with-geoip-lib=DIR      Alternate location for GeoIP library
--enable-geoip            Enable GeoIP country lookup
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2006-12-01 04:19:04 UTC
No, it couldn't; those two patches don't work together, as noted in the ebuild a couple of times.
Comment 2 Hans Rakers 2006-12-01 04:30:38 UTC
Wow, this answer was so predictable... Please could you actually READ bug reports?!

Yes, I know geoip AND xtended *patches* dont work together, but the xtended patch already includes geolizer/geoip support! (as i tried to point out by adding the configure options that xtended introduces at the end of my initial report)

Problem is that there is no way to enable it atm, because the current ebuild doesnt provide a way to compile xtended with --enable-geoip. I have altered the ebuild myself and put it in an overlay and ONLY enabled xtended, but added a slight hack so it would compile with --enable geoip by adding

use xtended && myconf="${myconf} --enable-geoip"

to the src_compile section. This works like a charm, but it would be nice if this was possible in the regular ebuild.

Comment 3 Jakub Moc (RETIRED) gentoo-dev 2007-01-19 07:54:00 UTC
*** Bug 162757 has been marked as a duplicate of this bug. ***
Comment 4 Saurabh Singhvi 2007-01-19 07:58:42 UTC
Hans, 

could you attach the ebuild here ?? 

thanks
Saurabh
Comment 5 Hans Rakers 2007-01-19 10:21:23 UTC
Created attachment 107422 [details]
hacked ebuild

Here it is, it's a bit of a hack though...
Comment 6 Hannes Wyss 2007-07-09 12:13:51 UTC
Created attachment 124323 [details]
Updated webalizer-2.01.10-r15.ebuild to allow +xtended +geoip

Hi! 
I just ran afoul of the same issue. The attached ebuild works for me (TM).
Please consider using it as -r16

Thanks in advance
Hannes
Comment 7 Benedikt Böhm (RETIRED) gentoo-dev 2008-02-23 16:53:53 UTC
fixed in 2.01.10-r16