> On Sept. 2, 2015, 6:59 p.m., Mark Bretl wrote: > > pulse/build.gradle, line 4 > > <https://reviews.apache.org/r/38060/diff/2/?file=1062427#file1062427line4> > > > > Why add a custom pulseCompile configuration?
compile target inherits from parent gradle project and use newer versions of Spring framework jars. I tried hard but couldn't override those dependencies from within a sub-project. Besides creating new tasks, the other option is to remove those from parent and move them downstream to different sub-projects using it. Is there a different way? > On Sept. 2, 2015, 6:59 p.m., Mark Bretl wrote: > > pulse/build.gradle, line 8 > > <https://reviews.apache.org/r/38060/diff/2/?file=1062427#file1062427line8> > > > > Why add a custom testPulseCompile configuration? testPulseCompile config inherits from parent gradle project and use newer versions of Spring framework jars. I tried hard but couldn't override those dependencies from within a sub-project. Besides creating new tasks, the other option is to remove those from parent and move them downstream to different sub-projects using it. Is there a different way? > On Sept. 2, 2015, 6:59 p.m., Mark Bretl wrote: > > pulse/build.gradle, line 81 > > <https://reviews.apache.org/r/38060/diff/2/?file=1062427#file1062427line81> > > > > Add war file to archive list? Yes, Dan raised it too. Have removed that from the archive file. > On Sept. 2, 2015, 6:59 p.m., Mark Bretl wrote: > > pulse/build.gradle, line 93 > > <https://reviews.apache.org/r/38060/diff/2/?file=1062427#file1062427line93> > > > > This can be removed, only for commercial license Thanks, it is commented out right now. Will pick-up the actual edits with GEODE-301. - Nitin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38060/#review97513 ----------------------------------------------------------- On Sept. 2, 2015, 6:42 p.m., Nitin Lamba wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38060/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2015, 6:42 p.m.) > > > Review request for geode, Jens Deppe, Mark Bretl, and Dan Smith. > > > Repository: geode > > > Description > ------- > > Create new gradle script for PULSE to build archives (jar, war files) using > the dependencies from existing ant build.xml file. > > These chances will be tracked on GEODE-12 branch until all subtasks are > completed. > > > Diffs > ----- > > build.gradle 3bad39c > gemfire-assembly/build.gradle 0e51563 > gemfire-assembly/src/test/java/AgentUtilJUnitTest.java 0f7563b > pulse/build.gradle PRE-CREATION > settings.gradle 7f6ed61 > > Diff: https://reviews.apache.org/r/38060/diff/ > > > Testing > ------- > > - pulse.war file gets generated correctly and placed in tools/Pulse folder > - Starting locator starts pulse within the embedded jetty instance > - Unit and integration tests are blocked. GEODE-304 is tracking the issue > separately > > > Thanks, > > Nitin Lamba > >
