hg: build-infra/jdk8/corba: Excluding files from compile that aren't compiled in the old build.
Kelly O'Hair
kelly.ohair at oracle.com
Thu Jan 19 08:30:28 PST 2012
Some of the old old Makefile had Files.gmk files that explicitly listed the java source files.
Many were wrong.
Years ago when I tried to get rid of the explicit file lists, there were some that I did not fix because fixing the
issues would have added more class files to the JDK that I wasn't sure was right.
Unfortunately, when it comes to corba, no many people know any answers.
I doubt the exclusion was explicit or on purpose, but I don't know what the consequences are of adding them in.
Sometimes, the explicit lists of sources being incomplete was not an issue because due to implicit compilation
the missing sources would get dragged in because of dependencies and the class files showed up anyway.
Ignore the src.zip, it is delivered as an independent thing and has nothing to do with what is actually compiled, has some specific
rules about how it is created and what can get into it.
-kto
On Jan 19, 2012, at 5:19 AM, Erik Joelsson wrote:
> I find the excluded files here weird. The sources are still included in the src.zip. Does anyone know why these files aren't compiled into the product? It seems to me the exclusion isn't explicit in the old build system.
>
> The files in question are:
>
> com/sun/corba/se/PortableActivationIDL/*
> com/sun/corba/se/impl/presentation/rmi/JNDIStateFactoryImpl.java
> com/sun/corba/se/spi/presentation/rmi/StubWrapper.java \
> com/sun/org/omg/CORBA/IDLTypeOperations.java \
> com/sun/org/omg/CORBA/IRObjectOperations.java \
> org/omg/PortableInterceptor/UNKNOWN.java,\
>
> /Erik
>
> On 2012-01-19 13:23, erik.joelsson at oracle.com wrote:
>> Changeset: bc755e1c21cc
>> Author: erikj
>> Date: 2012-01-19 13:23 +0100
>> URL: http://hg.openjdk.java.net/build-infra/jdk8/corba/rev/bc755e1c21cc
>>
>> Excluding files from compile that aren't compiled in the old build.
>>
>> ! makefiles/Makefile
>>
More information about the build-infra-dev
mailing list