Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8245943

exploded-test-gtest uses images/ binaries

    Details

    • Type: Bug
    • Status: New
    • Priority: P4
    • Resolution: Unresolved
    • Affects Version/s: 11, 15, 16
    • Fix Version/s: tbd
    • Component/s: hotspot
    • Labels:
    • Subcomponent:

      Description

      When building and testing hotspot you can choose if you want to run a fully packaged JDK or the faster-to-build exploded JDK.

      make images => builds the fully packaged JDK
      make test => uses the JDK from make images

      The tests are using the <target>/images/jdk/ path.

      make jdk => builds the exploded JDK
      make exploded-test => uses and tests the exploded JDK

      The tests are then using the exploded <target>/jdk/ path.

      This means that I can take a test command line generated by exploded-test (with <target>/jdk paths) and use that for testing / debugging in a debugger. And if I want to update the JDK source, I only have to run make jdk, and can immediately use the mentioned test command line and it will use the up-to-date binaries.

      Now, if you run make exploded-test-gtest it copies the gtestLauncher to the <target>/images/test directory. The command line looks like this:

      <target>/images/test/hotspot/gtest/server/gtestLauncher -jdk <target>/jdk --gtest_output=xml:<target>/test-results/gtest_all_server/gtest.xml

      However, if I update the HotSpot source, run make jdk (which builds the gtest binaries) and rerun the printed test command line, it will run the old binaries, not the ones created by make jdk. make jdk updates the binaries in:
      <target>/hotspot/variant-server/libjvm/gtest/gtestLauncher

      Could we change so that exploded-test-gtest uses the binaries created by make jdk?

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              stefank Stefan Karlsson
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: