> On June 16, 2015, 11:26 p.m., Dan Smith wrote: > > dev-tools/build/Dockerfile, line 7 > > <https://reviews.apache.org/r/35541/diff/1/?file=986122#file986122line7> > > > > This probably should be jdk-7u79. I think there were some issues with > > some of the middle versions of jdk-7, and I know at one point geode issued > > warning if you were using less than 72 or something like that.
Agree with Dan, must use JDK7 build > 72. Since 79 is the last in the JDK7 line, we should use it instead of build 55 - Mark ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35541/#review88157 ----------------------------------------------------------- On June 16, 2015, 11:13 p.m., jun aoki wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35541/ > ----------------------------------------------------------- > > (Updated June 16, 2015, 11:13 p.m.) > > > Review request for geode. > > > Bugs: GEODE-60 > https://issues.apache.org/jira/browse/GEODE-60 > > > Repository: geode > > > Description > ------- > > Added Dockerfile for docker to build a container which lets users to build > Geode. > > > Diffs > ----- > > README.md 8ac89a0 > dev-tools/build/Dockerfile PRE-CREATION > > Diff: https://reviews.apache.org/r/35541/diff/ > > > Testing > ------- > > Tested locally. > > > Thanks, > > jun aoki > >
