[GitHub] [pulsar] asafm commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

[GitBox <gi...@apache.org>]

asafm commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973678845


##########
README.md:
##########
@@ -138,6 +138,47 @@ Requirements:
 > Note: this project includes a [Maven Wrapper](https://maven.apache.org/wrapper/) that can be used instead of a system installed Maven.
 > Use it by replacing `mvn` by `./mvnw` on Linux and `mvnw.cmd` on Windows in the commands below.
 
+### Install JDK on M1

Review Comment:
   This issue happened to me and 5 other engineers I know of just as an example. 
   Our approach, especially on getting started should be giving you everything you need to get started - in the README case, it's building the product. M1 is notorious to have many quirks. Anything we can do to help will be greatly appreciated.
   
   Just mentioning a few people recently running into those issues @geomagilles @teabot @nahguam just to name a few
   
   Bear in mind, that they can ignore this recommendation. Most newcomers will appreciate it as much as I did.
   
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org