Revert "support JAVA_HOME environment variable (#756)" (#761)

wing328 · web-flow · commit 2a9aef9ca4bd · 2024-03-06T13:39:38.000+08:00
This reverts commit 3e949d8.
diff --git a/apps/generator-cli/src/README.md b/apps/generator-cli/src/README.md
@@ -10,7 +10,7 @@ OpenAPI Generator allows generation of API client libraries (SDK generation), se
 configuration automatically given an OpenAPI Spec (both 2.0 and 3.0 are supported). Please see
 [OpenAPITools/openapi-generator](https://github.com/OpenAPITools/openapi-generator).
 
-The OpenAPI Generator is a Java project. `openapi-generator-cli` will download the appropriate JAR file and invoke the `java` executable to run the OpenAPI Generator. You must have the `java` binary executable available on your `PATH` or have set `JAVA_HOME` correctly for this to work.
+The OpenAPI Generator is a Java project. `openapi-generator-cli` will download the appropriate JAR file and invoke the `java` executable to run the OpenAPI Generator. You must have the `java` binary executable available on your `PATH` for this to work.
 
 If you find this tool useful, please consider sponsoring this project financially via https://opencollective.com/openapi_generator or directly to [Kay Schecker](https://github.com/sponsors/kay-schecker) (the author of this tool) :pray:
 
diff --git a/apps/generator-cli/src/app/helpers/index.ts b/apps/generator-cli/src/app/helpers/index.ts
diff --git a/apps/generator-cli/src/app/services/generator.service.spec.ts b/apps/generator-cli/src/app/services/generator.service.spec.ts
@@ -1,7 +1,6 @@
 import { Test } from '@nestjs/testing';
 import { GeneratorService } from './generator.service';
 import { LOGGER } from '../constants';
-import { javaCmd } from '../helpers';
 import { VersionManagerService } from './version-manager.service';
 import { ConfigService } from './config.service';
 
@@ -127,21 +126,20 @@ describe('GeneratorService', () => {
         });
       });
 
