Github user karensmolermiller commented on the pull request:
https://github.com/apache/incubator-geode/pull/114#issuecomment-199974130
I've revised the BUILDING.md file such that the redundant installDist
command line option is no longer included.
I have not changed the Windows build instructions to suggest using the
gradlew.bat script. The instructions modified in this PR are those that should
be followed by someone who downloads source from the Releases page of the
http://geode.incubator.apache.org/ site. The source from that location does not
include a Windows gradlew.bat script, per Apache rules. I suggest that
instructions for building from source when acquiring source from GitHub belong
on the cwiki; the gradlew.bat script is still included in the
apache/incubator-geode repository. For the longer term, I further suggest that
the Windows gradlew.bat script be rewritten such that it can be included in the
source of a release. Then, there can be a single set of build instructions,
and the build instructions become independent of the location of the source
download.
---
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.
---
