Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (nike.apache.org: domain of sebbaz@gmail.com designates
 209.85.220.171 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAOGo0VYKuEQ_kfMf8V0VBwUSBcqZJtQq4fOi6+HVxKnodAPMSg@mail.gmail.com>
References: <D954A8F2-1601-4F1D-9FCA-E64B98841BE6@apache.org>
	<CACZkXPzDdAj9tQWi-LQpAGSSiZytx24oOb5sZ-vehz+WuVi-+A@mail.gmail.com>
	<CAOGo0VZscfegvPLaJ0HQECOnRwntGqAgs=Fz55nRHJw=YFBctQ@mail.gmail.com>
	<CAOGo0VYKuEQ_kfMf8V0VBwUSBcqZJtQq4fOi6+HVxKnodAPMSg@mail.gmail.com>
Date: Mon, 28 Nov 2011 10:15:24 +0000
Message-ID: 
 <CAOGo0VYedc_WBWS4Jf_NVW_bZiN_GbAFbdDDYvPZsg720y-J7w@mail.gmail.com>
Subject: Re: [VOTE] Release JEXL 2.1 based on RC1
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On 28 November 2011 09:40, sebb <sebbaz@gmail.com> wrote:
> On 28 November 2011 01:40, sebb <sebbaz@gmail.com> wrote:
>> On 28 November 2011 01:03, Gary Gregory <garydgregory@gmail.com> wrote:
>>> I see 35 Clirr Errors that point to backwards incompatibility.
>>
>> Many of these relate to the parser, which is not really part of the publ=
ic API.
>
> Just checked, and the clirr plugin supports an excludes configuration
> element which could be used to ignore these.

I've fixed the pom so it ignores the parser classes (and tidied up a
bit; there were some unnecessary entries).

There are possibly one or two other ignorable classes - e.g. the nested cla=
ss

org.apache.commons.jexl2.UnifiedJEXL$Expression

All its existing subclasses are private, so I wonder why it is public?

It's OK to break binary compat if the class is not part of the API, so
it's worth checking what is public because it is part of the API and
what is public merely because the code uses multiple package names
(and so needs public rather than package access).

What about the following packages:

org.apache.commons.jexl2.internal
org.apache.commons.jexl2.introspection

I assume the former is not part of the API - what about the latter?

>> However, there are some which do look like they break binary compatibili=
ty
>>
>>> So... do we have a Commons-wide policy of changing package names and ma=
king
>>> a major release (3.0 vs. 2.1 here) or not?
>>
>> I think Jexl should be binary compatible.
>>
>> If the changes cannot be re-done in a binary compatible way, then I
>> think Jexl needs a major release and package change.
>>
>>> Gary
>>>
>>> On Sun, Nov 27, 2011 at 1:34 PM, Henri Biestro <henrib@apache.org> wrot=
e:
>>>
>>>>
>>>> Dear all,
>>>>
>>>> After a pretty long cycle, JEXL 2.1 is ready for review.
>>>> Here is a quick list of new features (from the release notes):
>>>>
>>>> What's new in 2.1:
>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> * A more thorough arithmetic (JexlArithmetic) that allows fine control
>>>> over decimals (scale and precision), a
>>>> =A0new syntax for numeric literals (OGNL inspired Big and Huge notatio=
ns)
>>>> and a better type handling keeping the most
>>>> =A0appropriate representation in casual operations.
>>>> * The introduction of script variables and parameters that reduce cont=
ext
>>>> dependencies and methods; this allows to
>>>> =A0perform checks after script creation (light static checking hints).=
 Plus
>>>> the ability to call script from scripts.
>>>> * A sandoxing feature to restrict and rename what JEXL can access from=
 the
>>>> environment allowing tighter control over security.
>>>> * Extensions to UnifiedJEXL that allow the creation of templates.
>>>>
>>>> New features in 2.1:
>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> * JEXL-114: =A0 =A0 Allow scripts to create local variables // Add ret=
urn
>>>> keyword
>>>> * JEXL-113: =A0 =A0 Add functions to extract which variables, paramete=
rs and
>>>> local variables are used to evaluate a script
>>>> * JEXL-118: =A0 =A0 Provide an IN operator
>>>> * JEXL-115: =A0 =A0 Add support for asynchronous script execution and
>>>> cancellation
>>>> * JEXL-116: =A0 =A0 Add control over classes, methods, constructors an=
d
>>>> properties allowed in scripts
>>>> * JEXL-120: =A0 =A0 Add simple template features
>>>> * JEXL-119: =A0 =A0 Allow indexed properties container resolution in
>>>> expressions
>>>>
>>>> Tested against Java 1.{5,6} / Maven{2,3}, Windows 7/Linux/Mac OS.
>>>>
>>>> Tag:
>>>>
>>>>
>>>> https://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL=
_2_1_RC1/
>>>>
>>>> Site:
>>>>
>>>> https://people.apache.org/~henrib/jexl-2.1
>>>>
>>>> Binaries:
>>>>
>>>> =A0https://repository.apache.org/content/repositories/orgapachecommons=
-258/
>>>>
>>>> This vote will close in 72 hours, 08:00PM GMT, Nov 30th.
>>>>
>>>> =A0[ ] +1 Release these artifacts
>>>> =A0[ ] +0 OK, but...
>>>> =A0[ ] -0 OK, but really should fix...
>>>> =A0[ ] -1 I oppose this release because...
>>>>
>>>> Many thanks,
>>>> Regards.
>>>> Henrib
>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>> E-Mail: garydgregory@gmail.com | ggregory@apache.org
>>> JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
>>> Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
>>> Blog: http://garygregory.wordpress.com
>>> Home: http://garygregory.com/
>>> Tweet! http://twitter.com/GaryGregory
>>>
>>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org