Update to a new Mapnik and Kosmtik versions

[kocio-pl]

Expected behavior

After upgrade to newest Kosmtik version from git master (see kosmtik/kosmtik#295) rendering should not change.

Actual behavior

There are some warnings and the rendering is broken.

Links and screenshots illustrating the problem

Example warnings:

Warning: landcover.mss:608:6 Styles do not match layer selector #landcover-low-zoom.
Warning: landcover.mss:608:6 Styles do not match layer selector #landcover.

Luxembourg - after:

[kocio-pl]

This change is needed to finally merge long awaited #3399.

[pnorman]

If we require a new mapnik version we should be doing that in a major release

[matthijsmelissen]

After upgrade to newest Mapnik version from git master

It's either master or the latest (version) tag, it cannot be both :)

Does OSMF Operations already run the latest version? Logistically it would probably be easier if they bump their version first, before we release a version requiring the new Mapnik.

[sommerluk]

On my local machine, it renders correctly. But I have deleted all .index files for the shapefiles. I suspect these files make problems because there is a version missmatch between the Mapnik version that generates the index files (either the downloaded index files or these you might re-create locally using your distribution's Mapnik package) and the node-mapnik version used by Kosmtik. If you delete the index files, does it render correctly?

[kocio-pl]

I have upgraded my system to Ubuntu 19.04 and the whole environment is not working, currently I have:

$ node -v
v10.15.2

$ npm -v
5.8.0

Maybe I will try to downgrade them to a node v8.10.0 and npm 3.5.2, as before.

[sommerluk]

Does OSMF Operations already run the latest version?

Yes, operation is yet running Carto 1.2.0.

[sommerluk]

There are some warnings

This is a consequence of a change in CartoCSS which outputs now much more warnings than before, but still generates the very same XML result. Its consequences for openstreetmap-carto have been discussed in #3183 and it has been considered a non-issue.

[sommerluk]

Closing as we use higher Mapnik and Kosmtik versions now.