search

tum-gis / cesium-terrain-builder-docker

Dockerfile for the geo-data/cesium-terrain-builder app with quantized mesh support.
Apache License 2.0
204 stars 51 forks source link

Update README.md #17

Closed ruanqizhen closed 1 year ago

ruanqizhen commented 1 year ago

Fix the Windows command in README file. The old command got "invalid reference format" error on Windows.

BWibo commented 1 year ago

Thx for the contribution. I just tested this and the original command works fine on my Win 10 + Docker Desktop:

grafik

What Windows and Docker are you using? Can you please test this again.

ruanqizhen commented 1 year ago

Thx for the contribution. I just tested this and the original command works fine on my Win 10 + Docker Desktop:

grafik

What Windows and Docker are you using? Can you please test this again.

I'm using Powershell, I guess that is the problem. I can only run it like: docker run -i -t --name ctb -v c:\projects\terrain:/data tumgis/ctb-quantized-mesh

BWibo commented 1 year ago

I did some testing and found that probably the double quotation was the problem: -v "/some/path":"/data" -> -v "/some/path:/data" resolved it.

Thank's for your input and testing. I added a separate example for powershell in https://github.com/tum-gis/cesium-terrain-builder-docker/commit/e2481a09de94ab35d9c1918f9e12d5af54334295.