broken docker image maptiler/tileserver-gl-light:v4.6.3 about tileserver-gl HOT 7 CLOSED

The Dockerfile is missing apt-get install -y nodejs;.

Also the npm ci instruction is ran inside a series of commands separated by ";" instead of "&", which probably explains why it failed but did not break the image build.

from tileserver-gl.

I didn't realize i broke this when i was troubleshooting why the npm ci was failing on arm.

If you want to submit a PR to fix this it will help me approve it easier.

from tileserver-gl.

Sure, I will prepare a PR.

from tileserver-gl.

Is it on purpose that Dockerfile_light references docker-entrypoint.sh and not docker-entrypoint_light.sh ?

from tileserver-gl.

Nevermind, I just fixed is manually, so a pr should not be needed unless you think more changes should be made.

And yes, the name is on purpose. when the publish.js script copies files it renames this file
https://github.com/maptiler/tileserver-gl/blob/master/publish.js#L51

from tileserver-gl.

Ok, thanks !

from tileserver-gl.

v4.6.5 light version should be fixed

from tileserver-gl.