On Tue, 4 Apr 2023 01:49:09 GMT, Tyler Steele <tste...@openjdk.org> wrote:

>> Minor changes to the build system to recognize and warn that an incompatible 
>> toolchain is detected.
>
> Tyler Steele has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Change min xlx FP level to 11

LGTM.

However I am not sure about the number extraction itself on AIX, maybe you 
could check this please ?


#!/bin/sh

COMPILER_VERSION_OUTPUT=`xlC -qversion 2>&1`

COMPILER_VERSION_NUMBER=`echo $COMPILER_VERSION_OUTPUT | sed -e 's/^.*Version: 
(@<:@1-9@:>@@<:@0-9.@:>@*).*$/\1/'`

echo $COMPILER_VERSION_OUTPUT
echo $COMPILER_VERSION_NUMBER


gives me
IBM XL C/C++ for AIX, V16.1.0 (5725-C72, 5765-J12) Version: 16.01.0000.0013
IBM XL C/C++ for AIX, V16.1.0 (5725-C72, 5765-J12) Version: 16.01.0000.0013

So I wonder does the version number extraction really work on AIX ?

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

Marked as reviewed by mbaesken (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13086#pullrequestreview-1395173662

Reply via email to