I fixed the versions: https://github.com/apache/zookeeper/pull/772 But this is just the same fix your patch contains Enrico (but anyway, we can have it in a separate PR.)
The error in 2) is because somehow the dependency did not install in local maven repository. ZooInspector has a jar attached in the source code, that is required to be on the classpath. The "nice" way to do this in maven (unfortunately, it is not in central repo), is to install the jar in the local .m2 repo. I do this in the initialization phase with maven-install-plugin. I think the problem is that after you updated the versions, you have to force snapshots to update dependencies with the -U switch. And one more thing: unfortunately versions are also wrong on 3.5, so I will create a PR for that also. Regards, Norbert On Sun, Jan 13, 2019 at 11:46 AM Norbert Kalmar <[email protected]> wrote: > Hi Enrico, > > I can agree on skipping spotbugs on contrib. There was even a plan to > remove most contrib to a seperate top-level project, out of ZooKeeper. Most > of them isn't even maintained for a long time now. > Anyway, that's a non-binding +1 from me. > > I will check the pom's on 3.4, sorry about that. Looks like even though I > double checked the versions still left some errors during backport. I will > have a patch ready today or tomorrow. > And that should also fix 3) of course. > > Thanks Enrico for reporting this! > > Regards, > Norbert > > On Fri, Jan 11, 2019 at 11:28 PM Enrico Olivelli <[email protected]> > wrote: > >> Hi, >> we have some problems on 3.4 branch with Maven migration. >> >> 1) There are references to 3.6.0-SNAPSHOT >> >> https://github.com/apache/zookeeper/blob/b8044eae3e700650a646969352a508122a4e5a98/zookeeper-contrib/zookeeper-contrib-loggraph/pom.xml#L26 >> >> 2) Build fails with this error (after fixing poms): >> [INFO] >> ------------------------------------------------------------------------ >> [ERROR] Failed to execute goal on project >> zookeeper-contrib-zooinspector: Could not resolve dependencies for >> project >> org.apache.zookeeper:zookeeper-contrib-zooinspector:jar:3.4.14-SNAPSHOT: >> Could not find artifact com.nitido.utils:toaster:jar:1.0.4 in central >> (https://repo.maven.apache.org/maven2) -> [Help 1] >> >> 3) I am porting spotbugs patch now, but there came a lot of new code, >> so I had to fix recipes and I have disabled the 'contrib' modules >> which are not compliant at all with spotbugs >> 4) There are a lot of spotbugs errors on 3.4 branch, not found on 3.5 and >> master >> >> This is my patch which is solving 1) and 3) and 4) >> https://github.com/apache/zookeeper/pull/764 >> >> I can split it, let me know. >> >> An alternative approach is to fix the poms in one patch (Norbert?). >> >> I guess I will have to re-work on spotbugs on master and branch-3.5 >> now that recipes and contrib landed to Maven world. >> >> I would prefer to skip Spotbugs on "contrib". >> I think spotbugs must pass in "recipes", if you take a look at my >> commit, there are many warnings, and I think they may be real bugs >> >> Cheers >> Enrico >> >
