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

Alexander Matveev almatvee at openjdk.org
Thu Jan 9 00:28:48 UTC 2025


On Wed, 8 Jan 2025 23:29:22 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.
>
> Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix copyright year

Looks good.

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

Marked as reviewed by almatvee (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22983#pullrequestreview-2538523767


More information about the core-libs-dev mailing list