From 6e67a3b4927f71e4caf9b22cb67ace26b506ff8a Mon Sep 17 00:00:00 2001
From: Nick Boldt <nboldt@redhat.com>
Date: Wed, 18 Oct 2023 17:38:57 -0300
Subject: [PATCH] comment out node10 (updated to node18) and maven 3.6 (updated
 to 3.8) installs, as we likely don 't need them (either not required or
 handled by GH action) #22616

Change-Id: I5d0920b58e413247fd5466a269fb26b1af60578c
Signed-off-by: Nick Boldt <nboldt@redhat.com>
---
 .github/workflows/release.yml |  2 +-
 make-release.sh               | 38 ++++++++++++++++++++---------------
 2 files changed, 23 insertions(+), 17 deletions(-)

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index a1f7daaf63..68ccfc2488 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2020 Red Hat, Inc.
+# Copyright (c) 2020-2023 Red Hat, Inc.
 # This program and the accompanying materials are made
 # available under the terms of the Eclipse Public License 2.0
 # which is available at https://www.eclipse.org/legal/epl-2.0/
diff --git a/make-release.sh b/make-release.sh
index 160012d25c..1ee3905bda 100755
--- a/make-release.sh
+++ b/make-release.sh
@@ -50,21 +50,22 @@ loadMvnSettingsGpgKey() {
     gpg --version
 }
 
-installDebDeps(){
-    set +x
-    # TODO should this be node 12?
-    curl -sL https://deb.nodesource.com/setup_10.x | sudo -E bash -
-    sudo apt-get install -y nodejs
-}
+# comment out: why do we think we need nodejs installed for a java build?
+# installDebDeps(){
+#     set +x
+#     curl -sL https://deb.nodesource.com/setup_18.x | sudo -E bash -
+#     sudo apt-get install -y nodejs
+# }
 
-installMaven(){
-    set -x
-    mkdir -p /opt/apache-maven && curl -sSL https://downloads.apache.org/maven/maven-3/3.6.3/binaries/apache-maven-3.6.3-bin.tar.gz | tar -xz --strip=1 -C /opt/apache-maven
-    export M2_HOME="/opt/apache-maven"
-    export PATH="/opt/apache-maven/bin:${PATH}"
-    mvn -version || die_with "mvn not found in path: ${PATH} !"
-    set +x
-}
+# comment out: already installed via GH action so no need to do it again
+# installMaven(){
+#     set -x
+#     mkdir -p /opt/apache-maven && curl -sSL https://downloads.apache.org/maven/maven-3/3.8.8/binaries/apache-maven-3.8.8-bin.tar.gz | tar -xz --strip=1 -C /opt/apache-maven
+#     export M2_HOME="/opt/apache-maven"
+#     export PATH="/opt/apache-maven/bin:${PATH}"
+#     mvn -version || die_with "mvn not found in path: ${PATH} !"
+#     set +x
+# }
 
 evaluateCheVariables() {
     echo "Che version: ${CHE_VERSION}"
@@ -414,9 +415,14 @@ updateImageTagsInCheServer() {
     popd >/dev/null
 }
 
-installMaven
+# already installed via GH action so no need to do it again
+# installMaven
+
 loadMvnSettingsGpgKey
-installDebDeps
+
+# why do we think we need nodejs installed for a java build?
+# installDebDeps 
+
 set -x
 setupGitconfig