aMule Bug Tracker - aMule
View Issue Details
0001712aMuleMiscpublic2012-03-12 14:552012-03-22 22:01
0001712: Optional GeoIp update/bundle [^]

Mostly the optionality check should be inside CIP2Country::Enable

But there's a
which only wraps:

If there are no strong reason for that, then handling the database once fully downloaded should happens in src/IP2Country.cpp itself so disabling it when wanted can be straightforward.
No tags attached.
patch 1712-geoip-optional-update.patch (2,232) 2012-03-12 14:58
Issue History
2012-03-12 14:55gibborisNew Issue
2012-03-12 14:55gibborisOperating System => Any
2012-03-12 14:58gibborisFile Added: 1712-geoip-optional-update.patch
2012-03-12 14:58gibborisNote Added: 0003610
2012-03-20 22:39sturedmanNote Added: 0003615
2012-03-21 10:07gibborisNote Added: 0003619
2012-03-22 22:01sturedmanNote Added: 0003620

2012-03-12 14:58   
The attached patch may be a good starting point.
But I'm stuck on going further because of [0001086] which should, IMHO, be considered as CRITICAL !
2012-03-20 22:39   
Now why would you not want GeoIP to auto-update?

And we are not moving to a different Gui designer.
2012-03-21 10:07   
GeoIP is not this kind of thing for which users need strongly to be on the bleeding edge.
I'm fine with a 1y or even 3y old DB.

About [0001086], do you consider amule as FLOSS if we, users, can't even modify such simple elements of the GUI ?
2012-03-22 22:01   
You did not explain why the update would hurt you.

I do not like wxDesigner that much either (because you can't properly version, let alone merge .wdr), but at the current stage of the project a switch to a different GUI designer is not going to happen.
Anyway, this is a bug tracker, not a forum. The forum is where this kind (well, any kind) of discussion belongs.