On Tue, 22 Feb 2022 22:04:38 GMT, Tyler Steele <d...@openjdk.java.net> wrote:
>> This test had two failing sections on AIX related to an incorrect expected >> value for LIBPATH. The two (previously failing) test sections are below. >> >> - Test Runtime.exec(...envp...) with envstrings with initial `=' >> - Test Runtime.exec(...envp...) with envstrings containing NULs >> >> This PR modifies the environment passed to the process at ...exec(cmdp, >> **envp**) to include the LIBPATH of the parent. With this change, the >> expected libpath matches the libpath returned by the process. >> >> ### Alternatives >> >> An equivalent change would be to modify the libpath variable used to set the >> expected value for the test without explicitly setting the LIBPATH in >> process invocation. This would involve removing the libpath for >> .../jtreg/native that is added by the test runner by command-line option >> `-J-Dtest.nativepath=...images/test/jdk/jtreg/native `. This change would be >> reasonable, but I prefer the approach taken in this PR. >> >> ### Testing >> >> This test now passes on my test machine running AIX 7.1. > > Tyler Steele has updated the pull request incrementally with one additional > commit since the last revision: > > Updates copyright year & adds bugid ok, I closed JDK-8282239 as a duplicate. This PR can be closed. Comments appreciated on https://github.com/openjdk/jdk/pull/7574. ------------- PR: https://git.openjdk.java.net/jdk/pull/7581
