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

Bug #90: Make the 'find java' task recognize Homebrew installed jdk on macOS #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

erikbra
Copy link

@erikbra erikbra commented Nov 17, 2024

Bug #90: Make the 'find java' task recognize Homebrew installed jdk on macOS

The JDK installed using Homebrew on macOS has an extra 'Homebrew ' string at the end of
the 'OpenJDK Runtime Environment' string, before the build number. This was not recognized,
and the build task said that it couldn't find Java.

Example output from JDK installed via Homebrew om macOS:

openjdk version "23.0.1" 2024-10-15
OpenJDK Runtime Environment Homebrew (build 23.0.1)
OpenJDK 64-Bit Server VM Homebrew (build 23.0.1, mixed mode, sharing)

Other chores (for devX):

  • Changed hard-coded #!/usr/bin/bash in some of the .sh scripts to #!/usr/bin/env bash, so that they
    work on systems where bash is not found at /usr/bin/bash (as e.g. on macOS, where it is in /bin/bash)
  • Set executable bit on the .sh scripts, to be able to run them just like ./test.sh

… jdk on macOS

The JDK installed using Homebrew on macOS has an extra 'Homebrew ' string at the end of
the 'OpenJDK Runtime Environment' string, before the build number. This was not recognized,
and the build task said that it couldn't find Java.

Example output from JDK installed via Homebrew om macOS:

openjdk version "23.0.1" 2024-10-15
OpenJDK Runtime Environment Homebrew (build 23.0.1)
OpenJDK 64-Bit Server VM Homebrew (build 23.0.1, mixed mode, sharing)

Other chores (for devX):

* Changed hard-coded #!/usr/bin/bash in some of the .sh scripts to #!/usr/bin/env bash, so that they
work on systems where bash is not found at /usr/bin/bash (as e.g. on macOS, where it is in /bin/bash)
* Set executable bit on the .sh scripts, to be able to run them just like `./test.sh`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant