On Fri, 6 Mar 2026 17:13:45 GMT, Erik Joelsson <[email protected]> wrote:

>> Jan Lahoda has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Removing an unnecessary dependency as suggested.
>
> make/CompileInterimLangtools.gmk line 148:
> 
>> 146:     INCLUDES := javacserver, \
>> 147:     BIN := $(BUILDTOOLS_OUTPUTDIR)/langtools_javacserver_classes, \
>> 148:     DEPENDS := 
>> $(BUILDTOOLS_OUTPUTDIR)/gensrc/java.base.interim/jdk/internal/javac/PreviewFeature.java,
>>  \
> 
> I can't see why this dependency is needed, but maybe I'm missing something?

I think this was a left over from my attempts to solve the dependency issue, 
sorry for that. Removed now. Thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30072#discussion_r2897317751

Reply via email to