Skip to content

Commit a9d67d6

Browse files
committed
Merge branch 'jc/ci-skip-unavailable-external-software'
Further refinement on CI messages when an optional external software is unavailable (e.g. due to third-party service outage). * jc/ci-skip-unavailable-external-software: ci: download JGit from maven, not eclipse.org ci: update the message for unavailble third-party software
2 parents bd99d6e + 956acbe commit a9d67d6

File tree

1 file changed

+8
-13
lines changed

1 file changed

+8
-13
lines changed

ci/install-dependencies.sh

+8-13
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ begin_group "Install dependencies"
99

1010
P4WHENCE=https://cdist2.perforce.com/perforce/r23.2
1111
LFSWHENCE=https://github.com/github/git-lfs/releases/download/v$LINUX_GIT_LFS_VERSION
12-
JGITWHENCE=https://repo.eclipse.org/content/groups/releases//org/eclipse/jgit/org.eclipse.jgit.pgm/6.8.0.202311291450-r/org.eclipse.jgit.pgm-6.8.0.202311291450-r.sh
12+
JGITWHENCE=https://repo1.maven.org/maven2/org/eclipse/jgit/org.eclipse.jgit.pgm/6.8.0.202311291450-r/org.eclipse.jgit.pgm-6.8.0.202311291450-r.sh
1313

1414
# Make sudo a no-op and execute the command directly when running as root.
1515
# While using sudo would be fine on most platforms when we are root already,
@@ -71,24 +71,19 @@ ubuntu-*|i386/ubuntu-*|debian-*)
7171
chmod a+x "$CUSTOM_PATH/p4d" "$CUSTOM_PATH/p4" || {
7272
rm -f "$CUSTOM_PATH/p4"
7373
rm -f "$CUSTOM_PATH/p4d"
74-
echo >&2 "P4 download (optional) failed"
7574
}
7675

7776
wget --quiet \
7877
"$LFSWHENCE/git-lfs-linux-amd64-$LINUX_GIT_LFS_VERSION.tar.gz" &&
7978
tar -xzf "git-lfs-linux-amd64-$LINUX_GIT_LFS_VERSION.tar.gz" \
8079
-C "$CUSTOM_PATH" --strip-components=1 \
8180
"git-lfs-$LINUX_GIT_LFS_VERSION/git-lfs" &&
82-
rm "git-lfs-linux-amd64-$LINUX_GIT_LFS_VERSION.tar.gz" || {
83-
rm -f "$CUSTOM_PATH/git-lfs"
84-
echo >&2 "LFS download (optional) failed"
85-
}
81+
rm "git-lfs-linux-amd64-$LINUX_GIT_LFS_VERSION.tar.gz" ||
82+
rm -f "$CUSTOM_PATH/git-lfs"
8683

8784
wget --quiet "$JGITWHENCE" --output-document="$CUSTOM_PATH/jgit" &&
88-
chmod a+x "$CUSTOM_PATH/jgit" || {
89-
rm -f "$CUSTOM_PATH/jgit"
90-
echo >&2 "JGit download (optional) failed"
91-
}
85+
chmod a+x "$CUSTOM_PATH/jgit" ||
86+
rm -f "$CUSTOM_PATH/jgit"
9287
;;
9388
esac
9489
;;
@@ -151,23 +146,23 @@ then
151146
echo "$(tput setaf 6)Perforce Client Version$(tput sgr0)"
152147
p4 -V
153148
else
154-
echo >&2 "WARNING: perforce wasn't installed, see above for clues why"
149+
echo >&2 "::warning:: perforce wasn't installed, see above for clues why"
155150
fi
156151

157152
if type git-lfs >/dev/null 2>&1
158153
then
159154
echo "$(tput setaf 6)Git-LFS Version$(tput sgr0)"
160155
git-lfs version
161156
else
162-
echo >&2 "WARNING: git-lfs wasn't installed, see above for clues why"
157+
echo >&2 "::warning:: git-lfs wasn't installed, see above for clues why"
163158
fi
164159

165160
if type jgit >/dev/null 2>&1
166161
then
167162
echo "$(tput setaf 6)JGit Version$(tput sgr0)"
168163
jgit version
169164
else
170-
echo >&2 "WARNING: JGit wasn't installed, see above for clues why"
165+
echo >&2 "::warning:: JGit wasn't installed, see above for clues why"
171166
fi
172167

173168
end_group "Install dependencies"

0 commit comments

Comments
 (0)