[Tile-serving] [openstreetmap/osm2pgsql] Need for a "minimum_version" option for osm2pgsql in Lua flex styles? (Discussion #1638)
mboeringa
notifications at github.com
Fri Feb 4 17:24:33 UTC 2022
With the continued work on extending osm2pgsql's flex processing, e.g. the addition of more advanced geometry processing:
https://github.com/openstreetmap/osm2pgsql/pull/1636
this raises a question for me if there shouldn't be some way to specify in an osm2pgsql Lua flex style fila a "minimum version" of osm2pgsql required to run this Lua file?
With different versions of osm2pgsql out there that now have quite distinct capabilities, it becomes increasingly difficult to ascertain if a specific Lua style file is compatible with the osm2pgsql version present on a system.
I am not thinking of something really complex here requiring rocket science. Just a properly documented variable, e.g.
**local minimum_version_osm2pgsql=X**
that could be recognized by osm2pgsql and allow the processing to halt immediately with a user friendly error message, e.g.:
**"osm2pgsql has detected that the selected Lua flex style 'X' requires a minimum version of osm2pgsql 'Y' to run, while your system has version 'Z'."**
Of course, the onus would be fully on the developer of the style to set the appropriate version, it is not the responsibility of osm2pgsql to fail gracefully if it is set wrong, but it could fail gracefully if it is set appropriately. The style developer also most likely tested his/her work, so he/she should really have no trouble setting the value.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/discussions/1638
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/osm2pgsql/repo-discussions/1638 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20220204/db27ce43/attachment-0001.htm>
More information about the Tile-serving
mailing list