[openstreetmap/openstreetmap-website] Docker: clear out apt repo after installing packages (PR #6464)

[Pablo Brasero]

At https://github.com/openstreetmap/openstreetmap-website/pull/5157, this `apt-get clean` was removed. The man page describes this command as follows:

> clean clears out the local repository of retrieved package files. It removes everything but the lock file from /var/cache/apt/archives/ and /var/cache/apt/archives/partial/.

Normally I wouldn't have thought much about it, but what tickled me was that the comment at the top still says "then clean up to minimize image size". So this left me wondering whether I should update the comment or re-introduce the command. I couldn't see in the PR conversation anything about why it was removed, so I thought I'd start by proposing the re-introduction.

I didn't go as far as adding the `rm -rf /var/lib/apt/lists/*` line as I don't feel comfortable messing up with APT internals.

Thoughts? Perhaps @kcne remembers?
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/6464

-- Commit Summary --

  * Clear out apt repo after installing packages

-- File Changes --

    M Dockerfile (49)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/6464.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6464.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6464
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6464 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251022/35d716fb/attachment.htm>