Skip to content

Commit cf3e355

Browse files
committed
add windows case
1 parent b97e00c commit cf3e355

File tree

5 files changed

+17
-13
lines changed

5 files changed

+17
-13
lines changed

.github/workflows/workflow.yml

+6-6
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
uses: ./
4040
with:
4141
java-version: 13.0.2
42-
vendor: adoptopenjdk
42+
distro: adoptopenjdk
4343
- name: Verify Java 13
4444
if: runner.os != 'windows'
4545
run: __tests__/verify-java.sh 13.0.2 "${{ steps.setup-java-adopt.outputs.path }}" "${{ steps.setup-java-adopt.outputs.version }}"
@@ -66,7 +66,7 @@ jobs:
6666
uses: ./
6767
with:
6868
java-version: 13.0.2
69-
vendor: zulu
69+
distro: zulu
7070
- name: Verify Java 13
7171
if: runner.os != 'windows'
7272
run: __tests__/verify-java.sh 13.0.2 "${{ steps.setup-java-zulu.outputs.path }}" "${{ steps.setup-java-zulu.outputs.version }}"
@@ -94,9 +94,9 @@ jobs:
9494
uses: ./
9595
with:
9696
java-version: 13
97-
vendor: 'adoptopenjdk'
97+
distro: 'adoptopenjdk'
9898
- name: Verify Java 13
99-
run: __tests__/verify-java.sh 13.0.2
99+
run: __tests__/verify-java.sh 13.0.2 "${{ steps.setup-java.outputs.path }}" "${{ steps.setup-java.outputs.version }}"
100100

101101

102102
test-proxy-zulu:
@@ -120,7 +120,7 @@ jobs:
120120
uses: ./
121121
with:
122122
java-version: 13.0.2
123-
vendor: zulu
123+
distro: zulu
124124
- name: Verify Java 13
125125
run: __tests__/verify-java.sh 13.0.2 "${{ steps.setup-java.outputs.path }}" "${{ steps.setup-java.outputs.version }}"
126126

@@ -138,6 +138,6 @@ jobs:
138138
uses: ./
139139
with:
140140
java-version: 13.0.2
141-
vendor: zulu
141+
distro: zulu
142142
- name: Verify Java 13
143143
run: __tests__/verify-java.sh 13.0.2 "${{ steps.setup-java.outputs.path }}" "${{ steps.setup-java.outputs.version }}"

dist/setup/index.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -17461,6 +17461,9 @@ function getJavaAdoptOpenJDK(version, javaPackage, arch) {
1746117461
case 'darwin':
1746217462
OS = 'mac';
1746317463
break;
17464+
case 'win32':
17465+
OS = 'windows';
17466+
break;
1746417467
}
1746517468
const http = new httpm.HttpClient('setup-java', undefined, {
1746617469
allowRetries: true,
@@ -28769,13 +28772,12 @@ function run() {
2876928772
if (!version) {
2877028773
version = core.getInput(constants.INPUT_JAVA_VERSION, { required: true });
2877128774
}
28772-
const vendor = core.getInput('vendor', { required: true });
2877328775
const arch = core.getInput(constants.INPUT_ARCHITECTURE, { required: true });
2877428776
const javaPackage = core.getInput(constants.INPUT_JAVA_PACKAGE, {
2877528777
required: true
2877628778
});
2877728779
const jdkFile = core.getInput(constants.INPUT_JDK_FILE, { required: false });
28778-
const distro = core.getInput('vendor', { required: true });
28780+
const distro = core.getInput('distro', { required: true });
2877928781
yield installer.getJava(version, distro, arch, jdkFile, javaPackage);
2878028782
const matchersPath = path.join(__dirname, '..', '..', '.github');
2878128783
core.info(`##[add-matcher]${path.join(matchersPath, 'java.json')}`);

package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
"main": "dist/index.js",
77
"scripts": {
88
"build": "ncc build -o dist/setup src/setup-java.ts && ncc build -o dist/cleanup src/cleanup-java.ts",
9-
"format": "prettier --write **/*.ts",
10-
"format-check": "prettier --check **/*.ts",
9+
"format": "prettier --write **/*.ts src/distro/*.ts",
10+
"format-check": "prettier --check **/*.ts src/distro/*.ts",
1111
"prerelease": "npm run-script build",
1212
"release": "git add -f dist/setup/index.js dist/cleanup/index.js",
1313
"test": "jest"

src/distro/adoptopenjdk.ts

+4-1
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,13 @@ export async function getJavaAdoptOpenJDK(
1111

1212
let OS = process.platform.toString();
1313

14-
switch(process.platform) {
14+
switch (process.platform) {
1515
case 'darwin':
1616
OS = 'mac';
1717
break;
18+
case 'win32':
19+
OS = 'windows';
20+
break;
1821
}
1922

2023
const http = new httpm.HttpClient('setup-java', undefined, {

src/setup-java.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,12 @@ async function run() {
1111
if (!version) {
1212
version = core.getInput(constants.INPUT_JAVA_VERSION, {required: true});
1313
}
14-
const vendor = core.getInput('vendor', {required: true});
1514
const arch = core.getInput(constants.INPUT_ARCHITECTURE, {required: true});
1615
const javaPackage = core.getInput(constants.INPUT_JAVA_PACKAGE, {
1716
required: true
1817
});
1918
const jdkFile = core.getInput(constants.INPUT_JDK_FILE, {required: false});
20-
const distro = core.getInput('vendor', {required: true});
19+
const distro = core.getInput('distro', {required: true});
2120

2221
await installer.getJava(version, distro, arch, jdkFile, javaPackage);
2322

0 commit comments

Comments
 (0)