Skip to content

Commit

Permalink
[4.3.6] Fix --java out -D jvm deprecation warning (#11739)
Browse files Browse the repository at this point in the history
* Fix detection of --java out -D jvm

* [tests] add test for 11737
  • Loading branch information
kLabz authored Jul 29, 2024
1 parent bd79571 commit 7290013
Show file tree
Hide file tree
Showing 10 changed files with 13 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/compiler/compiler.ml
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@ module Setup = struct
| Java ->
Java.before_generate com;
if defined com Define.Jvm then begin
if not actx.jvm_flag then com.warning WDeprecated com.warning_options ("--java out.jar -D jvm is deprecated; use --jvm out.jar directly") null_pos;
add_std "jvm";
com.package_rules <- PMap.remove "jvm" com.package_rules;
end;
Expand Down
5 changes: 2 additions & 3 deletions src/compiler/generate.ml
Original file line number Diff line number Diff line change
Expand Up @@ -81,10 +81,9 @@ let generate ctx tctx ext actx =
| Cs ->
Gencs.generate,"cs"
| Java ->
if Common.defined com Jvm then begin
com.warning WDeprecated com.warning_options ("--java out.jar -D jvm is deprecated; use --jvm out.jar directly") null_pos;
if Common.defined com Jvm then
Genjvm.generate actx.jvm_flag,"java"
end else
else
Genjava.generate,"java"
| Python ->
Genpy.generate,"python"
Expand Down
1 change: 1 addition & 0 deletions tests/misc/projects/Issue11737/Main.hx
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
function main() {}
1 change: 1 addition & 0 deletions tests/misc/projects/Issue11737/_setup.hxml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
--cmd haxelib install --quiet hxjava
3 changes: 3 additions & 0 deletions tests/misc/projects/Issue11737/compile.hxml
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
--main Main
--java bin
-D jvm
1 change: 1 addition & 0 deletions tests/misc/projects/Issue11737/compile.hxml.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Warning : (WDeprecated) --java out.jar -D jvm is deprecated; use --jvm out.jar directly
2 changes: 2 additions & 0 deletions tests/misc/projects/Issue11737/compile1.hxml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
--main Main
--jvm bin/jvm.jar
Empty file.
2 changes: 2 additions & 0 deletions tests/misc/projects/Issue11737/compile2.hxml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
--main Main
--java bin
Empty file.

0 comments on commit 7290013

Please sign in to comment.