Skip to content

Update leaflet to 1.9.4 and carto to 5.8.0#403

Open
centic9 wants to merge 2 commits into
Overv:masterfrom
centic9:update_leaflet_and_carto
Open

Update leaflet to 1.9.4 and carto to 5.8.0#403
centic9 wants to merge 2 commits into
Overv:masterfrom
centic9:update_leaflet_and_carto

Conversation

@centic9
Copy link
Copy Markdown

@centic9 centic9 commented Dec 10, 2023

This PR updates Leaflet to 1.9.4 and openstreetmap-carto to 5.8.0 and adjusts steps in the Dockerfile as necessary.

This then allows to inject file:// URLs to work around download issues discussed in #244

This includes openstreetmap-carto/openstreetmap-carto#4468 and thus allows to work around issue Overv#244 via local file downloads
files are now in a sub-dir "dist" in the download-package
Comment thread Dockerfile
&& git clone --single-branch --branch v5.4.0 https://github.com/gravitystorm/openstreetmap-carto.git --depth 1 \
&& git clone --single-branch --branch v5.8.0 https://github.com/gravitystorm/openstreetmap-carto.git --depth 1 \
&& cd openstreetmap-carto \
&& sed -i 's/, "unifont Medium", "Unifont Upper Medium"//g' style/fonts.mss \
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All these font-replacements are not necessary any more now

Comment thread Dockerfile
&& wget https://github.com/Leaflet/Leaflet/releases/download/v1.8.0/leaflet.zip \
&& wget https://github.com/Leaflet/Leaflet/releases/download/v1.9.4/leaflet.zip \
&& unzip leaflet.zip \
&& mv dist/* . \
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Zip-Package now contains a sub-dir "dist", so we need to move the content back to the main directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant