On Tue, 26 Nov 2024 15:40:13 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
>> make/modules/java.desktop/lib/ClientLibraries.gmk line 166: >> >>> 164: jfdctflt.o jfdctfst.o jfdctint.o jidctflt.o jidctfst.o >>> jidctint.o \ >>> 165: jidctred.o jmemmgr.o jmemnobs.o jpegdecoder.o jquant1.o >>> jquant2.o \ >>> 166: jutils.o >> >> If libsplashscreen is already excluded in StaticLibs.gmk, why do we need to >> exclude obj files here? >> >> Would it be possible to generate this list? I assume it's all the src files >> in libjavajpeg, but with the object file suffix. > > It's only excluded on Linux and Windows. (And that is just a silly one > function with a duplicated name, that we could easily fix) > > It's all src files from libjavajpeg that originates from upstream, but we add > in some additional files. But if you prefer I can rewrite this as a wildcard > for libjavajpeg and then filter out our files. (And replace the file suffix, > of course.) Fixed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20837#discussion_r1858909321