Skip to content

DEV: pin external dependencies and update#1073

Merged
SamSaffron merged 1 commit into
mainfrom
supply
Jun 11, 2026
Merged

DEV: pin external dependencies and update#1073
SamSaffron merged 1 commit into
mainfrom
supply

Conversation

@SamSaffron

Copy link
Copy Markdown
Member

Vendor NodeSource and Yarn signing keys instead of fetching them during builds, and install pups from a verified local gem.

Pin git-sourced build inputs to immutable commits and refresh image dependencies including ImageMagick, jemalloc, oxipng, and Redis. Use HTTPS for Redis downloads and avoid installing the extra jemalloc 5 library on non-arm64 builds.

Vendor NodeSource and Yarn signing keys instead of fetching them during builds, and install pups from a verified local gem.

Pin git-sourced build inputs to immutable commits and refresh image dependencies including ImageMagick, jemalloc, oxipng, and Redis. Use HTTPS for Redis downloads and avoid installing the extra jemalloc 5 library on non-arm64 builds.

@fitzy101 fitzy101 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hardening these installs LGTM

echo "2db82d1e7119df3e71b7640219b6dfe84789bc0537983c3b7ac4f7189aecfeaa jemalloc-5.3.0.tar.bz2" | sha256sum -c
tar --strip-components=1 -xjf jemalloc-5.3.0.tar.bz2
wget -q https://github.com/jemalloc/jemalloc/releases/download/5.3.1/jemalloc-5.3.1.tar.bz2
sha256sum jemalloc-5.3.1.tar.bz2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this line leftover from someone wanting to read the checksum? (I realise this was here already, but it's probably not needed?)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yeah I think we can drop it... we dont need 5.3.1

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

oh it is needed for ARM

@SamSaffron SamSaffron merged commit 1f3580b into main Jun 11, 2026
7 checks passed
@SamSaffron SamSaffron deleted the supply branch June 11, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants