> On Feb. 4, 2015, 4:25 p.m., Amareshwari Sriramadasu wrote:
> > src/site/apt/gettingstarted/install-steps.apt, line 30
> > <https://reviews.apache.org/r/30608/diff/1/?file=847323#file847323line30>
> >
> >     Let us mention that download contains both source release and 
> > convienient binary distribution.
> >     
> >     Also, I'm thinking building from source release should be here itself.
> 
> Himanshu Gahlaut wrote:
>     If intended audience for source release is contributing developer, then 
> shall we have all source release information only in developer doc ?
> 
> Amareshwari Sriramadasu wrote:
>     Intended audience for source release is not only contributing developer, 
> but also all who wants to verify/consume source release. You can refer 
> http://www.apache.org/dev/release.html#what-must-every-release-contain. So, 
> my concern is putting it developer doc will be hard for other to how to build 
> it. So, we can mention the release contains both source and convienient 
> binary distribution and may be link building from source release here.

And i agree with Himanshu, install_and_run page is not the page to mention 
about how to build from source release. Not sure if we have to include them in 
a differnt place.

Srikanth Sundarrajan, thoughts?


- Amareshwari


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30608/#review70973
-----------------------------------------------------------


On Feb. 7, 2015, 6:50 a.m., Himanshu Gahlaut wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30608/
> -----------------------------------------------------------
> 
> (Updated Feb. 7, 2015, 6:50 a.m.)
> 
> 
> Review request for lens.
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> LENS-221: Install and Run Documentation improvement
> 
> 
> Diffs
> -----
> 
>   src/site/apt/admin/deployment.apt 57be5211f080c9d931afde5c7a3c90b5b34d25c5 
>   src/site/apt/developer/contribute.apt 
> 8015d68e2bd53043b27181533aabc6ca3a3c6664 
>   src/site/apt/gettingstarted/install-steps.apt 
> ef5724de1587c214eb241ee860d9c71ed59c27a8 
>   src/site/apt/gettingstarted/pseudo-distributed-setup.apt 
> f83aa72ec06a87fcc4e98ad8ba5d4e295e16dbff 
>   src/site/apt/gettingstarted/quick-start.apt 
> ec97d073a23f603c2e34f0d46c43aa73fce2f69c 
>   src/site/apt/lenshome/pseudo-distributed-setup.apt PRE-CREATION 
>   src/site/site.xml f0f0a55d4e67e52eb06bdb62f1e8bc4c72b4e6e8 
> 
> Diff: https://reviews.apache.org/r/30608/diff/
> 
> 
> Testing
> -------
> 
> Ran site on local machine to test contents.
> 
> 
> Thanks,
> 
> Himanshu Gahlaut
> 
>

Reply via email to