actions/setup-java
2
0
Fork 0
mirror of https://github.com/actions/setup-java.git synced 2025-04-21 02:16:45 +00:00
Code Issues Projects Releases Packages Wiki Activity

Refactor arch mapping to distributionArchitecture method

Browse source
This commit is contained in:
Wes Morgan 2022-08-31 13:05:33 -06:00
parent cca7653671
commit e169777d50
No known key found for this signature in database
GPG key ID: 5639E4CBFA17DC84
2 changed files with 9 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/distributions/base-installer.ts
View file

@ -149,4 +149,8 @@ export abstract class JavaBase {
core.setOutput('path', toolPath);
core.setOutput('version', version);
}
protected distributionArchitecture(): string {
return this.architecture;
}
}

10
src/distributions/temurin/installer.ts
View file

@ -20,7 +20,6 @@ export class TemurinDistribution extends JavaBase {
private readonly jvmImpl: TemurinImplementation
) {
super(`Temurin-${jvmImpl}`, installerOptions);
installerOptions.architecture = this.osArchToDistributionArch(installerOptions.architecture);
}
protected async findPackageForDownload(version: string): Promise<JavaDownloadRelease> {
@ -87,7 +86,7 @@ export class TemurinDistribution extends JavaBase {
private async getAvailableVersions(): Promise<ITemurinAvailableVersions[]> {
const platform = this.getPlatformOption();
const arch = this.architecture;
const arch = this.distributionArchitecture();
const imageType = this.packageType;
const versionRange = encodeURI('[1.0,100.0]'); // retrieve all available versions
const releaseType = this.stable ? 'ga' : 'ea';
@ -154,8 +153,9 @@ export class TemurinDistribution extends JavaBase {
}
}
private osArchToDistributionArch(osArch: string): string {
switch (osArch) {
protected distributionArchitecture(): string {
// Temurin has own architecture names so need to map them
switch (this.architecture) {
case 'amd64':
return 'x64';
case 'ia32':
@ -163,7 +163,7 @@ export class TemurinDistribution extends JavaBase {
case 'arm64':
return 'aarch64';
default:
return osArch;
return this.architecture;
}
}
}