On Wed, 8 Jan 2025 22:29:04 GMT, Alexey Semenyuk <[email protected]> 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 <[email protected]> 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
