[Tile-serving] [mod_tile] Drop internal copy of iniParser (#5)

[apmon]

This is a bit of a tricky issue. In general I agree that things shouldn't be statically included but depend on external libraries in the correct way. However, I want to make sure mod_tile / renderd is as easy as possible to compile and run and adding extra dependencies does not help in that respect. Particularly if that dependency is not in the package repository of one of the most important distributions (debian / ubuntu). So removing iniparser would make it significantly more complicated to install for the majority of users.

I don't think you can compare iniparser with postgresql or apache. Iniparser is a tiny fairly trivial piece of code that isn't really being maintained upstream anyway. In contrast to the huge project of postgres or apache. Indeed, in mod_tile we don't even use iniparser and simply wrote our own code with a bunch of if statements and a bit of string parsing. 

If there is a better, more maintained alternative that is available in all of the repositories of major distributions, I'd be happy to accept a patch to get rid of iniparser. However, other than that, I would very much prefer to keep iniparser as is. The advantages are simply not big enough for the effort. 

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/mod_tile/pull/5#issuecomment-34900420
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20140212/ec26195e/attachment.html>