search

y-ken / fluent-plugin-geoip

Fluentd output plugin to geolocate with geoip.
http://rubygems.org/gems/fluent-plugin-geoip
Other
103 stars 23 forks source link

Update README.md #73

Closed Guik closed 6 years ago

Guik commented 6 years ago

Add Maxmind dependancy for Debian to avoid compilation error "fatal error: maxminddb_config.h: No such file or directory"

okkez commented 6 years ago

Hmm, it's werid. fluent-plugin-geoip depends geoip2_c gem, and it bundles libmaxminddb and we can build libmaxminddb while installing geoip2_c gem.

I can install fluent-plugin-geoip without installing libmaxminddb-dev.

andygeorge commented 6 years ago

FWIW, geoip2_c doesn't install in Fedora, either, without explicitly installing libmaxminddb-devel first.

okkez commented 6 years ago

@andygeorge On Fedora28, I can install fluent-plugin-geoip without installing libmaxminddb-devel.

Could you check logs after failed to install fluent-plguin-geoip. Maybe, you can run gem install geoip2_c -N for debugging.

FYI, I've installed following packages:

$ sudo dnf group install "Development Tools"
$ sudo dnf group install "C Development Tools and Libraries"

We need autoconf, automake, libtool, and etc. to build bundled libmaxminddb.

andygeorge commented 6 years ago

We need autoconf, automake, libtool, and etc. to build bundled libmaxminddb.

Correct, which is not included in the readme, and is probably not ideal for a production deploy.

Because libmaxminddb-devel already exists as a Fedora package, it should be used instead of bundling and building it on the fly (at least for Fedora).

okkez commented 6 years ago

Because libmaxminddb-devel already exists as a Fedora package, it should be used instead of bundling and building it on the fly (at least for Fedora).

Right. But this is not the scope of this plugin. And you can install libmaxminddb-devel at your own risk.

I've added about libmaxminddb-dev/libmaxminddb-devel to geoip2_c's README.md.