> On Feb. 17, 2015, 3:38 p.m., Srikanth Sundarrajan wrote: > > bin-dist-files/README, line 57 > > <https://reviews.apache.org/r/31116/diff/1/?file=866324#file866324line57> > > > > I know we are also creating a deb file. Do we intend to make it > > available as a conveninent binary ? If so, we should have notes on how to > > setup using deb, else you can ignore. > > > > In general there needs to documentation on how deb can be used for > > installation, but that can be taken up in an independent jira and not a > > blocker for release.
Yes. Debians are not part of binary distribution as of now. There is mention of debians in http://lens.incubator.apache.org/developer/contribute.html#Building_from_source doc. We can put a separate section and add more details about. > On Feb. 17, 2015, 3:38 p.m., Srikanth Sundarrajan wrote: > > src/site/apt/releases/release-history.apt, line 29 > > <https://reviews.apache.org/r/31116/diff/1/?file=866341#file866341line29> > > > > Why is this in release history and not in README ? Should we state why > > it is excluded as well ? Added the reason and put in README as well. - Amareshwari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31116/#review72721 ----------------------------------------------------------- On Feb. 18, 2015, 5:25 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31116/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2015, 5:25 a.m.) > > > Review request for lens, Chris Douglas and Srikanth Sundarrajan. > > > Bugs: LENS-311 > https://issues.apache.org/jira/browse/LENS-311 > > > Repository: lens > > > Description > ------- > > Changes include : > - Change source release artifact name to include apache-lens. Changed > lens-parent to apache-lens. > - Add exclusion for org.acplt:oncrpc in apache-release profile > - Add LICENSE and NOTICE in war packaging > - Update README.md for source distribution > - Update bin-dist-files/README for binary distribution > > Have some questions : > - LICENSE and NOTICE in lens-server.war looks the following. Is this > acceptable? > $ ls webapp/lens-server/ > META-INF/ WEB-INF/ static/ > > $ ls webapp/lens-server/META-INF/ > MANIFEST.MF maven/ > > $ ls webapp/lens-server/WEB-INF/ > classes/ lib/ log4j.properties web.xml > > $ ls webapp/lens-server/WEB-INF/classes/ > META-INF/ lens-build-info.properties > lensserver-default.xml lenssession-default.xml org/ > > $ ls webapp/lens-server/WEB-INF/classes/META-INF/ > DEPENDENCIES LICENSE NOTICE > > - Is it ok to have README file README.md for source distribution? > - bin-dist-files folder is included in src distribution. Is it acceptable? > > > Diffs > ----- > > README.md 07a3249 > bin-dist-files/README 3e74fb9 > lens-api/pom.xml b955c1a > lens-cli/pom.xml 26adc54 > lens-client/pom.xml e56df03 > lens-cube/pom.xml 274d1ea > lens-dist/pom.xml ddebf16 > lens-driver-hive/pom.xml be0c1b7 > lens-driver-jdbc/pom.xml daab50d > lens-examples/pom.xml 2163785 > lens-ml-lib/pom.xml 3cdca92 > lens-query-lib/pom.xml 95d9d93 > lens-regression/pom.xml d506eaa > lens-server-api/pom.xml f490dee > lens-server/pom.xml 0986ab9 > lens-storage-db/pom.xml fb35e12 > pom.xml 38d1693 > src/site/apt/developer/contribute.apt 91e7b81 > src/site/apt/lenshome/install-and-run.apt 6031cf9 > src/site/apt/releases/release-history.apt f5f6139 > > Diff: https://reviews.apache.org/r/31116/diff/ > > > Testing > ------- > > Verified packaging is fine by untarring the distribution. Will do some more > testing and update here. > > > Thanks, > > Amareshwari Sriramadasu > >
