[8u] Request for enhancement backport approval for CR 8044235 - nashorn (and other) sources are missing in src.zip

Jiri Vanek jvanek at redhat.com
Thu Jan 5 12:15:16 UTC 2017


Hello!

There is a lot of sources missing in src.zip from jdk8 build. All those are present in jdk9 src.zip.
Even as jdk8 had intention to hide "private" sources, from the src.zip, missing nashorn can be 
considered as bug.
However, when jdk9 reconsidered, jdk8 should follow. Most of the distros adapted the patch from 
8044235[1] anyway.

There are two approaches  possible:
  - add only nashorn [2] and maybe later (or during this review) also zipfs
  - backport (and adapt) whole 8044235 patch [3]

For [2], the src.zip grows by 1MB (from 23->24MB)
For [3], the src.zip grow double - 51MB (jdk9 have src.zip of 62MB)

Only SDK image is affected.

note - src.zip i used only for IDEs to make debugging easy, and when debugging jdk itself (when 
*your* application is misbehaving), you are stepping through all binaries, so it is not correct to 
hide the sources for something you can step inside anyway.


Skippable longstory:
Recently I was debugging nashorn in jdk8, and realized, that nashorn sources are missing in it, 
although they are present in jdk9.
Quick patch followed to build-dev: 
http://mail.openjdk.java.net/pipermail/build-dev/2017-January/018460.html
There I was made aware about https://bugs.openjdk.java.net/browse/JDK-8044235 and fact that it was 
long ago recommended for backport, but never done.
However, the backport is much more complex then simple "add nashorn sources": 
http://mail.openjdk.java.net/pipermail/build-dev/2017-January/018462.html

 From review later flown up taht it should be considered as enhancement instead of simple bugfix,but 
in al cases it should go over 8u-dev so here it is.

Please see the whole thread of "RFC: nashorn sources are missing in src.zip" on 
http://mail.openjdk.java.net/pipermail/build-dev/2017-January/thread.html#start for complex info.
As the result of this thread I copied the patches from fedorapeople.org to cr.openjdk.net.

Bes regards from CZ,
   J.

[1] https://bugs.openjdk.java.net/browse/JDK-8044235
[2] http://cr.openjdk.java.net/~jvanek/nashornMissingInSrcZip/v1/webrev/
[3] http://cr.openjdk.java.net/~jvanek/nashornMissingInSrcZip/v2/webrev/


More information about the jdk8u-dev mailing list