GraalVM Version parser is too strict

[brunocaballero]

Describe the bug

When building a Native Image using GraalVM, Quarkus parses the version of native-image on the following class:

https://github.com/quarkusio/quarkus/blob/main/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/GraalVM.java

The check fails for some custom Runtimes we are developing.

In particular Quarkus expects the runtime to be OpenJDK or GraalVM, and the vm name OpenJDK 64-Bit Server or Substrate

private static final String RUNTIME_NAME = "(?<RUNTIME>(?:OpenJDK|GraalVM) Runtime Environment) ";
private static final String VM_NAME = "(?<VM>(?:OpenJDK 64-Bit Server|Substrate) VM) ";

Expected behavior

I would expect to accept other runtimes.

Maybe something like:

private static final String RUNTIME_NAME = "(?<RUNTIME>.*) Runtime Environment) ";
private static final String VM_NAME = "(?<VM>.*) VM) ";

Actual behavior

Other runtimes are currently not supported by recent version of Quarkus.

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

Mandrel or GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

[quarkus-bot]

/cc @Karm (mandrel), @galderz (mandrel), @zakkak (mandrel)

[zakkak]

Hi @brunocaballero, that makes sense.

Could you please open a PR with the necessary changes, extending the tests in https://github.com/quarkusio/quarkus/blob/main/core/deployment/src/test/java/io/quarkus/deployment/pkg/steps/GraalVMTest.java to cover your case as well?

cc @jerboaa

[jerboaa]

I think that one got fixed with #37651?

[brunocaballero]

yes, fixed in #37651