Re: RFR: 8282219: jdk/java/lang/ProcessBuilder/Basic.java fails on AIX [v5]

2022-02-25 Thread Roger Riggs
On Fri, 25 Feb 2022 17:45:39 GMT, Ichiroh Takiguchi  
wrote:

>> Run jtreg:jdk/java/lang/ProcessBuilder/Basic.java on AIX.
>> The test was failed by:
>>   Incorrect handling of envstrings containing NULs
>> 
>> According to my investigation, this issue was happened after following 
>> change was applied.
>> JDK-8272600: (test) Use native "sleep" in Basic.java
>> 
>> test.nativepath value was added into AIX's LIBPATH during running this 
>> testcase.
>> On AIX, test.nativepath value should be removed from LIBPATH value before 
>> comparing the values.
>
> Ichiroh Takiguchi has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Pass LIBPATH setting for AIX

Marked as reviewed by rriggs (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/7574


Re: RFR: 8282219: jdk/java/lang/ProcessBuilder/Basic.java fails on AIX [v5]

2022-02-25 Thread Ichiroh Takiguchi
> Run jtreg:jdk/java/lang/ProcessBuilder/Basic.java on AIX.
> The test was failed by:
>   Incorrect handling of envstrings containing NULs
> 
> According to my investigation, this issue was happened after following change 
> was applied.
> JDK-8272600: (test) Use native "sleep" in Basic.java
> 
> test.nativepath value was added into AIX's LIBPATH during running this 
> testcase.
> On AIX, test.nativepath value should be removed from LIBPATH value before 
> comparing the values.

Ichiroh Takiguchi has updated the pull request incrementally with one 
additional commit since the last revision:

  Pass LIBPATH setting for AIX

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7574/files
  - new: https://git.openjdk.java.net/jdk/pull/7574/files/b6d6afe7..60f8e96e

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=7574=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=7574=03-04

  Stats: 27 lines in 1 file changed: 8 ins; 17 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7574.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7574/head:pull/7574

PR: https://git.openjdk.java.net/jdk/pull/7574