Skip to content

Commit 95c78f9

Browse files
authored
Merge pull request #274 from SkyBlade1978/master-1.12
Support hosted Java selectors in the release dispatcher
2 parents a5d5565 + 1fae0da commit 95c78f9

1 file changed

Lines changed: 11 additions & 2 deletions

File tree

‎.github/workflows/deploy-release.yml‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ jobs:
4747
loader_display: ${{ steps.validate.outputs.loader_display }}
4848
java_version: ${{ steps.validate.outputs.java_version }}
4949
java_toolchain_version: ${{ steps.validate.outputs.java_toolchain_version }}
50+
java_setup_version: ${{ steps.validate.outputs.java_setup_version }}
5051
gradle_java_version: ${{ steps.validate.outputs.gradle_java_version }}
5152
install_gradle_java: ${{ steps.validate.outputs.install_gradle_java }}
5253
curseforge_project_id: ${{ steps.validate.outputs.curseforge_project_id }}
@@ -138,6 +139,7 @@ jobs:
138139
loader_code="$(value loader_code)"
139140
java_version="$(value java_version)"
140141
java_toolchain_version="$(value java_toolchain_version)"
142+
java_setup_version="$(value java_setup_version)"
141143
gradle_java_version="$(value gradle_java_version)"
142144
curseforge_project_id="$(value curseforge_project_id)"
143145
@@ -149,6 +151,7 @@ jobs:
149151
fi
150152
done
151153
java_toolchain_version="${java_toolchain_version:-$java_version}"
154+
java_setup_version="${java_setup_version:-$java_toolchain_version}"
152155
153156
java_major() {
154157
local version="$1"
@@ -164,6 +167,11 @@ jobs:
164167
[[ "$toolchain_major" != "$2" ]]
165168
}
166169
170+
if [[ "$(java_major "$java_setup_version")" != "$(java_major "$java_toolchain_version")" ]]; then
171+
echo "Hosted Java selector $java_setup_version does not match toolchain $java_toolchain_version" >&2
172+
exit 1
173+
fi
174+
167175
java_setup_contracts=(
168176
'8.0.502+7 17 true'
169177
'16.0.2+7 17 true'
@@ -237,6 +245,7 @@ jobs:
237245
echo "loader_display=$loader_display" >> "$GITHUB_OUTPUT"
238246
echo "java_version=$java_version" >> "$GITHUB_OUTPUT"
239247
echo "java_toolchain_version=$java_toolchain_version" >> "$GITHUB_OUTPUT"
248+
echo "java_setup_version=$java_setup_version" >> "$GITHUB_OUTPUT"
240249
echo "gradle_java_version=$gradle_java_version" >> "$GITHUB_OUTPUT"
241250
echo "install_gradle_java=$install_gradle_java" >> "$GITHUB_OUTPUT"
242251
echo "curseforge_project_id=$curseforge_project_id" >> "$GITHUB_OUTPUT"
@@ -282,7 +291,7 @@ jobs:
282291
uses: actions/setup-java@b6effb05e454b25005698d916606bdc6ffcbf961 # v5
283292
with:
284293
distribution: temurin
285-
java-version: ${{ needs.preflight.outputs.java_toolchain_version }}
294+
java-version: ${{ needs.preflight.outputs.java_setup_version }}
286295

287296
- name: Install separate Java for Gradle
288297
if: needs.preflight.outputs.install_gradle_java == 'true'
@@ -437,7 +446,7 @@ jobs:
437446
uses: actions/setup-java@b6effb05e454b25005698d916606bdc6ffcbf961 # v5
438447
with:
439448
distribution: temurin
440-
java-version: ${{ needs.preflight.outputs.java_toolchain_version }}
449+
java-version: ${{ needs.preflight.outputs.java_setup_version }}
441450

442451
- name: Install separate Java for Gradle
443452
if: needs.preflight.outputs.install_gradle_java == 'true'

0 commit comments

Comments
 (0)