Skip to content

Commit

Permalink
8338108: Give better error message in configure if a full XCode is mi…
Browse files Browse the repository at this point in the history
…ssing

Reviewed-by: jwaters, erikj, shade
  • Loading branch information
magicus committed Aug 12, 2024
1 parent 61d1dc5 commit a36fb36
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions make/autoconf/toolchain.m4
Original file line number Diff line number Diff line change
Expand Up @@ -678,6 +678,9 @@ AC_DEFUN_ONCE([TOOLCHAIN_DETECT_TOOLCHAIN_EXTRA],
test_metal=`$METAL --version 2>&1`
if test $? -ne 0; then
AC_MSG_RESULT([no])
AC_MSG_NOTICE([A full XCode is required to build the JDK (not only command line tools)])
AC_MSG_NOTICE([If you have XCode installed, you might need to reset the Xcode active developer directory])
AC_MSG_NOTICE([using 'sudo xcode-select -r'])
AC_MSG_ERROR([XCode tool 'metal' neither found in path nor with xcrun])
else
AC_MSG_RESULT([yes, will be using '$METAL'])
Expand Down

1 comment on commit a36fb36

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.