JDK-8245610 : remove in-tree copy on gtest
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: test
  • Affected Version: 15
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2020-05-22
  • Updated: 2024-09-30
  • Resolved: 2020-05-26
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 15
15 b25Fixed
Related Reports
Relates :  
Description
w/ JEP 381 (JDK-8241787 / JDK-8244224) being integrated, all compilers used by OpenJDK became supported by gtest out-of-box, so there is no need to have our copy of gtest framework (including gmock) within OpenJDK source tree. instead, it can be treated like any other external dependencies, and a pointer to the directory w/ gtest code can be passed via configure options.

  
Comments
https://mail.openjdk.java.net/pipermail/hotspot-dev/2020-May/041936.html
26-05-2020

URL: https://hg.openjdk.java.net/jdk/jdk/rev/21c94b6f23e7 User: iignatyev Date: 2020-05-26 20:34:01 +0000
26-05-2020

I figure I will post it here as well for future references: this patch does *not* change the version of gtest we use in hotspot, that's to say if you want to build/run gtest tests, the source of gtest-1.8.1 should be provided to --with-gtest (unless you use jib which would download and pass source directory --with-gtest). one can download the source bundle from [1] or clone googletest, checkout release-1.8.1[2]. [1] https://github.com/google/googletest/releases/tag/release-1.8.1 [2] git clone https://github.com/google/googletest git checkout release-1.8.1
22-05-2020

https://mail.openjdk.java.net/pipermail/hotspot-dev/2020-May/041900.html
22-05-2020

build changes: http://cr.openjdk.java.net/~iignatyev/8245610/webrev.00/ + removal of test/fmw/gtest: http://cr.openjdk.java.net/~iignatyev/8245610/webrev.00+removal
22-05-2020