Can I please get a review of this change which proposes to address the issue 
noted in https://bugs.openjdk.org/browse/JDK-8393884?

When `java` is launched using `java -jar <jarfile>`, the native C code in the 
launcher, parses the manifest of the JAR file to detect the presence of any 
`SplashScreen-Image` manifest attribute. If the manifest file is missing in the 
JAR file, then it throws an error and exits the launcher. 
https://bugs.openjdk.org/browse/JDK-8392966 recently introduced a test which 
verifies the error message text in the absence of a manifest file in the JAR.

When the `java.awt.headless` system property is set to `true`, then the 
launcher code skips the code meant to lookup the manifest file from the JAR and 
parse the `SplashScreen-Image` manifest attribute. Instead, at a later point in 
the code path, the Java side of the launcher tries to lookup the manifest file. 
This Java code throws a different error message when the manifest is missing 
from the JAR file. As a result, the newly introduced test now fails due to the 
unexpected error message.

The change in this PR now updates the other error message to match the one we 
had introduced in JDK-8392966. As a result, even when the `java.awt.headless` 
system property is set to `true`, the error message will now be the same as the 
one when that property isn't set. The test now passes with this change.

tier testing is currently in progress in our CI.


---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

-------------

Commit messages:
 - 8393884: MiscTests.testExecutableJARNoManifest fails when 
-Djava.awt.headless=true is in test VM options

Changes: https://git.openjdk.org/jdk/pull/33268/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=33268&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8393884
  Stats: 6 lines in 2 files changed: 4 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/33268.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/33268/head:pull/33268

PR: https://git.openjdk.org/jdk/pull/33268

Reply via email to