* [ISSUE] libgeoip: remove dependency on geoip-data
@ 2019-11-07 23:50 voidlinux-github
2019-11-09 19:47 ` voidlinux-github
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: voidlinux-github @ 2019-11-07 23:50 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 765 bytes --]
New issue by user318 on void-packages repository
https://github.com/void-linux/void-packages/issues/16216
Description:
Hello,
Currently libgeoip depends on geoip-data. But as I understand, data is not strictly required for the lib - you can get your own data anywhere. So now, for example, when I try to install nginx on a small system, I get this:
```
geoip-data install - 20171002_2 25MB
libgeoip install - 1.6.12_1 56KB
nginx install - 1.16.1_1 347KB
```
As you can see geoip-data takes up a huge part of packages install even if I do not need it. I think it is a bad practice and geoip-data should be removed from the dependencies of libgeoip.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: libgeoip: remove dependency on geoip-data
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
@ 2019-11-09 19:47 ` voidlinux-github
2019-11-09 21:53 ` voidlinux-github
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-11-09 19:47 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 281 bytes --]
New comment by xtraeme on void-packages repository
https://github.com/void-linux/void-packages/issues/16216#issuecomment-552131193
Comment:
You can `ignore` the `geoip-data` pkg dependency easily with xbps:
```
# echo "ignorepkg=geoip-data" > /etc/xbps.d/99-custom.conf
```
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: libgeoip: remove dependency on geoip-data
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
2019-11-09 19:47 ` voidlinux-github
@ 2019-11-09 21:53 ` voidlinux-github
2019-11-09 21:59 ` voidlinux-github
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-11-09 21:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 234 bytes --]
New comment by user318 on void-packages repository
https://github.com/void-linux/void-packages/issues/16216#issuecomment-552141088
Comment:
@xtraeme Thank you for the tip, it is useful! But isn't this a redundant dependency anyway?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: libgeoip: remove dependency on geoip-data
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
2019-11-09 19:47 ` voidlinux-github
2019-11-09 21:53 ` voidlinux-github
@ 2019-11-09 21:59 ` voidlinux-github
2019-11-09 22:33 ` voidlinux-github
2020-04-23 16:50 ` [ISSUE] [CLOSED] " xtraeme
4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-11-09 21:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 202 bytes --]
New comment by xtraeme on void-packages repository
https://github.com/void-linux/void-packages/issues/16216#issuecomment-552141449
Comment:
I'm not sure, what would happen if geoip-data is not there?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: libgeoip: remove dependency on geoip-data
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
` (2 preceding siblings ...)
2019-11-09 21:59 ` voidlinux-github
@ 2019-11-09 22:33 ` voidlinux-github
2020-04-23 16:50 ` [ISSUE] [CLOSED] " xtraeme
4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-11-09 22:33 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 683 bytes --]
New comment by user318 on void-packages repository
https://github.com/void-linux/void-packages/issues/16216#issuecomment-552143864
Comment:
Lib wouldn't stop working. But you need to feed it with some data for it to do something meaningful, of course. It's like what would your image viewer do if there were no images around. geoip-data is a pack of Maxmind's lite databases, as I can see. One may not want to use all of those together, but only one of them. Alternatively one could use paid version of Maxmind's database or any other geoip database provider out there. So, yes, geoip-data may be of use in some cases, but there are also many cases when it is useless or overkill.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [ISSUE] [CLOSED] libgeoip: remove dependency on geoip-data
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
` (3 preceding siblings ...)
2019-11-09 22:33 ` voidlinux-github
@ 2020-04-23 16:50 ` xtraeme
4 siblings, 0 replies; 6+ messages in thread
From: xtraeme @ 2020-04-23 16:50 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 768 bytes --]
Closed issue by user318 on void-packages repository
https://github.com/void-linux/void-packages/issues/16216
Description:
Hello,
Currently libgeoip depends on geoip-data. But as I understand, data is not strictly required for the lib - you can get your own data anywhere. So now, for example, when I try to install nginx on a small system, I get this:
```
geoip-data install - 20171002_2 25MB
libgeoip install - 1.6.12_1 56KB
nginx install - 1.16.1_1 347KB
```
As you can see geoip-data takes up a huge part of packages install even if I do not need it. I think it is a bad practice and geoip-data should be removed from the dependencies of libgeoip.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-04-23 16:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-07 23:50 [ISSUE] libgeoip: remove dependency on geoip-data voidlinux-github
2019-11-09 19:47 ` voidlinux-github
2019-11-09 21:53 ` voidlinux-github
2019-11-09 21:59 ` voidlinux-github
2019-11-09 22:33 ` voidlinux-github
2020-04-23 16:50 ` [ISSUE] [CLOSED] " xtraeme
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).