Please support --with-gtest=system

Bug #2012316 reported by Vladimir Petko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openjdk-17 (Ubuntu)
Fix Released
Undecided
Unassigned
openjdk-20 (Ubuntu)
Fix Released
Undecided
Unassigned
openjdk-21 (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

OpenJDK 17 contains hotspot tests using google test which can be configured with --with-gtest option. Some of those tests are part of tier1 (basic functionality tests).

Implement `--with-gtest=system` option which will allow to use distro's google test package.

Related branches

Vladimir Petko (vpa1977)
tags: added: fr-3686
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openjdk-20 - 20+36~us1-0ubuntu1

---------------
openjdk-20 (20+36~us1-0ubuntu1) lunar; urgency=medium

  * OpenJDK 20 release, build 36. See:
    https://mail.openjdk.org/pipermail/jdk-dev/2023-March/007517.html
  * d/{control, compat}: update debhelper-compat version, add jtreg7
    dependencies, regenerate control.
  * d/t/{jdk,hotspot,jaxp,lantools}: run tier1 and tier2 jtreg tests only,
    add test options from OpenJDK makefile.
  * d/t/*: fix test environment: add missing -nativepath (LP: #2001563).
  * d/t/jdk: provide dbus session for the window manager (LP: #2001576).
  * d/p/build_gtest.patch: build OpenJDK with the system googletest
    (LP: #2012316).
  * d/p/*: add patches for jtreg tests:
    - disable-thumb-assertion.patch: fix JDK-8305481.
    - update-assertion-for-armhf.patch: fix JDK-8305480.
    - misalign-pointer-for-armhf.patch: packaging-specific patch to fix test
      failure introduced by d/p/m68k-support.diff.
    - log-generated-classes-test.patch: workaround JDK-8166162.
    - update-permission-test.patch: add security permissions for testng 7.
    - ldap-timeout-test-use-ip.patch, test-use-ip-address.patch: Ubuntu-specific
      patches to workaround missing DNS resolver on the build machines.
    - exclude_broken_tests.patch: quarantine failing tests.
  * d/rules: package external debug symbols (LP: #2015835).

 -- Vladimir Petko <email address hidden> Mon, 03 Apr 2023 11:27:49 +1200

Changed in openjdk-20 (Ubuntu):
status: New → Fix Released
Revision history for this message
Vladimir Petko (vpa1977) wrote :

Validation:

Refer to the build logs[1]

Passed: gtest/LargePageGtests.java#use-large-pages
Passed: gtest/LargePageGtests.java#use-large-pages-1G
Passed: gtest/LargePageGtests.java#use-large-pages-sysV
Passed: gtest/MetaspaceGtests.java#balanced-no-ccs
Passed: gtest/MetaspaceGtests.java#reclaim-aggressive-ndebug
Passed: gtest/MetaspaceGtests.java#reclaim-none-ndebug
Passed: gtest/GTestWrapper.java

Gtest JTREG tests pass.

[1] https://launchpad.net/ubuntu/+source/openjdk-17/17.0.6+10-1ubuntu3

tags: added: verification-done-lunar
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openjdk-17 - 17.0.6+10-1ubuntu3

---------------
openjdk-17 (17.0.6+10-1ubuntu3) lunar; urgency=medium

  * d/control: add jtreg6 dependencies, regenerate control.
  * d/t/{jdk,hotspot,jaxp,lantools}: run tier1 and tier2 jtreg tests only,
    add test options from OpenJDK makefile.
  * d/t/*: fix test environment: add missing -nativepath (LP: #2001563).
  * d/t/jdk: provide dbus session for the window manager (LP: #2001576).
  * d/p/build_gtest.patch: build OpenJDK with the system googletest
    (LP: #2012316).
  * d/p/*: add patches for jtreg tests:
    - disable-thumb-assertion.patch: fix JDK-8305481.
    - update-assertion-for-armhf.patch: fix JDK-8305480.
    - misalign-pointer-for-armhf.patch: packaging-specific patch to fix test
      failure introduced by d/p/m68k-support.diff.
    - log-generated-classes-test.patch: workaround JDK-8166162.
    - update-permission-test.patch: add security permissions for testng 7.
    - ldap-timeout-test-use-ip.patch, test-use-ip-address.patch: Ubuntu-specific
      patches to workaround missing DNS resolver on the build machines.
    - exclude_broken_tests.patch: quarantine failing tests.
  * d/rules: package external debug symbols (LP: #2015835).
  * drop d/p/{jaw-classpath.diff, jaw-optional.diff}: the atk wrapper is disabled
    and these patches cause class data sharing tests to fail (LP: #2016194).
  * d/p/exclude-broken-tests.patch: add OpenJDK 17 failures.
  * d/t/jtreg-autopkgtest.in: pass JTREG home to locate junit.jar, regenerate
    d/t/jtreg-autopkgtest.sh (LP: #2016206).

 -- Vladimir Petko <email address hidden> Mon, 27 Mar 2023 11:41:46 +1300

Changed in openjdk-17 (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openjdk-21 - 21~28ea-1

---------------
openjdk-21 (21~28ea-1) unstable; urgency=medium

  * OpenJDK 21 snapshot, Build 28.
  * Add support for alpha, m68k, ppc, sh, x32.

 -- Matthias Klose <email address hidden> Mon, 26 Jun 2023 07:48:49 +0200

Changed in openjdk-21 (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.