[jdk21u-dev] RFR: 8347300: Don't exclude the "PATH" var from the environment when running app launchers in jpackage tests

[Goetz Lindenmaier]

On Mon, 10 Nov 2025 13:43:32 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I backport this to get the jpackage testing up-to-date. This simplifies further backports.
> 
> This change, [JDK-8347300](https://bugs.openjdk.org/browse/JDK-8347300), is included in the second commit.
> 
> To make it work, I backport  changes from https://bugs.openjdk.org/browse/JDK-8340311 "8340311: JPackage app-image exe launches multiple exe's in JDK 22+" as prerequisite.
> Planned later backport 8365790 needs the adaptions of the test helper classes in 8340311, too.
> 
> **8340311 backport adaptions**
> 
> The 8340311 test helper changes patch almost cleanly to 21.
> But I also need to take some changes to tests where the interface
> to the helper classes changed.
> 
> Adaptions:
> 
> *    I had to modify the location of test/jdk/tools/jpackage/share/jdk/jpackage/tests/BasicTest.java which was moved by a later change already backported.
> *    I resolved the copyright in LauncherAsServiceVerifier.java
>  *   I omitted the changes to Win8301247Test.java. These mostly depend on previous, not backported changes and do not apply well.
> *    Naturally I omitted the new tests in 8340311.
> 
> **8347300 backport adaptions**
> 
> To apply [JDK-8347300](https://bugs.openjdk.org/browse/JDK-8347300) for the second commit, I had to resolve several chunks because removePath was renamed to removePathEnvVar.

ok, so no. I won't backport it.

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/2454#issuecomment-3520412671