On Tue, Jun 18, 2019 at 11:02 PM Enrico Olivelli <eolive...@gmail.com> wrote:
> Il mer 19 giu 2019, 06:38 Patrick Hunt <ph...@apache.org> ha scritto: > > > What's the current status on maven replacing ant? Is there a current > > summary somewhere? Perhaps someone could create a wiki page to track the > > status across the three branches and plans/remainder for each? > > > > Good idea! > I will do. > > > > > > I noticed today that even with the "full-build" profile it doesn't look > > like maven is building the c client (did I miss something?), > > > We are building the C client on travis in precommit > > mvn clean install -Pfull-build > > Hm, something is odd/broken then - looking back through my history I ran $ mvn clean install -DskipTests -Pfull-build the "configure" script was run but no make was run. I would expect that a full-build install would generate the c client even if testing was skipped. Any ideas what's up there? Patrick > > even though > > ant compile-native is. Stuff like this we can't lose and will need to be > > working before we can fully migrate. > > > > Sure > > > > > Thanks, > > > > Patrick > > > > > > On Thu, May 23, 2019 at 9:37 AM Fangmin Lv <lvfang...@gmail.com> wrote: > > > > > We’re still using ant as a build block internally on 3.6, working on > > moving > > > to maven, will change soon, so it’s fine to remove ant. > > > > > > Fangmin > > > > > > On Thu, May 23, 2019 at 2:55 AM Norbert Kalmar > > > <nkal...@cloudera.com.invalid> > > > wrote: > > > > > > > 3.4 will remain on Ant, we have decided/voted/discussed (not sure > which > > > was > > > > it :) ) that not long ago. Even maven is faulty on 3.4 ... > > > > So only 3.5 and master is on the vote. > > > > > > > > Thanks Enrico for working on this! > > > > > > > > On Wed, May 22, 2019 at 7:06 PM Enrico Olivelli <eolive...@gmail.com > > > > > > wrote: > > > > > > > > > Il mer 22 mag 2019, 12:17 Norbert Kalmar > > <nkal...@cloudera.com.invalid > > > > > > > > ha > > > > > scritto: > > > > > > > > > > > And "release with Ant" I mean the 3.5.5 src tarball containing > the > > > Ant > > > > > > script as well, so people have the option to build it with Ant. > > > > > > > > > > > > > > > > IMHO this is not a big deal. The release has been performed with > > Maven > > > > and > > > > > the resulting artifacts are different from the Ant results. > > > > > But I don't have a strong opinion here > > > > > > > > > > Enrico > > > > > > > > > > > > > > > > > > > > > > > On Wed, May 22, 2019 at 12:13 PM Norbert Kalmar < > > > nkal...@cloudera.com> > > > > > > wrote: > > > > > > > > > > > > > Sorry, I was too quick to reply, didn't think it through. > > > > > > > We did release the first stable 3.5 ZooKeeper with Ant, so I > > guess > > > we > > > > > > will > > > > > > > need to support on 3.5.x > > > > > > > > > > > > > > On Wed, May 22, 2019 at 12:12 PM Norbert Kalmar < > > > > nkal...@cloudera.com> > > > > > > > wrote: > > > > > > > > > > > > > >> +1, let's remove it. I would say lets remove Ant from 3.5 > branch > > > as > > > > > > well. > > > > > > >> Having 2 build system is just a huge source of confusion, > > > especially > > > > > > that > > > > > > >> we have dependency versions in two different locations. > > > > > > >> > > > > > > >> Regards, > > > > > > >> Norbert > > > > > > >> > > > > > > >> On Wed, May 22, 2019 at 9:44 AM Tamas Penzes > > > > > > <tam...@cloudera.com.invalid> > > > > > > >> wrote: > > > > > > >> > > > > > > >>> Hi All, > > > > > > >>> > > > > > > >>> Not that long ago we did have a discuss about removing Ant > from > > > > > > >>> ZooKeeper. > > > > > > >>> I'd like to restart that discussion since ZooKeeper 3.5.5 is > GA > > > and > > > > > it > > > > > > >>> was > > > > > > >>> built and released with Maven. > > > > > > >>> Is it time to remove Ant from the master branch? > > > > > > >>> > > > > > > >>> That would mean that Ant would not be available from the next > > > minor > > > > > > >>> version, which is probably 3.6.0. > > > > > > >>> > > > > > > >>> Please share your opinion. > > > > > > >>> > > > > > > >>> Thanks, Tamaas > > > > > > >>> > > > > > > >> > > > > > > > > > > > > > > > > > > > > >