Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/737#discussion_r39566345
--- Diff: DEVELOPER.md ---
@@ -222,6 +222,19 @@ To pull in a merge request you should generally follow
the command line instruct
# Build the code and run the tests
+## Prerequisites
+You need the `ruby` package manager `rvm` which can be installed using
`curl -L https://get.rvm.io | bash -s stable --autolibs=enabled && source
~/.profile` (see the [rvm installation
instructions](https://github.com/rvm/rvm) for details) and the `nodejs` package
manager `nvm` which can be installed using `wget -qO-
https://raw.githubusercontent.com/creationix/nvm/v0.26.1/install.sh | bash &&
source ~/.bashrc` (see the [nvm installation
instructions](https://github.com/creationix/nvm) for details).
--- End diff --
@krichter722
It's better to clarify that Storm actually requires Python / Ruby / Node,
not RVM, and NVM.
So we would better to explain below first,
```
You need the Python, Ruby, Node to build the codes.
Python doesn't have package manager so if you don't have python, you need
to install manually. (see the [python download
page](https://www.python.org/downloads/) for details)
```
and continue explaining RVM and NVM with clarifying it's for convenience,
not requirement. Users can still install Ruby and Node manually.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
