osdir.com


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [parent] japicmp problems


+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 <chtompki@xxxxxxxxx> 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 <bodewig@xxxxxxxxxx> 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-unsubscribe@xxxxxxxxxxxxxxxxxx
> > For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxxxx
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxxxxxx
> For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxxxx
>
>