Add support for compile-time input files in java_binary and java_library #21688
Labels
P2
We'll consider working on this in future. (Assignee optional)
team-Rules-Java
Issues for Java rules
type: feature request
Description of the feature request:
Introduce a new attribute, tentatively named
java_compiler_inputs
, that would specifically hold references to compile-time input files. Additionally, provide support for expanding these file paths using$(location)
or similar mechanisms, either within the existing javacopts attribute or a potentially new, toolchain-flag-specific attribute.Example: Code snippet
Which category does this issue belong to?
java rules
What underlying problem are you trying to solve with this feature?
I am working with a custom Java toolchain (specifically, ECJ https://github.com/salesforce/bazel-jdt-java-toolchain) that supports compile-time options like -properties to consume configuration files during compilation. Currently, there is no dedicated way to provide such files to the Java compiler within Bazel's Java rules. Including these files in the srcs attribute works in some sandbox environments but leads to failures with the processwrapper sandbox.
This feature would improve flexibility when using custom Java toolchains and ensure compatibility across different sandboxing strategies within Bazel. It would be valuable to draw parallels with a similar mechanism already present in Bazel's C++ rules for linker inputs (additional_linker_inputs and user_link_flags).
Which operating system are you running Bazel on?
RHEL7
What is the output of
bazel info release
?7.0.2
If
bazel info release
returnsdevelopment version
or(@non-git)
, tell us how you built Bazel.No response
What's the output of
git remote get-url origin; git rev-parse HEAD
?No response
Have you found anything relevant by searching the web?
Discussion on providing -properties file, no answers: salesforce/bazel-jdt-java-toolchain#19
Any other information, logs, or outputs that you want to share?
[Sandbox Error]: with processwrapper sandbox
The text was updated successfully, but these errors were encountered: