On Thu, 8 Oct 2026 12:21:23 GMT, Jaikiran Pai <[email protected]> wrote:
> 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). This pull request has now been integrated. Changeset: 6e6a595c Author: Jaikiran Pai <[email protected]> URL: https://git.openjdk.org/jdk/commit/6e6a595c600605d89bde41887abf16bc6956968c Stats: 14 lines in 2 files changed: 8 ins; 0 del; 6 mod 8393884: MiscTests.testExecutableJARNoManifest fails when -Djava.awt.headless=true is in test VM options Reviewed-by: lancea, azeller, mbaesken ------------- PR: https://git.openjdk.org/jdk/pull/33268
