You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by ke...@deenlo.com on 2015/02/03 01:40:23 UTC

Re: Review Request 30236: Reorganize README


> On Jan. 27, 2015, 8:04 p.m., Christopher Tubbs wrote:
> > BUILD.md, lines 35-71
> > <https://reviews.apache.org/r/30236/diff/5/?file=836993#file836993line35>
> >
> >     I'm not sure this section is even necessary here. It seems more appropriate to developer documentation on the website.
> 
> kturner wrote:
>     Its useful information, so it needs to go somewhere.  Build instructions for 1.4, 1.5 and 1.6 are very different.  Its nice when dealing with those older versions to be able to look at their build instructions in the README.  I am not sure if version specific build instructions would be properly maintained on the website.   IMO keeping these instructions versioned with source is perferable to putting them on the website.  If we do keep these instructions in source, I think BUILD.md is an ok place.
> 
> Christopher Tubbs wrote:
>     We have to consider the target audience. For build instructions for the target audience of "consumers of the source tarball" (the official release), I think the BUILD.md file makes sense. However, these instructions I've highlighted do not target that audience (as stated explicitly in the paragraph under the `Iterative Experimentation` header). I think these instructions would confuse the primary target audience and should not be in the BUILD.md file. It is useful, though, and we need to put it somewhere... I just think that somewhere should be in a developer section on the website, rather in the BUILD.md file targeted towards non-developers.
> 
> kturner wrote:
>     I dropped the instructions and moved contents of BUILD.md to README.md

Forgot to mention :  Had an offline discussion w/ Christopher about this.  One of the things we disccused was that complexity of these instructions vs `mvn package -Passemble; tar ...`   which basically does the same thing.


- kturner


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


On Jan. 31, 2015, 1:38 a.m., kturner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30236/
> -----------------------------------------------------------
> 
> (Updated Jan. 31, 2015, 1:38 a.m.)
> 
> 
> Review request for accumulo.
> 
> 
> Bugs: ACCUMULO-1515
>     https://issues.apache.org/jira/browse/ACCUMULO-1515
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> Reorganized information in README and converted to markdown.  
> 
> At this point I like the INSTALL.md document, but do not really like the content of the README.md ATM.  Putting this up for review to get suggestions.
> 
> See how the markdown looks on GH : https://github.com/keith-turner/accumulo/tree/ACCUMULO-1515
> 
> 
> Diffs
> -----
> 
>   INSTALL.md PRE-CREATION 
>   NOTICE af212c2 
>   README 4ebb078 
>   README.md PRE-CREATION 
>   TESTING cf2afba 
>   TESTING.md PRE-CREATION 
>   assemble/src/main/assemblies/component.xml 3f18da3 
> 
> Diff: https://reviews.apache.org/r/30236/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> kturner
> 
>