With Sai's suggestion, the instructions are even better.  It should be more
clear that the gfsh
script is available from 2 distinct locations after building from source,
and that the developer
does not need to install from the distribution.

On Thu, Apr 7, 2016 at 11:09 AM, Sai Boorlagadda <[email protected]>
wrote:

> 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
>

Reply via email to