You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Karen Miller <km...@pivotal.io> on 2016/03/19 00:20:25 UTC
source code build instruction improvements on the cwiki
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
Re: source code build instruction improvements on the cwiki
Posted by William Markito <wm...@pivotal.io>.
Thank you Karen, looks good!
On Fri, Mar 18, 2016 at 4:20 PM, Karen Miller <km...@pivotal.io> 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
>
--
~/William
Re: source code build instruction improvements on the cwiki
Posted by Karen Miller <km...@pivotal.io>.
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 <sb...@pivotal.io>
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 <km...@pivotal.io> 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 <ds...@pivotal.io> 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 <km...@pivotal.io>
> > 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
>
Re: source code build instruction improvements on the cwiki
Posted by Sai Boorlagadda <sb...@pivotal.io>.
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 <km...@pivotal.io> 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 <ds...@pivotal.io> 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 <km...@pivotal.io>
> 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
Re: source code build instruction improvements on the cwiki
Posted by Karen Miller <km...@pivotal.io>.
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 <ds...@pivotal.io> 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 <km...@pivotal.io> 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
> >
>
Re: source code build instruction improvements on the cwiki
Posted by Dan Smith <ds...@pivotal.io>.
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 <km...@pivotal.io> 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
>