Integrated: 8347300: Don't exclude the "PATH" var from the environment when running app launchers in jpackage tests
Alexey Semenyuk
asemenyuk at openjdk.org
Fri Jan 10 19:33:50 UTC 2025
On Wed, 8 Jan 2025 22:29:04 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
> - Don't remove the `PATH` variable from the environment when running app launchers in jpackage tests;
> - Fix tests that use `Executor` rather than `HelloApp` class to run app launchers: apply the same changes to the environment as would be applied by `HelloApp` class.
> - Replace `Executor.setRemovePathEnvVar()` with `Executor.removeEnvVar()` to support removal of any variable from the environment of a child process executed in jpackage tests;
> - Add support of a new boolean property `jpackage.test.clear-app-launcher-java-env-vars` in jpackage tests. If it is set to "true", the `JAVA_TOOL_OPTIONS` and `_JAVA_OPTIONS` variables will be removed from the environment of app launchers executed in jpackage tests. It is handy in local testing using jpackage test runner instead of the jtreg.
>
> The first and the second changes are unrelated to other changes. They are bundled together as they apply to the same source code.
This pull request has now been integrated.
Changeset: d69463e4
Author: Alexey Semenyuk <asemenyuk at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/d69463e4bcbddd346b9486059c5ad3a1cb555632
Stats: 38 lines in 4 files changed: 19 ins; 3 del; 16 mod
8347300: Don't exclude the "PATH" var from the environment when running app launchers in jpackage tests
Reviewed-by: almatvee
-------------
PR: https://git.openjdk.org/jdk/pull/22983
More information about the core-libs-dev
mailing list