> On Feb. 28, 2013, 4:17 p.m., Travis Crawford wrote: > > http://svn.apache.org/repos/asf/incubator/hcatalog/trunk/src/docs/src/documentation/content/xdocs/install.xml, > > line 60 > > <https://reviews.apache.org/r/9641/diff/1/?file=262977#file262977line60> > > > > Forrest is no longer required when building a release package. Can you > > remove this section too? > > Mohammad Islam wrote: > is it true for 0.5.0 too? >
Yes, this is true for 0.5.0-incubating. Forrest is only required when building the website, not HCatalog itself. Unless you're making a website change and want to preview/publish, you don't need forrest. - Travis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9641/#review17193 ----------------------------------------------------------- On March 7, 2013, 7:09 a.m., Mohammad Islam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9641/ > ----------------------------------------------------------- > > (Updated March 7, 2013, 7:09 a.m.) > > > Review request for hcatalog and Ashutosh Chauhan. > > > Description > ------- > > Cut-pasted from JIRA: > Source : > http://incubator.apache.org/hcatalog/docs/r0.5.0/install.html#Server+Installation+from+Source > Need to update these: > 1. ant command specify the target "tar" which is no longer a valid target. > Can be replaced with "package" instead. > 2. parameter "-Dhcatalog.version=0.5.0" needs to be replaced by > "-Dhcatalog.version=0.5.0-incubating". > 3. In one place, it refers to hive 0.10 and in another place, it refers to > 0.9 as an example. Making them consistent through out the doc will be less > confusing. For example, make it 0.20.0. > > > This addresses bug HCATALOG-625. > https://issues.apache.org/jira/browse/HCATALOG-625 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/incubator/hcatalog/trunk/src/docs/src/documentation/content/xdocs/install.xml > 1450533 > > Diff: https://reviews.apache.org/r/9641/diff/ > > > Testing > ------- > > Manual. > Build and visualize through browser. > > > Thanks, > > Mohammad Islam > >
