Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[master] Fixes version parsing for WARNING situations #442

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

Karm
Copy link
Collaborator

@Karm Karm commented Aug 19, 2024

fixes graalvm/mandrel#766
fixes #429

From my test run 24.1 branch aarch64:

...
DEBUG [OperatingSystem] Execute [/home/tester/karm/mx/mx, graalvm-version] in /home/tester/karm/graal/substratevm with environment variables [MX_PYTHON=python]
INFO [build] mx's graalvm-version output: WARNING: No platform-specific definition is available for distribution MUSL_CMAKE_TOOLCHAIN for your architecture (aarch64)
INFO [build] mx's graalvm-version output: 24.1.0-dev
...
[1/8] Initializing...                                                                                    (3.7s @ 0.16GB)
 Java version: 23-beta+36-ea, vendor version: Mandrel-24.1.0-devc014e6a1067
 Graal compiler: optimization level: 2, target machine: compatibility
 C compiler: gcc (redhat, aarch64, 11.4.1)
...
INFO [build] Congratulations you successfully built Mandrel 24.1.0-devc014e6a1067 based on Java 23-beta+36-ea
...

Note the WARNING is logged as INFO, so it should be always visible.

@Karm Karm requested a review from jerboaa August 19, 2024 14:48
@Karm Karm self-assigned this Aug 19, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 19, 2024
build.java Show resolved Hide resolved
@Karm Karm merged commit 665c887 into graalvm:master Aug 19, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
2 participants