[GitHub] [arrow-datafusion] andygrove commented on a diff in pull request #2419: docs: Update the Ballista dev env instructions

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

andygrove commented on code in PR #2419:
URL: https://github.com/apache/arrow-datafusion/pull/2419#discussion_r863081253


##########
ballista/docs/dev-env.md:
##########
@@ -23,17 +23,29 @@ You will need a standard Rust development environment. The easiest way to achiev
 
 ## Install OpenSSL
 
-Follow instructions for [setting up OpenSSL](https://docs.rs/openssl/0.10.28/openssl/). For Ubuntu users, the following
+Follow instructions for [setting up OpenSSL](https://docs.rs/openssl/latest/openssl/). For Ubuntu users, the following
 command works.
 
 ```bash
 sudo apt-get install pkg-config libssl-dev
 ```
 
+For MacOS users, the following command works.
+
+```bash
+brew install openssl@1.1
+```
+
 ## Install CMake
 
 You'll need cmake in order to compile some of ballista's dependencies. Ubuntu users can use the following command:
 
 ```bash
 sudo apt-get install cmake
 ```
+
+MacOS users cau use the follow command:

Review Comment:
   ```suggestion
   MacOS users can use the following command:
   ```



-- 
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: github-unsubscribe@arrow.apache.org

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