RFR: 8361216: Do not fork javac in J2DBench ant build

Sergey Bylokhov serb at openjdk.org
Thu Jul 3 21:12:41 UTC 2025


On Thu, 3 Jul 2025 19:59:26 GMT, Eric Caspole <ecaspole at openjdk.org> wrote:

>> src/demo/share/java2d/J2DBench/build.xml line 42:
>> 
>>> 40:   <property name="resources"  location="resources"/>
>>> 41: 
>>> 42:   <condition property="source" value="8">
>> 
>> This would be inconsistent with the Make target. It doesn't seem right to update just the ant-based target.
>
> Yes I realized that yesterday, I never actually looked at or used the Makefile before. I will update the Makefile so it least this PR in its own right is consistent, regardless of these other discussion points.

I think we can just drop ant and leave the makefile as it is. The source/target settings are set to that low level to indicate that the source code should be compatible with older versions of the JDK. Currently, JDK 1.4 compatibility is still maintained.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26077#discussion_r2183757974


More information about the client-libs-dev mailing list