-      const cmd = (name: string, javaCmd: string, appendix: string[]) => ({
+      const cmd = (name, appendix: string[]) => ({
         name,
-        command: `${javaCmd} -jar "/path/to/4.2.1.jar" generate ${appendix.join(
+        command: `java -jar "/path/to/4.2.1.jar" generate ${appendix.join(
           ' '
         )}`,
       });
 
       const cmdWithCustomJar = (
         name: string,
-        javaCmd: string,
         customJar: string,
         appendix: string[]
       ) => ({
         name,
-        command: `${javaCmd} -cp "/path/to/4.2.1.jar:${customJar}" org.openapitools.codegen.OpenAPIGenerator generate ${appendix.join(
+        command: `java -cp "/path/to/4.2.1.jar:${customJar}" org.openapitools.codegen.OpenAPIGenerator generate ${appendix.join(
           ' '
         )}`,
       });
@@ -150,19 +148,19 @@ describe('GeneratorService', () => {
         [
           'foo.json',
           [
-            cmd('[angular] abc/app/pet.yaml', javaCmd, [
+            cmd('[angular] abc/app/pet.yaml', [
               `--input-spec="${cwd}/abc/app/pet.yaml"`,
               `--output="${cwd}/generated-sources/openapi/typescript-angular/pet"`,
               `--generator-name="typescript-angular"`,
               `--additional-properties="fileNaming=kebab-case,apiModulePrefix=Pet,npmName=petRestClient,supportsES6=true,withInterfaces=true"`,
             ]),
-            cmd('[angular] abc/app/car.yaml', javaCmd, [
+            cmd('[angular] abc/app/car.yaml', [
               `--input-spec="${cwd}/abc/app/car.yaml"`,
               `--output="${cwd}/generated-sources/openapi/typescript-angular/car"`,
               `--generator-name="typescript-angular"`,
               `--additional-properties="fileNaming=kebab-case,apiModulePrefix=Car,npmName=carRestClient,supportsES6=true,withInterfaces=true"`,
             ]),
-            cmd('[baz] def/app/pet.yaml', javaCmd, [
+            cmd('[baz] def/app/pet.yaml', [
               `--input-spec="${cwd}/def/app/pet.yaml"`,
               `--name="pet"`,
               `--name-uc-first="Pet"`,
@@ -176,7 +174,7 @@ describe('GeneratorService', () => {
               '--some-bool',
               '--some-int=1',
             ]),
-            cmd('[baz] def/app/car.json', javaCmd, [
+            cmd('[baz] def/app/car.json', [
               `--input-spec="${cwd}/def/app/car.json"`,
               `--name="car"`,
               `--name-uc-first="Car"`,
@@ -195,12 +193,12 @@ describe('GeneratorService', () => {
         [
           'bar.json',
           [
-            cmd('[bar] api/cat.yaml', javaCmd, [
+            cmd('[bar] api/cat.yaml', [
               `--input-spec="${cwd}/api/cat.yaml"`,
               `--output="bar/cat"`,
               '--some-bool',
             ]),
-            cmd('[bar] api/bird.json', javaCmd, [
+            cmd('[bar] api/bird.json', [
               `--input-spec="${cwd}/api/bird.json"`,
               `--output="bar/bird"`,
               '--some-bool',
@@ -210,26 +208,16 @@ describe('GeneratorService', () => {
         [
           'bar.json',
           [
-            cmdWithCustomJar(
-              '[bar] api/cat.yaml',
-              javaCmd,
-              '../some/custom.jar',
-              [
-                `--input-spec="${cwd}/api/cat.yaml"`,
-                `--output="bar/cat"`,
-                '--some-bool',
-              ]
-            ),
-            cmdWithCustomJar(
-              '[bar] api/bird.json',
-              javaCmd,
-              '../some/custom.jar',
-              [
-                `--input-spec="${cwd}/api/bird.json"`,
-                `--output="bar/bird"`,
-                '--some-bool',
-              ]
-            ),
+            cmdWithCustomJar('[bar] api/cat.yaml', '../some/custom.jar', [
+              `--input-spec="${cwd}/api/cat.yaml"`,
+              `--output="bar/cat"`,
+              '--some-bool',
+            ]),
+            cmdWithCustomJar('[bar] api/bird.json', '../some/custom.jar', [
+              `--input-spec="${cwd}/api/bird.json"`,
+              `--output="bar/bird"`,
+              '--some-bool',
+            ]),
           ],
           '../some/custom.jar',
         ],
@@ -238,7 +226,7 @@ describe('GeneratorService', () => {
         [
           'no-glob.json',
           [
-            cmd('[noGlob] http://example.local/openapi.json', javaCmd, [
+            cmd('[noGlob] http://example.local/openapi.json', [
               `--input-spec="http://example.local/openapi.json"`,
               `--output="no-glob/openapi"`,
               `--name="openapi"`,
diff --git a/apps/generator-cli/src/app/services/generator.service.ts b/apps/generator-cli/src/app/services/generator.service.ts
@@ -10,7 +10,6 @@ import * as os from 'os';
 import { VersionManagerService } from './version-manager.service';
 import { ConfigService } from './config.service';
 import { LOGGER } from '../constants';
-import { javaCmd } from '../helpers';
 
 interface GeneratorConfig {
   glob: string;
@@ -242,8 +241,7 @@ export class GeneratorService {
           this.isWin() ? ';' : ':'
         )}" org.openapitools.codegen.OpenAPIGenerator`
       : `-jar "${cliPath}"`;
-
-    return [javaCmd, process.env['JAVA_OPTS'], subCmd, 'generate', appendix]
+    return ['java', process.env['JAVA_OPTS'], subCmd, 'generate', appendix]
       .filter(isString)
       .join(' ');
   };
diff --git a/apps/generator-cli/src/app/services/pass-through.service.spec.ts b/apps/generator-cli/src/app/services/pass-through.service.spec.ts
@@ -2,7 +2,6 @@ import { Test } from '@nestjs/testing';
 import chalk from 'chalk';
 import { Command, createCommand } from 'commander';
 import { COMMANDER_PROGRAM, LOGGER } from '../constants';
-import { javaCmd } from '../helpers';
 import { GeneratorService } from './generator.service';
 import { PassThroughService } from './pass-through.service';
 import { VersionManagerService } from './version-manager.service';
@@ -193,7 +192,7 @@ describe('PassThroughService', () => {
             await program.parseAsync([name, ...argv], { from: 'user' });
             expect(childProcess.spawn).toHaveBeenNthCalledWith(
               1,
-              `${javaCmd} -jar "/some/path/to/4.2.1.jar"`,
+              'java -jar "/some/path/to/4.2.1.jar"',
               [name, ...argv],
               {
                 stdio: 'inherit',
@@ -207,7 +206,7 @@ describe('PassThroughService', () => {
             await program.parseAsync([name, ...argv], { from: 'user' });
             expect(childProcess.spawn).toHaveBeenNthCalledWith(
               1,
-              `${javaCmd} java-opt-1=1 -jar "/some/path/to/4.2.1.jar"`,
+              'java java-opt-1=1 -jar "/some/path/to/4.2.1.jar"',
               [name, ...argv],
               {
                 stdio: 'inherit',
@@ -225,7 +224,7 @@ describe('PassThroughService', () => {
 
             expect(childProcess.spawn).toHaveBeenNthCalledWith(
               1,
-              `${javaCmd} -cp "${[
+              `java -cp "${[
                 '/some/path/to/4.2.1.jar',
                 '../some/custom.jar',
               ].join(cpDelimiter)}" org.openapitools.codegen.OpenAPIGenerator`,
@@ -304,7 +303,7 @@ describe('PassThroughService', () => {
                   it('spawns the correct process', () => {
                     expect(childProcess.spawn).toHaveBeenNthCalledWith(
                       1,
-                      `${javaCmd} -jar "/some/path/to/4.2.1.jar"`,
+                      'java -jar "/some/path/to/4.2.1.jar"',
                       cmd.split(' '),
                       { stdio: 'inherit', shell: true }
                     );
diff --git a/apps/generator-cli/src/app/services/pass-through.service.ts b/apps/generator-cli/src/app/services/pass-through.service.ts
@@ -4,7 +4,6 @@ import { exec, spawn } from 'child_process';
 import { Command } from 'commander';
 import { isString, startsWith, trim } from 'lodash';
 import { COMMANDER_PROGRAM, LOGGER } from '../constants';
-import { javaCmd } from '../helpers';
 import { GeneratorService } from './generator.service';
 import { VersionManagerService } from './version-manager.service';
 import { ConfigService } from './config.service';
@@ -143,7 +142,7 @@ export class PassThroughService {
         )}" org.openapitools.codegen.OpenAPIGenerator`
       : `-jar "${cliPath}"`;
 
-    return [javaCmd, process.env['JAVA_OPTS'], subCmd]
+    return ['java', process.env['JAVA_OPTS'], subCmd]
       .filter(isString)
       .join(' ');
   }
