Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 727928 - >net-dns/bind-9.16 - QA Notice: Unrecognized configure options: --with-geoip
Summary: >net-dns/bind-9.16 - QA Notice: Unrecognized configure options: --with-geoip
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Christian Ruppert (idl0r)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-11 12:43 UTC by Jeroen Roovers (RETIRED)
Modified: 2020-06-29 17:48 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeroen Roovers (RETIRED) gentoo-dev 2020-06-11 12:43:17 UTC
~/portage/net-dns $
grep -r ENABLE.*geoip
bind-9.16.1/work/bind-9.16.1/configure.ac:AC_ARG_ENABLE([geoip],
bind-9.16.3/work/bind-9.16.3/configure.ac:AC_ARG_ENABLE([geoip],

So that should be $(use_enable geoip).
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2020-06-11 12:50:21 UTC
Also:

~/portage/net-dns $
grep -r WITH.*geoip
bind-9.16.1/work/bind-9.16.1/configure.ac:AC_ARG_WITH([geoip2],
bind-9.16.3/work/bind-9.16.3/configure.ac:AC_ARG_WITH([geoip2],

Relevant bits:

#
# GeoIP support?
#
# Should be on by default if libmaxminddb exists.
#
AC_ARG_WITH([geoip2],
        [AS_HELP_STRING([--with-geoip2],
                [deprecated, use --with-maxminddb])],
        [AC_MSG_WARN([--with-geoip2 is DEPRECATED and will be removed in a future release, use --with-maxminddb instead])],
        [with_geoip2="auto"])

AC_ARG_ENABLE([geoip],
          [AS_HELP_STRING([--disable-geoip],
                  [support GeoIP2 geolocation ACLs if available [default=yes]])],
          [], [enable_geoip="yes"])

AC_ARG_WITH([maxminddb],
        [AS_HELP_STRING([--with-maxminddb=PATH],
                [Build with MaxMind GeoIP2 support (auto|yes|no|path) [default=auto]])],
        [], [with_maxminddb="$with_geoip2"])


Concluding:

- dev-libs/geoip support was dropped
- $(use_with geoip) should be replaced with $(use_enable geoip)
- $(use_with geoip2) should be replaced with $(use_with maxminddb) but could be replaced with $(use_with geoip maxminddb) as that is effectively the same thing - maxminddb is the only current implementation it seems.
Comment 2 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2020-06-29 17:48:13 UTC
Thanks. Gonna commit the latest version.