RFR: 8347300: Don't exclude the "PATH" var from the environment when running app launchers in jpackage tests
Alexey Semenyuk
asemenyuk at openjdk.org
Wed Jan 8 23:29:22 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.
@sashamatveev PTAL
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22983#issuecomment-2578870413
More information about the core-libs-dev
mailing list