This is an automated email from the ASF dual-hosted git repository.
slawrence pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/daffodil.git
The following commit(s) were added to refs/heads/main by this push:
new 6cca2600b Upgrade nodejs in the release candidate container
6cca2600b is described below
commit 6cca2600b9732efb4b38b9b60adb4ef4e2d6bbe1
Author: Steve Lawrence <[email protected]>
AuthorDate: Thu Jun 6 09:08:47 2024 -0400
Upgrade nodejs in the release candidate container
The version of nodejs that ships with Fedora 40 has a bug that causes a
SIGILL error when runing npm on some architectures. This installs
nodejs/npm from the Fedora "updates" repo which includes a fix.
---
containers/release-candidate/Dockerfile | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/containers/release-candidate/Dockerfile
b/containers/release-candidate/Dockerfile
index dc54cb7ae..fa218bfff 100644
--- a/containers/release-candidate/Dockerfile
+++ b/containers/release-candidate/Dockerfile
@@ -24,6 +24,11 @@ RUN rm -rf /root/*
# Install dependencies, explicitly not enabling the updates repo so we are
# pinned at a particular fedora release.
+#
+# Note that the default nodejs package shipped with fedora 40 has a bug that
+# can sometimes lead to SIGILL errors on some architectures. So we also upgrade
+# to the latest nodejs version to fix this. However we only upgrade the one
+# package so we are still mostly pinned to the Fedora 40 release.
RUN \
dnf -y --quiet --repo=fedora install \
clang \
@@ -41,6 +46,8 @@ RUN \
vim-minimal \
wine \
winetricks && \
+ dnf -y --quiet --repo=updates upgrade-minimal \
+ nodejs && \
dnf clean all
# Enable sbt-pgp plugin