Il giorno mar 16 lug 2019 alle ore 07:44 Andor Molnar <an...@apache.org> ha scritto:
> Yes, my main concern is: why does the maven succeed, if we're still > having javadoc issues? > if I run in zookeeper-server this command I can't find any WARNING or ERROR mvn javadoc:javadoc only this ignorable one (I think) INFO] --- maven-javadoc-plugin:3.0.1:javadoc (default-cli) @ zookeeper --- [ERROR] Error fetching link: /home/diennea.lan/enrico.olivelli/dev/zookeeper/zookeeper-jute/target/site/apidocs/package-list. Ignored it. and javadocs are generated correctly Does Idea fail the build ? do you have logs to share ? (I don't have Idea, I am using NetBeans that apparently runs maven from the command line when building) Enrico > > Andor > > > -----Original Message----- > From: Enrico Olivelli <eolive...@gmail.com> > Reply-To: dev@zookeeper.apache.org > To: dev@zookeeper.apache.org > Subject: Re: Intellij Idea warning as error with -Xdoclint > Date: Mon, 15 Jul 2019 21:58:09 +0200 > > Il lun 15 lug 2019, 13:53 Andor Molnar < > an...@apache.org > > ha scritto: > > > Hi Enrico, > > > > This is still an issue for me in IDEA: > > > > Javadoc problems found in zookeeper-jute and zookeeper-server > > projects, so > > the build eventually fails. Strange thing is that running maven from > > command line doesn’t show any warning nor error. > > > > From javac params -Werror and -Xdoclint I think IDEA’s behaviour is > > correct. > > > > Most warning messages in IDEA are about missing javadoc comments, so > > changing it to "-Xdoclint:-missing” solves the problem. > > > > I can try to understand better. > If I understand correctly your point is that apparently Idea is working > as > expected but Maven not and this is not good because CI runs with Maven. > > I will be back with news. > Hopefully we can fix javadocs and have a reliable way to detect issues > on > docs on CI/precommit/nightly > > Enrico > > > > Regards, > > Andor > > > > > > > > > > > On 2019. Jun 20., at 20:28, Andor Molnar < > > > an...@apache.org > > > > wrote: > > > > > > Done. #983 merged, problem gone. > > > Thanks maoling! > > > > > > Andor > > > > > > > > > > > > > On 2019. Jun 20., at 15:43, maoling < > > > > maoling199210...@sina.com > > > > > wrote: > > > > > > > > I also noticed this when running UT with IDEA.may we can merge > > > > this PR: > > > > https://github.com/apache/zookeeper/pull/988 > > > > > > > > Or this one: > > > > https://github.com/apache/zookeeper/pull/983 > > > > to remove > > > > that warning. > > > > 发送自 Windows 10 版邮件应用 > > > > > > > > 发件人: Andor Molnar > > > > 发送时间: 2019年6月20日 5:46 > > > > 收件人: DevZooKeeper > > > > 主题: Intellij Idea warning as error with -Xdoclint > > > > > > > > > > > > Hi Enrico, > > > > > > > > I have the following error message in Idea since -Xdoclint is > > > > enabled > > > > in the main pom.xml file: > > > > > > > > /Users/andormolnar/git/my-zookeeper/zookeeper- > > jute/src/main/java/org/apache/jute/Utils.java > > > > Error:(194, 15) java: @param name not found > > > > Error:(231, 15) java: @param name not found > > > > > > > > Strange that I don’t see the same warnings in console when > > > > running ‘mvn > > > > install’. > > > > I confirm that removing “-Xdoclint” (or fixing javadoc issues) > > > > solves > > > > the problem. > > > > Andor > > > > > > > > > > > > > > > > > > > > > >