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

Reply via email to