> On April 11, 2016, 3:43 p.m., Bruce Schuchardt wrote: > > geode-assembly/build.gradle, line 116 > > <https://reviews.apache.org/r/45993/diff/1/?file=1338503#file1338503line116> > > > > Is this modifying the assembled product's lib directory? > > Anthony Baker wrote: > I agree, doesn't seem right to modify the contents of lib/ to run a unit > test.
gradle change reverted - Udo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45993/#review128137 ----------------------------------------------------------- On April 10, 2016, 9:59 p.m., Udo Kohlmeyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45993/ > ----------------------------------------------------------- > > (Updated April 10, 2016, 9:59 p.m.) > > > Review request for geode, Bruce Schuchardt, Hitesh Khamesra, and Jianxia Chen. > > > Repository: geode > > > Description > ------- > > GEODE-1201: Adding compileRuntimeLibs to geode-assembly for tests Amending > bind address configuration for http-service-bind-address > > > Diffs > ----- > > geode-assembly/build.gradle b7d05e2ce5fe34d3324e2b55212ff7c8ca02d9e0 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/RestAgent.java > a91df05a9931bddd860a79e6b8783fc991ecc2a4 > > geode-web-api/src/main/java/com/gemstone/gemfire/rest/internal/web/swagger/config/RestApiPathProvider.java > a8921d7d6f2a062a9aa5e27a8376a7e39c8503d5 > > Diff: https://reviews.apache.org/r/45993/diff/ > > > Testing > ------- > > battery tests rest.bt > > > Thanks, > > Udo Kohlmeyer > >
