-
Notifications
You must be signed in to change notification settings - Fork 728
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
Add build instructions for OpenJDK8 for macOS #3772
Conversation
@babsingh - please can you check the instructions to make sure I didn't miss anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, requesting @jdekonin for a secondary review or quick proof-read.
Based on content in eclipse-openj9#3323, added a section for macOS. [ci skip] Signed-off-by: Sue Chaplain <[email protected]>
e35fc56
to
4a76a3a
Compare
@babsingh - finally got around to making the changes from your review. Please can you recheck. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm.
@DanHeidinga - is this good to merge please? |
@pshipton @DanHeidinga |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Merged. Thanks for the reminder @SueChaplain! |
Based on content in #3323, added a section
for macOS.
[ci skip]
Signed-off-by: Sue Chaplain [email protected]