Skip to content

Commit

Permalink
Remove C# and Java targets (#11551)
Browse files Browse the repository at this point in the history
* remove generators and gencommon

* remove C# target

* remove ilib

* remove C# std

* remove C# from tests

* disable Java tests too

* remove from docgen

* remove Java target

* remove javalib

* purge ilib and javalib

* disable Java-specific test

* fix merge mixhaps

* remove C# tests again

* fix some tests

* annoying

* also annoying

* one more

* still install format

* fix

* more fix

* bring back gitkeep and delete more stuff

* Remove unused std files

Or at least shouldn't be used...

* Update docgen

* That can't be used anymore

* Keep hxjava

* Leftovers

* [ci] update workflow sources

* [tests] update special handling

* [docgen] whoops

* More special handling removal

* Might be very old, but still seems to apply to jvm too..

* They are everywhere...

* What the hell was that?

* Remove more...

* Remove even more...

* Fix platform

* Warnings

* [ci] break jvm

* [tests] Sockets don't seem to be implemented for jvm

* [tests] Fix Simon's favorite test again

* Remove java.internal.StringExt

* Remove more java internals

* That's not a target anymore

* Forget about cs

* Warnings

* [typer] don't type trailing optional bind arguments so weirdly

see #11531

* [tests] jvm is not ready for this one yet

* Use -jvm

* More --jvm/-jvm arg stuff

* Fix -jvm...

* Output jar files

* jvm is now a reserved define

---------

Co-authored-by: Rudy Ges <k@klabz.org>
  • Loading branch information
