poi-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dominik Stadler <dominik.stad...@gmx.at>
Subject Re: [VOTE] Apache POI 3.15 (RC2)
Date Wed, 14 Sep 2016 19:41:54 GMT
Hi,

I would rather look at using some tools like
https://github.com/siom79/japicmp on the CI machine and check the output on
a regular basis or fail the build if there are incompatible changes. This
way we do not rely on having unit-tests for a method and the old source
often will not compile any more with newer code for arbitrary
reasonshttps://bz.apache.org/bugzilla/show_bug.cgi?id=60134.

I have created https://bz.apache.org/bugzilla/show_bug.cgi?id=60134 to
track this idea.

Dominik.

On Wed, Sep 14, 2016 at 2:27 AM, Javen O'Neal <javenoneal@gmail.com> wrote:

> Could we do some checks for backwards compatibility? We need to test an
> application written for a previous version of POI that exercises most or
> all of the features, and what better than to use the unit tests from a
> previous release plus the latest source code? This will include false
> positives such as removal and rename of deprecated classes and methods.
>
> $ svn co https://svn.apache.org/repos/asf/tag/REL_3_15_FINAL
> $ cd REL_3_15_FINAL
> $ svn up -r1732982 src/testcases src/ooxml/testcases
> src/scratchpad/testcases
> $ ant clean jenkins
>
> This doesn't compile due to deletion of deprecated features and any
> backwards incompatibilities.
>
> Is there a better way of doing this?
>
> On Sep 13, 2016 09:06, "Javen O'Neal" <javenoneal@gmail.com> wrote:
>
> > I will commit a fix for this today with the goal for backwards
> > compatibility.
> >
> > Here's the plan:
> > getX() returns int
> > getXEnum() returns enum
> > setX(int)
> > setX(enum)
> >
> > I will also take a look at bug 59907 (client anchor enum)
> >
> > On Sep 13, 2016 6:58 AM, "David North" <dtn-poi@corefiling.co.uk> wrote:
> >
> >> Javen, any thoughts on this one?
> >>
> >> On 13/09/16 12:14, Dominik Stadler wrote:
> >> > Hi,
> >> >
> >> > I really hate to delay this further, but unfortunately we have a
> similar
> >> > problem in class CellValue as we tried to fix in Cell in RC2, the
> >> > getCellType() is now an enum whereas it was an int before, so
> something
> >> > like the following in user-code does break in POI 3.15:
> >> >
> >> > CellValue cellValue = checkAndGetCellValue(evaluator, sheet, line);
> >> >
> >> >     switch (cellValue.getCellType()) {
> >> >         case Cell.CELL_TYPE_STRING:
> >> >
> >> >
> >> > I am sorry that I did not see this earlier but this can lead to the
> same
> >> > incompatibility as we had in Cell before.
> >> >
> >> > Dominik.
> >> >
> >> > On Sun, Sep 11, 2016 at 9:46 PM, David North <dnorth@apache.org>
> wrote:
> >> >
> >> >> Hi everyone,
> >> >>
> >> >> My apologies for going AWOL in the middle of the last release
> attempt.
> >> I
> >> >> didn't anticipate that we'd find problems in review twice in a row,
> and
> >> >> things have been very busy for me at work lately. However, I've now
> >> >> rolled a second RC for 3.15.
> >> >>
> >> >> https://dist.apache.org/repos/dist/dev/poi/3.15-RC2/
> >> >>
> >> >> Areas to review:
> >> >>
> >> >> * Does it work?
> >> >> * Are the sigs and hashes valid?
> >> >> * Have the issues with the last RC been fixed?
> >> >> * Are the release notes now in good shape?
> >> >>
> >> >> The vote starts now and ends at 20:55 BST on Tuesday 13 September
> 2016.
> >> >>
> >> >> Here is my +1.
> >> >>
> >> >> After this release is done, I'll try and find some time to profile
> the
> >> >> build & tests - 15 minutes is quite a wait on an SSD (it's possible
> we
> >> >> might want some multi-threaded options on the tests).
> >> >>
> >> >> Thanks,
> >> >>
> >> >> --
> >> >> David North | www.dnorth.net
> >> >>
> >> >>
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
> >> For additional commands, e-mail: dev-help@poi.apache.org
> >>
> >>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message