[Tile-serving] [osm2pgsql] Set polygon/linestring differently for some key values differently than the key in default style.lua (#346)

[math1985]

This is now ready to be merged. I can still squash the commits if you like.

> A general remark: You do say that the PR is intended as an example implementation but at the same time I see a tagging discussion unfold here with plenty of references to the carto style. That tells me this might be the wrong repository for that change.

I understand your point, but I'm not sure if I agree. I think it would be good if osm2pgsql provides sane defaults. The decision whether to treat a certain tag, like natural=tree_row, as polygon or line is a data representation decision, not a rendering decision. I would therefore expect this to be solved in osm2pgsql. It doesn't matter whether I want to use PostGIS-database for some kind of rendering or something else, I'd consider the current default, where a closed natural=tree_row is treated as polygon, as an non-sane default.

However, if the osm2pgsql maintainers say this repository is not the right place for this PR, I would understand. Maintaining an openstreetmap-carto specific .lua-file would work for us too.

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/346#issuecomment-187463352
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20160222/312fab73/attachment-0001.html>