Simn and kLabz authored Feb 6, 2024
1 parent dbc9952 commit 2dfc0d9
Show file tree
Hide file tree
Showing 423 changed files with 472 additions and 42,013 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ jobs:
fail-fast: false
matrix:
ocaml: ["4.08.1", "5.0.0"]
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, lua, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, lua, flash, neko]
include:
- target: hl
APT_PACKAGES: cmake ninja-build libturbojpeg-dev
Expand Down Expand Up @@ -519,7 +519,7 @@ jobs:
haxe dox.hxml
mkdir resources
cp ../../src-json/* resources
cpp/Dox -i ../../xmldoc -ex microsoft -ex javax -ex cs.internal -theme $(haxelib libpath dox)/themes/default
cpp/Dox -i ../../xmldoc -ex microsoft -ex javax -theme $(haxelib libpath dox)/themes/default
working-directory: ${{github.workspace}}/tests/docgen

linux-arm64:
Expand Down Expand Up @@ -690,7 +690,7 @@ jobs:
fail-fast: false
matrix:
# TODO enable lua after https://github.com/HaxeFoundation/haxe/issues/10919
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, flash, neko]
steps:
- uses: actions/checkout@main
with:
Expand Down Expand Up @@ -787,7 +787,7 @@ jobs:
matrix:
# TODO jvm: https://github.com/HaxeFoundation/haxe/issues/8601
# TODO enable lua after https://github.com/HaxeFoundation/haxe/issues/10919
target: [macro, js, hl, cpp, java, cs, php, python, flash, neko]
target: [macro, js, hl, cpp, php, python, flash, neko]
steps:
- uses: actions/checkout@main
with:
Expand Down Expand Up @@ -881,7 +881,7 @@ jobs:
strategy:
fail-fast: false
matrix:
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, flash, neko]
include:
- target: hl
BREW_PACKAGES: ninja
Expand Down
3 changes: 1 addition & 2 deletions .vscode/schemas/meta.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,7 @@
"flash",
"php",
"cpp",
"cs",
"java",
"jvm",
"python",
"hl",
"eval"
Expand Down
13 changes: 0 additions & 13 deletions Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,6 @@ xmldoc:
RUN haxelib newrepo
RUN haxelib git hxcpp https://github.com/HaxeFoundation/hxcpp
RUN haxelib git hxjava https://github.com/HaxeFoundation/hxjava
RUN haxelib git hxcs https://github.com/HaxeFoundation/hxcs
RUN haxe doc.hxml

ARG COMMIT
Expand Down Expand Up @@ -271,11 +270,6 @@ test-environment-php:
DO +INSTALL_PACKAGES --PACKAGES="php-cli php-mbstring php-sqlite3"
SAVE IMAGE --cache-hint

test-environment-cs:
FROM +test-environment
DO +INSTALL_PACKAGES --PACKAGES="mono-devel mono-mcs"
SAVE IMAGE --cache-hint

test-environment-hl:
FROM +test-environment
DO +INSTALL_PACKAGES --PACKAGES="cmake ninja-build libturbojpeg-dev libpng-dev zlib1g-dev libvorbis-dev libsqlite3-dev"
Expand Down Expand Up @@ -361,12 +355,6 @@ test-jvm:
ENV GITHUB_ACTIONS=$GITHUB_ACTIONS
DO +RUN_CI --TEST=jvm

test-cs:
FROM +test-environment-cs
ARG GITHUB_ACTIONS
ENV GITHUB_ACTIONS=$GITHUB_ACTIONS
DO +RUN_CI --TEST=cs

test-php:
FROM +test-environment-php
ARG GITHUB_ACTIONS
Expand Down Expand Up @@ -400,7 +388,6 @@ test-all:
BUILD +test-python
BUILD +test-java
BUILD +test-jvm
BUILD +test-cs
BUILD +test-cpp
BUILD +test-lua
BUILD +test-js
Expand Down
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,6 @@ xmldoc:
$(CURDIR)/$(HAXELIB_OUTPUT) newrepo && \
$(CURDIR)/$(HAXELIB_OUTPUT) git hxcpp https://github.com/HaxeFoundation/hxcpp && \
$(CURDIR)/$(HAXELIB_OUTPUT) git hxjava https://github.com/HaxeFoundation/hxjava && \
$(CURDIR)/$(HAXELIB_OUTPUT) git hxcs https://github.com/HaxeFoundation/hxcs && \
PATH="$(CURDIR):$(PATH)" $(CURDIR)/$(HAXE_OUTPUT) doc.hxml

$(INSTALLER_TMP_DIR):
Expand Down
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ Haxe allows you to compile for the following targets:

* JavaScript
* C++
* C#
* Java
* JVM
* Lua
* PHP 7
Expand Down
9 changes: 2 additions & 7 deletions extra/ImportAll.hx
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class ImportAll {

static function isSysTarget() {
return Context.defined("neko") || Context.defined("php") || Context.defined("cpp") ||
Context.defined("java") || Context.defined("python") ||
Context.defined("jvm") || Context.defined("python") ||
Context.defined("lua") || Context.defined("hl") || Context.defined("eval");
}

Expand All @@ -51,12 +51,8 @@ class ImportAll {
if(!isSysTarget()) return;
case "sys.thread":
if ( !Context.defined("target.threaded") ) return;
case "java":
if( !Context.defined("java") ) return;
case "jvm":
case "java" | "jvm":
if( !Context.defined("jvm") ) return;
case "cs":
if( !Context.defined("cs") ) return;
case "python":
if ( !Context.defined("python") ) return;
case "hl":
Expand Down Expand Up @@ -96,7 +92,6 @@ class ImportAll {
case "haxe.remoting.SocketWrapper": if( !Context.defined("flash") ) continue;
case "haxe.remoting.SyncSocketConnection": if( !(Context.defined("neko") || Context.defined("php") || Context.defined("cpp")) ) continue;
case "neko.vm.Ui" | "sys.db.Sqlite" | "sys.db.Mysql" if ( Context.defined("interp") ): continue;
case "sys.db.Sqlite" | "sys.db.Mysql" | "cs.db.AdoNet" if ( Context.defined("cs") ): continue;
case "haxe.atomic.AtomicBool" if(!Context.defined("target.atomics")): continue;
case "haxe.atomic.AtomicInt" if(!Context.defined("target.atomics")): continue;
case "haxe.atomic.AtomicObject" if(!Context.defined("target.atomics") || Context.defined("js") || Context.defined("cpp")): continue;
Expand Down
9 changes: 0 additions & 9 deletions extra/all.hxml
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,10 @@
-xml cpp.xml
-D HXCPP_MULTI_THREADED

--next
-java all_java
-xml java.xml

--next
--jvm all_jvm
-xml jvm.xml

--next
-cs all_cs
-D unsafe
-xml cs.xml

--next
-python all_python
-xml python.xml
Expand Down
9 changes: 2 additions & 7 deletions extra/doc.hxml
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,8 @@
-D HXCPP_MULTI_THREADED

--next
-java all_java
-xml doc/java.xml

--next
-cs all_cs
-D unsafe
-xml doc/cs.xml
--jvm all_jvm
-xml doc/jvm.xml

--next
-python all_py
Expand Down
10 changes: 5 additions & 5 deletions extra/github-actions/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ jobs:
fail-fast: false
matrix:
ocaml: ["4.08.1", "5.0.0"]
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, lua, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, lua, flash, neko]
include:
- target: hl
APT_PACKAGES: cmake ninja-build libturbojpeg-dev
Expand Down Expand Up @@ -269,7 +269,7 @@ jobs:
haxe dox.hxml
mkdir resources
cp ../../src-json/* resources
cpp/Dox -i ../../xmldoc -ex microsoft -ex javax -ex cs.internal -theme $(haxelib libpath dox)/themes/default
cpp/Dox -i ../../xmldoc -ex microsoft -ex javax -theme $(haxelib libpath dox)/themes/default
working-directory: ${{github.workspace}}/tests/docgen

linux-arm64:
Expand Down Expand Up @@ -362,7 +362,7 @@ jobs:
fail-fast: false
matrix:
# TODO enable lua after https://github.com/HaxeFoundation/haxe/issues/10919
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, flash, neko]
steps:
- uses: actions/checkout@main
with:
Expand All @@ -389,7 +389,7 @@ jobs:
matrix:
# TODO jvm: https://github.com/HaxeFoundation/haxe/issues/8601
# TODO enable lua after https://github.com/HaxeFoundation/haxe/issues/10919
target: [macro, js, hl, cpp, java, cs, php, python, flash, neko]
target: [macro, js, hl, cpp, php, python, flash, neko]
steps:
- uses: actions/checkout@main
with:
Expand All @@ -413,7 +413,7 @@ jobs:
strategy:
fail-fast: false
matrix:
target: [macro, js, hl, cpp, 'java,jvm', cs, php, python, flash, neko]
target: [macro, js, hl, cpp, jvm, php, python, flash, neko]
include:
- target: hl
BREW_PACKAGES: ninja
Expand Down
2 changes: 1 addition & 1 deletion extra/release-checklist.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

- Check that haxelib is working
- Make sure to update the haxelib submodule
- Check that the run-time haxelibs are ready for release: hxcpp, hxjava, hxcs
- Check that the run-time haxelibs are ready for release: hxcpp, hxjava
- Check that the NEKO_VERSION variable in the "Makefile" is set to the latest Neko version

# Making the release
Expand Down
26 changes: 0 additions & 26 deletions libs/ilib/Makefile

This file was deleted.

38 changes: 0 additions & 38 deletions libs/ilib/dump.ml

This file was deleted.

15 changes: 0 additions & 15 deletions libs/ilib/dune

This file was deleted.

Loading

0 comments on commit 2dfc0d9

Please sign in to comment.