+1 using japicmp is maddening due to bugs. It feels unsupported... fork it and fix it within Commons?
Gary On Sun, Aug 12, 2018, 07:08 Rob Tompkins <chtom...@gmail.com> wrote: > I agree with this statement generally. That said, this doesn’t surprise me > that I made this error because japicmp is minimally cumbersome to work > with. It feels very unfinished/unpolished. Next time I get into [parent], > I’ll try to go up and submit more polishing work to the japicmp codebase. > > -Rob > > > On Aug 12, 2018, at 8:45 AM, Stefan Bodewig <bode...@apache.org> wrote: > > > > Hi > > > > while building the site for Compress the japicmp report was empty, > > taking a closer look there is a little warning that says "skipping > > report because skip is set to true" which in fact it is. > > > > > http://svn.apache.org/viewvc/commons/proper/commons-parent/trunk/pom.xml?r1=1832506&r2=1833335 > > added this (in a seemingly unrelated commit) and thus effectively > > disabled japicmp. I *think* we need to add <skip>false</skip> to the > > japicmp profile or better introduce a property like we have for most of > > the other reports. As I don't understand why skip was added in the first > > place I won't change the parent POM, though. Will fix Compress' POM > > locally. > > > > In related news trying to use japicmp 0.12.0 leads to a failed build with > > > > Caused by: java.lang.NullPointerException > > at japicmp.maven.JApiCmpMojo.isPomModuleNeedingSkip > (JApiCmpMojo.java:1243) > > at japicmp.maven.JApiCmpMojo.executeWithParameters > (JApiCmpMojo.java:130) > > > > in Compress. I'll explicitly revert to 0.11.1 as this seems to work > > locally. > > > > Stefan > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >