Merge pull request #11071 from tristan957/java-module

Java module 1.0.0 updates

1 files changed, 20 insertions, 46 deletions

diff --git a/mesonbuild/modules/java.py b/mesonbuild/modules/java.py
index 2779afa..792d70b 100644
--- a/mesonbuild/modules/java.py
+++ b/mesonbuild/modules/java.py
@@ -13,7 +13,6 @@
 # limitations under the License.
 from __future__ import annotations
 
-import os
 import pathlib
 import typing as T
 
@@ -23,6 +22,7 @@ from mesonbuild.compilers import detect_compiler_for
 from mesonbuild.interpreterbase.decorators import ContainerTypeInfo, FeatureDeprecated, FeatureNew, KwargInfo, typed_pos_args, typed_kwargs
 from mesonbuild.mesonlib import version_compare, MachineChoice
 from . import NewExtensionModule, ModuleReturnValue, ModuleInfo
+from ..interpreter.type_checking import NoneType
 
 if T.TYPE_CHECKING:
     from . import ModuleState
@@ -36,8 +36,8 @@ class JavaModule(NewExtensionModule):
     def __init__(self, interpreter: Interpreter):
         super().__init__()
         self.methods.update({
-            'generate_native_header': self.generate_native_header,
             'generate_native_headers': self.generate_native_headers,
+            'native_headers': self.native_headers,
         })
 
     def __get_java_compiler(self, state: ModuleState) -> Compiler:
@@ -45,59 +45,33 @@ class JavaModule(NewExtensionModule):
             detect_compiler_for(state.environment, 'java', MachineChoice.BUILD)
         return state.environment.coredata.compilers[MachineChoice.BUILD]['java']
 
-    @FeatureDeprecated('java.generate_native_header', '0.62.0', 'Use java.generate_native_headers instead')
-    @typed_pos_args('java.generate_native_header', (str, mesonlib.File))
-    @typed_kwargs('java.generate_native_header', KwargInfo('package', str, default=None))
-    def generate_native_header(self, state: ModuleState, args: T.Tuple[T.Union[str, mesonlib.File]],
-                               kwargs: T.Dict[str, T.Optional[str]]) -> ModuleReturnValue:
-        package = kwargs.get('package')
-
-        if isinstance(args[0], mesonlib.File):
-            file = args[0]
-        else:
-            file = mesonlib.File.from_source_file(state.source_root, state.subdir, args[0])
-
-        if package:
-            header = f'{package.replace(".", "_")}_{pathlib.Path(file.fname).stem}.h'
-        else:
-            header = f'{pathlib.Path(file.fname).stem}.h'
-
-        javac = self.__get_java_compiler(state)
-
-        target = CustomTarget(
-            os.path.basename(header),
-            state.subdir,
-            state.subproject,
-            state.environment,
-            mesonlib.listify([
-                javac.exelist,
-                '-d',
-                '@PRIVATE_DIR@',
-                '-h',
-                state.subdir,
-                '@INPUT@',
-            ]),
-            [file],
-            [header],
-            backend=state.backend,
-        )
-        # It is only known that 1.8.0 won't pre-create the directory. 11 and 16
-        # do not exhibit this behavior.
-        if version_compare(javac.version, '1.8.0'):
-            pathlib.Path(state.backend.get_target_private_dir_abs(target)).mkdir(parents=True, exist_ok=True)
-
-        return ModuleReturnValue(target, [target])
-
     @FeatureNew('java.generate_native_headers', '0.62.0')
+    @FeatureDeprecated('java.generate_native_headers', '1.0.0')
     @typed_pos_args(
         'java.generate_native_headers',
         varargs=(str, mesonlib.File, Target, CustomTargetIndex, GeneratedList))
     @typed_kwargs(
         'java.generate_native_headers',
         KwargInfo('classes', ContainerTypeInfo(list, str), default=[], listify=True, required=True),
-        KwargInfo('package', str, default=None))
+        KwargInfo('package', (str, NoneType), default=None))
     def generate_native_headers(self, state: ModuleState, args: T.Tuple[T.List[mesonlib.FileOrString]],
                                 kwargs: T.Dict[str, T.Optional[str]]) -> ModuleReturnValue:
+        return self.__native_headers(state, args, kwargs)
+
+    @FeatureNew('java.native_headers', '1.0.0')
+    @typed_pos_args(
+        'java.native_headers',
+        varargs=(str, mesonlib.File, Target, CustomTargetIndex, GeneratedList))
+    @typed_kwargs(
+        'java.native_headers',
+        KwargInfo('classes', ContainerTypeInfo(list, str), default=[], listify=True, required=True),
+        KwargInfo('package', (str, NoneType), default=None))
+    def native_headers(self, state: ModuleState, args: T.Tuple[T.List[mesonlib.FileOrString]],
+                       kwargs: T.Dict[str, T.Optional[str]]) -> ModuleReturnValue:
+        return self.__native_headers(state, args, kwargs)
+
+    def __native_headers(self, state: ModuleState, args: T.Tuple[T.List[mesonlib.FileOrString]],
+                         kwargs: T.Dict[str, T.Optional[str]]) -> ModuleReturnValue:
         classes = T.cast('T.List[str]', kwargs.get('classes'))
         package = kwargs.get('package')