nodejs/hookshot
nodejs/hookshot
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-hookshot.git synced 2025-03-10 13:17:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Pin the Node minor version for Docker builds (#949)

Browse Source 
* Pin the Node minor version for Docker builds

as 20.13 & 20.14 cause Hookshot to crash on startup (at least on arm64)

* Add changelog
This commit is contained in:
Andrew Ferrazzutti 2024-06-22 00:56:06 +09:00 committed by GitHub
parent 5dae23b2ad
commit 5c5e8ce767
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Dockerfile
View File

@ -1,7 +1,7 @@
# Stage 0: Build the thing # Stage 0: Build the thing
# Need debian based image to build the native rust module # Need debian based image to build the native rust module
# as musl doesn't support cdylib # as musl doesn't support cdylib
FROM node:20-slim AS builder FROM node:20.12-slim AS builder
# Needed in order to build rust FFI bindings. # Needed in order to build rust FFI bindings.
RUN apt-get update && apt-get install -y build-essential cmake curl pkg-config pkg-config libssl-dev RUN apt-get update && apt-get install -y build-essential cmake curl pkg-config pkg-config libssl-dev
@ -29,7 +29,7 @@ RUN yarn build
# Stage 1: The actual container # Stage 1: The actual container
FROM node:20-slim FROM node:20.12-slim
WORKDIR /bin/matrix-hookshot WORKDIR /bin/matrix-hookshot

1
changelog.d/949.misc Normal file
View File

@ -0,0 +1 @@
Pin the minor version of Node for Docker builds to avoid a startup crash on arm64.