[OpenJDK 2D-Dev] RFR: JDK-8003593: build-infra: Paths to optional platform-specific files should not be hardwired to src/closed

Erik Joelsson erik.joelsson at oracle.com
Fri Jul 1 18:16:33 UTC 2016

On 2016-07-01 19:59, Phil Race wrote:
> erik,
> FWIW "CLOSED" implies better to me what this is about than "CUSTOM".
"custom" is a term that we have been using for a while now instead of 
"closed" in the open parts of the build to refer to any kind of custom 
addition to OpenJDK. There are quite a few instances of macros and 
variables named that way, mostly in configure but also in the makefiles. 
If we were to change this variable to "closed", then the other places 
should go with it to match. I think that's a separate change that 
require a separate discussion.
> http://cr.openjdk.java.net/~erikj/8003593/webrev.01/jdk/make/mapfiles/libfontmanager/mapfile-vers.sdiff.html 
> Regarding all the freetype symbols in here .. they aren't used in the 
> Oracle JDK, so is there another
> closed version of this file for the 'custom' source ?
Yes, it's in the closed review. The pattern is to have open mapfiles in 
the open and OracleJDK specific mapfiles in the Oracle closed repository.
> 57 ifdef OPENJDK
> $(JDK_TOPDIR)/make/mapfiles/libjpeg/mapfile-vers
>  459 else
> $(JDK_TOPDIR)/make/mapfiles/libjpeg/mapfile-vers-closed
>  461   LIBJAVAJPEG_SRC += 
> $(JDK_TOPDIR)/src/closed/java.desktop/share/native/libjavajpeg
>  462 endif
>  463
> Where is the closed replacement for this ? In another review you will 
> send out internally ?
>>  I have tested these changes extensively using the compare script and 
>> -testset buildinfra in JPRT
> So this verifies the resulting "bits" are correct ?
Yes. I went through a number of iterations to iron out all the details 
using this technique. It's quite powerful.
>  .. and that includes the various combinations you are providing  ?
> - build openjdk in presence of only openjdk
> - build oracle jdk in presence of open+closed
> - build 'openjdk-only" in presence of open+closed
Actually the first one is missing because of how JPRT works, but I could 
run an extra round with just that. That case is the least complicated 
however since it will just build what is there.


More information about the 2d-dev mailing list