On Thu, 16 Mar 2023 18:55:34 GMT, Tyler Steele <tste...@openjdk.org> wrote:

> The GNU-utils df command supports a flag `-l` which displays information 
> about local disks only. The AIX df equivalent is `-T local`. However, my 
> build systems uses GNU-utils on AIX. In this case, the build system uses `df 
> -T local`, and incorrectly determines that the disk is non-local from the 
> error code.
> 
> This change corrects that issue by testing if the enabled version of df 
> returns cleanly with the `-T local` flag when AIX is the detected OS. If it 
> does, `-T local` is used, and if not, it defaults to `-l`.
> 
> @erikj79 Thanks for your review on #13065. I pushed to the wrong branch by 
> mistake, and it seems I must now re-create the PR to fix the issue.

This pull request has now been integrated.

Changeset: 4c8c9935
Author:    Tyler Steele <tste...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/4c8c9935eb23bfbabf311ad2e27498227f4ee932
Stats:     5 lines in 1 file changed: 4 ins; 0 del; 1 mod

8304364: [AIX] Build erroneously determines build disk is non-local when using 
GNU-utils df on AIX

Reviewed-by: erikj

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

PR: https://git.openjdk.org/jdk/pull/13066

Reply via email to