Skip to content

Conversation

@nblock
Copy link
Collaborator

@nblock nblock commented Oct 27, 2025

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

@nblock nblock requested a review from kradalby October 27, 2025 19:30
@nblock nblock marked this pull request as ready for review October 27, 2025 19:30
@nblock nblock merged commit 0a43aab into juanfont:main Oct 28, 2025
94 checks passed
@nblock nblock deleted the up branch October 28, 2025 04:55
Comment on lines 11 to 12
&& rm -rf /var/lib/apt/lists/* \
&& apt-get clean
Copy link
Contributor

@tianon tianon Nov 12, 2025

Choose a reason for hiding this comment

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

FWIW, apt-get clean is a no-op in the Debian (and Ubuntu) base images

Perhaps confusingly in relation to that, APT in Trixie now supports a separate apt-get dist-clean command that's similar to this rm -rf, but better (because it keeps the cryptographic downgrade-attack-preventing verification bits).

Suggested change
&& rm -rf /var/lib/apt/lists/* \
&& apt-get clean
&& apt-get dist-clean

Edit: sent a PR (#2880); hopefully it's a trivial enough change to not require much discussion, but I'm happy to discuss further! ❤️

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.

3 participants