pelias / placeholder

stand-alone coarse geocoder
https://placeholder.demo.geocode.earth
MIT License
313 stars 35 forks source link

Remove unnecessary compiler toolchain dependencies #211

Closed orangejulius closed 2 years ago

orangejulius commented 3 years ago

After better-sqlite3 added support for pre-compiled binaries, we no longer need to install a compiler toolchain to run npm install in our Docker images.

pelias/docker-baseimage#23 removes the compiler toolchain from our Pelias baseimages. In order for the toolchain to be removed from the Placeholder image in particular, we also need to remove those dependencies here.

Similar to the whosonfirst repository in https://github.com/pelias/whosonfirst/pull/532, this change by itself is effectively a no-op. After the baseimage removes the compiler toolchain, the size of the Placeholder docker image goes from 495MB to 266MB, an impressive 229MB savings!