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