RFR: 8282239 [testbug, AIX] ProcessBuilder/Basic.java fails with incorrect LIBPATH [v2]

Roger Riggs rriggs at openjdk.java.net
Wed Feb 23 14:31:46 UTC 2022


On Tue, 22 Feb 2022 22:04:38 GMT, Tyler Steele <duke at 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


More information about the core-libs-dev mailing list