> Can I please get a review of this change which removes the `java.compiler` > system property? This addresses https://bugs.openjdk.org/browse/JDK-8041676. > > A CSR has been filed for this change and is available at > https://bugs.openjdk.org/browse/JDK-8305998. The CSR has more details about > this proposed change, but to summarize, the `java.compiler` system property > wasn't used in any practical way. With the recent removal of > `java.lang.Compiler` interface from the JDK, the presence of this system > property wasn't of any practical importance. > > The commit in this PR, removes its reference from the > `System.getProperties()` documentation and also removes the specific > implementation from hotspot where it treated the values `NONE` and empty > string in the absence of `-Xdebug` in a specific manner and considered it to > be an instruction to run the application in interpreter-only mode. `-Xint` > option has been around for this purpose, so removal of this `java.compiler` > system property support from hotspot implementation wouldn't remove any > usable feature for the applications. > > Additionally, the hotspot implementation now logs a warning message when it > detects the presence of `java.compiler` system property when `java` is > launched. This warning message will be removed after a few releases. > > The current JDK source has reference to the `java.compiler` system property > in numerous NetBeans project build files. The usage of this system property > in such files isn't necessary. However, this PR doesn't intend to cleanup > those references, since it isn't clear which of those NetBeans projects are > still relevant. I think we can use a separate PR to do that cleanup. > > tier1, tier2, tier3 testing has been done with these changes and those tests > have passed.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: Roger's suggestion - improve warning message ------------- Changes: - all: https://git.openjdk.org/jdk/pull/13475/files - new: https://git.openjdk.org/jdk/pull/13475/files/3a4bf8f7..5c1846e1 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=13475&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=13475&range=00-01 Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/13475.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/13475/head:pull/13475 PR: https://git.openjdk.org/jdk/pull/13475
