[
https://issues.apache.org/jira/browse/GEODE-874?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15207091#comment-15207091
]
ASF GitHub Bot commented on GEODE-874:
--------------------------------------
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.
> v1.0.0-incubating.M1.RC2: Text file errors in the source distribution
> ---------------------------------------------------------------------
>
> Key: GEODE-874
> URL: https://issues.apache.org/jira/browse/GEODE-874
> Project: Geode
> Issue Type: Bug
> Components: docs
> Reporter: Kenneth Howe
> Assignee: Karen Smoler Miller
> Fix For: 1.0.0-incubating.M2
>
>
> gradlew script is not included in the Geode release packages, but it is
> called out in command examples in the following text files in the release
> source distribution:
> COMPILING.txt
> RUNNING.txt
> docker/Dockerfile
> gemfire-spark-connector/doc/1_building.md
> The easy fix is to change "gradlew" to "gradle" in affected files and include
> a requirement for the user to install gradle from gradle.org.
> There was a similar issue with README.md that was fixed for RC2.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
