Skip to content

Commit 0efa9e7

Browse files
committed
ci(jgit): use a more reliable link to download JGit
Git's dependency story is improvable: When, say, JGit cannot be downloaded, it currently fails the entire job without even running all the tests that do not require JGit (which is the vast, vast majority of them). It would be better to handle that gracefully by warning about the JGit download failure and then skipping the respective tests. This JGit download problem is far from theoretical: There is an incident on eclipse.org that has now lasted for an extended amount of time: https://www.eclipsestatus.io/incident/549796. While upstream Git finally chose to improve their CI story in b0026da (ci: skip unavailable external software, 2025-04-24), we should heed the suggestion by the JGit maintainer, Matthias Sohn, in https://discord.com/channels/1042895022950994071/1364872237710184520/1364885895865696479, to switch to the far more reliable Maven Central link (according to https://status.maven.org/uptime, there has not been any major downtime recently that wasn't resolved within 5 minutes, a far cry from the 30h of the current eclipse.org incident as of time of writing). Signed-off-by: Johannes Schindelin <[email protected]>
1 parent a704627 commit 0efa9e7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ci/install-dependencies.sh

Lines changed: 1 addition & 1 deletion
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,

0 commit comments

Comments
 (0)