Karen, Looks good to me.
Just a suggestion to continue with the flow of instructions, Do we need to point user to the gfsh available in the bin folder of the extracted binary distribution? As we instructed about extracting the binary and add the bin directory to the path. Sai On Tue, Apr 5, 2016 at 1:42 PM, Karen Miller <[email protected]> wrote: > I have again updated the build instructions, incorporating Dan's > suggestions for > for improvement. > > Note that while there is a gradlew.bat script (for a build on Windows) in > the develop branch > of the repository, there is *no* gradlew.bat script included with an Apache > Geode > (incubating) release of the source code. This means that the build > instructions differ > based on where the source is acquired from. These instructions assume > acquisition of the > repository from the develop branch, as I think should be the case for a > Geode developer. > > Please take a look and suggest further feedback. Karen > > > On Fri, Mar 18, 2016 at 4:27 PM, Dan Smith <[email protected]> wrote: > > > build depends on installDist so I think "build installDist" is redundant. > > It should be just > > ./gradlew build > > or > > ./gradlew build -Dskip.tests=true > > > > > > There is a gradlew.bat script for windows if you actually clone the > source. > > Should we tell people that only need to install gradle if they download > the > > source distribution? > > > > -Dan > > > > On Fri, Mar 18, 2016 at 4:20 PM, Karen Miller <[email protected]> > wrote: > > > > > Check out improvements to the cwiki page on building Geode from source > at > > > > > > > > > https://cwiki.apache.org/confluence/display/GEODE/Building+and+Running+Geode+from+Source > > > > > > I've also refined the Geode In 5 Minutes guide found at > > > https://cwiki.apache.org/confluence/display/GEODE/Index > > > > > > Please consider and provide feedback. Karen > > > > > > -- Sai Boorlagadda
