You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Emmanuel Bourg <eb...@apache.org> on 2015/12/18 17:05:32 UTC

[VOTE] Release JEXL 3.0 based on RC2

Hi all,

JEXL 3.0 RC2 is now available for review.

Distribution:
https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)

Maven artifacts:
https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/

Release notes:
https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
http://people.apache.org/~ebourg/jexl/site/changes-report.html

Tag:
http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/ (r1720787)

Site:
http://people.apache.org/~ebourg/jexl/site

KEYS:
https://www.apache.org/dist/commons/KEYS


Please review the release candidate and vote.
This vote will close no sooner that 72 hours from now, i.e. after Mon, 21 Dec 2015 16:05:00 GMT

[ ] +1 Release these artifacts
[ ] +0 OK, but...
[ ] -0 OK, but really should fix...
[ ] -1 I oppose this release because...

Thanks you,

Emmanuel Bourg


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Oliver Heger <ol...@oliver-heger.de>.
Build works fine with Java 1.6 and 1.8 on Windows 10. Artifacts and
site look good.

There are many findbugs and a bunch of PMD warnings, but this is not
blocking.

+1

Oliver

Am 18.12.2015 um 17:05 schrieb Emmanuel Bourg:
> Hi all,
> 
> JEXL 3.0 RC2 is now available for review.
> 
> Distribution: https://dist.apache.org/repos/dist/dev/commons/jexl/
> (r11664)
> 
> Maven artifacts: 
> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
>
>  Release notes: 
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
>
> 
http://people.apache.org/~ebourg/jexl/site/changes-report.html
> 
> Tag: 
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/
> (r1720787)
> 
> Site: http://people.apache.org/~ebourg/jexl/site
> 
> KEYS: https://www.apache.org/dist/commons/KEYS
> 
> 
> Please review the release candidate and vote. This vote will close
> no sooner that 72 hours from now, i.e. after Mon, 21 Dec 2015
> 16:05:00 GMT
> 
> [ ] +1 Release these artifacts [ ] +0 OK, but... [ ] -0 OK, but
> really should fix... [ ] -1 I oppose this release because...
> 
> Thanks you,
> 
> Emmanuel Bourg
> 

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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Matt Benson <gu...@gmail.com>.
On Fri, Dec 18, 2015 at 10:05 AM, Emmanuel Bourg <eb...@apache.org> wrote:
> Hi all,
>
> JEXL 3.0 RC2 is now available for review.
>
> Distribution:
> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
>
> Maven artifacts:
> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
>
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
> http://people.apache.org/~ebourg/jexl/site/changes-report.html
>
> Tag:
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/ (r1720787)
>
> Site:
> http://people.apache.org/~ebourg/jexl/site
>
> KEYS:
> https://www.apache.org/dist/commons/KEYS
>
>
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21 Dec 2015 16:05:00 GMT
>

- Site looks fine
- hashes and signatures check out
- `mvn install` works from source archive
- L&N present in all artifacts

> [X] +1 Release these artifacts
> [ ] +0 OK, but...
> [ ] -0 OK, but really should fix...
> [ ] -1 I oppose this release because...
>

Matt

> Thanks you,
>
> Emmanuel Bourg
>

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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Emmanuel Bourg <eb...@apache.org>.
Le 18/12/2015 17:05, Emmanuel Bourg a écrit :

> [X] +1 Release these artifacts

Tested on Debian with OpenJDK 7 & 8, and on Windows with Java 8.

Emmanuel Bourg



Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Jörg Schaible <jo...@gmx.de>.
Emmanuel Bourg wrote:

> Le 21/12/2015 12:36, Jörg Schaible a écrit :
> 
>> I find it rather confusing...
> 
> I agree it could be confusing if we already had previous jexl releases
> using commons-jexl<n>-*-bin.zip, and I wouldn't recommend that for lang,
> math, dbcp... But that's not the case here, it's just the continuity of
> the current naming used for the previous jexl archives in the
> distribution directory [1].
> 
> Emmanuel Bourg
> 
> [1] https://dist.apache.org/repos/dist/release/commons/jexl/source/

IMHO, it's very unfortunate that we had used different naming schemes. It 
would have been the best moment to correct it now with a new major release.

Therefore, just +0

Cheers,
Jörg


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Emmanuel Bourg <eb...@apache.org>.
Le 21/12/2015 12:36, Jörg Schaible a écrit :

> I find it rather confusing...

I agree it could be confusing if we already had previous jexl releases
using commons-jexl<n>-*-bin.zip, and I wouldn't recommend that for lang,
math, dbcp... But that's not the case here, it's just the continuity of
the current naming used for the previous jexl archives in the
distribution directory [1].

Emmanuel Bourg

[1] https://dist.apache.org/repos/dist/release/commons/jexl/source/


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Jörg Schaible <jo...@gmx.de>.
Emmanuel Bourg wrote:

> Le 21/12/2015 09:49, Jörg Schaible a écrit :
> 
>> -1
>> 
>> The file names of the distribution (source and binaries) do not match the
>> artifactId (compare e.g. with lang).
> 
> Thank you for the extensive tests Jörg.
> 
> The distribution files not strictly matching the artifactId is intended.
> I consider this as a nice refinement, because two versions in the
> filename could be confusing (foo3-3.0 could be mistaken for foo 3.3.0).
> 
> Emmanuel Bourg

Maybe, but it does not follow the pattern we've been using for all other 
components:

- http://commons.apache.org/proper/commons-lang/download_lang.cgi
- http://commons.apache.org/proper/commons-collections/download_collections.cgi
- http://commons.apache.org/proper/commons-configuration/download_configuration.cgi
- http://commons.apache.org/proper/commons-digester/download_digester.cgi
- http://commons.apache.org/proper/commons-dbcp/download_dbcp.cgi
- http://commons.apache.org/proper/commons-math/download_math.cgi
- http://commons.apache.org/proper/commons-pool/download_pool.cgi

Except for
- http://commons.apache.org/proper/commons-vfs/download_vfs.cgi

I find it rather confusing...

Cheers,
Jörg


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Emmanuel Bourg <eb...@apache.org>.
Le 21/12/2015 09:49, Jörg Schaible a écrit :

> -1
> 
> The file names of the distribution (source and binaries) do not match the 
> artifactId (compare e.g. with lang).

Thank you for the extensive tests Jörg.

The distribution files not strictly matching the artifactId is intended.
I consider this as a nice refinement, because two versions in the
filename could be confusing (foo3-3.0 could be mistaken for foo 3.3.0).

Emmanuel Bourg


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Jörg Schaible <jo...@gmx.de>.
Emmanuel Bourg wrote:

> Hi all,
> 
> JEXL 3.0 RC2 is now available for review.
> 
> Distribution:
> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
> 
> Maven artifacts:
> 
https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
> 
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
> http://people.apache.org/~ebourg/jexl/site/changes-report.html
> 
> Tag:
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/
> (r1720787)
> 
> Site:
> http://people.apache.org/~ebourg/jexl/site
> 
> KEYS:
> https://www.apache.org/dist/commons/KEYS
> 
> 
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21
> Dec 2015 16:05:00 GMT
> 
> [ ] +1 Release these artifacts
> [ ] +0 OK, but...
> [ ] -0 OK, but really should fix...
> [ ] -1 I oppose this release because...
> 
> Thanks you,
> 
> Emmanuel Bourg

-1

The file names of the distribution (source and binaries) do not match the 
artifactId (compare e.g. with lang). However, I could build from source with 
my complete compiler zoo without any problems. Therefore, if it is simply 
possible to rename the files, I can change my vote without a new RC.

Cheers,
Jörg




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


Re: [RESULT][VOTE] Release JEXL 3.0 based on RC2

Posted by sebb <se...@gmail.com>.
On 21 December 2015 at 22:53, Emmanuel Bourg <eb...@apache.org> wrote:
> The following votes were received:
>
> +1
> Matt Benson
> Oliver Heger
> Henri Biestro (emeritus)
> Emmanuel Bourg
>
> +0
> Jörg Schaible
>
> -0
> Gary Gregory
>
> Henri went emeritus in 2012 but he is still active since he drove the
> JEXL 3.0 release. I don't know if his vote is considered binding in this
> situation, but that doesn't change the outcome of the vote. With a

Only current PMC member votes are binding.
Henri left the PMC in 201; emeritus just means that he used to be on the PMC.

> majority of at least 3 binding votes the release passes.
>
> Thanks all for your reviews and your votes, I'll proceed with the
> release in the next days.
>
> Emmanuel Bourg
>
>
> Le 18/12/2015 17:05, Emmanuel Bourg a écrit :
>> Hi all,
>>
>> JEXL 3.0 RC2 is now available for review.
>>
>> Distribution:
>> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
>>
>> Maven artifacts:
>> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
>>
>> Release notes:
>> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
>> http://people.apache.org/~ebourg/jexl/site/changes-report.html
>>
>> Tag:
>> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/ (r1720787)
>>
>> Site:
>> http://people.apache.org/~ebourg/jexl/site
>>
>> KEYS:
>> https://www.apache.org/dist/commons/KEYS
>>
>>
>> Please review the release candidate and vote.
>> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21 Dec 2015 16:05:00 GMT
>>
>> [ ] +1 Release these artifacts
>> [ ] +0 OK, but...
>> [ ] -0 OK, but really should fix...
>> [ ] -1 I oppose this release because...
>>
>> Thanks you,
>>
>> Emmanuel Bourg
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

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


[RESULT][VOTE] Release JEXL 3.0 based on RC2

Posted by Emmanuel Bourg <eb...@apache.org>.
The following votes were received:

+1
Matt Benson
Oliver Heger
Henri Biestro (emeritus)
Emmanuel Bourg

+0
Jörg Schaible

-0
Gary Gregory

Henri went emeritus in 2012 but he is still active since he drove the
JEXL 3.0 release. I don't know if his vote is considered binding in this
situation, but that doesn't change the outcome of the vote. With a
majority of at least 3 binding votes the release passes.

Thanks all for your reviews and your votes, I'll proceed with the
release in the next days.

Emmanuel Bourg


Le 18/12/2015 17:05, Emmanuel Bourg a écrit :
> Hi all,
> 
> JEXL 3.0 RC2 is now available for review.
> 
> Distribution:
> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
> 
> Maven artifacts:
> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
> 
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
> http://people.apache.org/~ebourg/jexl/site/changes-report.html
> 
> Tag:
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/ (r1720787)
> 
> Site:
> http://people.apache.org/~ebourg/jexl/site
> 
> KEYS:
> https://www.apache.org/dist/commons/KEYS
> 
> 
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21 Dec 2015 16:05:00 GMT
> 
> [ ] +1 Release these artifacts
> [ ] +0 OK, but...
> [ ] -0 OK, but really should fix...
> [ ] -1 I oppose this release because...
> 
> Thanks you,
> 
> Emmanuel Bourg
> 


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by henrib <he...@apache.org>.
+1
Thank you Emmanuel !!



--
View this message in context: http://apache-commons.680414.n4.nabble.com/VOTE-Release-JEXL-3-0-based-on-RC2-tp4681229p4681272.html
Sent from the Commons - Dev mailing list archive at Nabble.com.

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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Gary Gregory <ga...@gmail.com>.
Note: I expected the following to be front and center on the site:

========================================================================================================================
Release 3.0
========================================================================================================================

Version 3.0 is a major release changing the public JEXL API.

Due to binary and source incompatibilities and the scope of
modifications, JEXL has moved to the
org.apache.commons.jexl3 package.
However, this is a drop-in replacement through JSR-223 and casual code
usage should find easy equivalence between
jexl-2.x and 3.x (besides the package name).
Also note that JEXL 3.0 now requires Java 6.


What's new in 3.0:
==================
* A redesigned API that properly segregates the "public" part and the
"internal" parts of JEXL.
* A real difference between Expression and Script (scripts accept
statements: for, while, etc).
* The ability to create JEXL functions in script, closure/variables
hoisting/currying included.
* A set of assignment operators (+=, *= , ...).
* A set of String matching operators (^= startsWith, $= endsWith) and
their negative counterparts (!^, !$)
* A range operator (x .. y) to ease writing loops (for(var x : 1 .. 10) ... )
* A literal set syntax to ease creating sets: { 1, 2...}

Gary


On Fri, Dec 18, 2015 at 8:05 AM, Emmanuel Bourg <eb...@apache.org> wrote:

> Hi all,
>
> JEXL 3.0 RC2 is now available for review.
>
> Distribution:
> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
>
> Maven artifacts:
>
> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
>
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
> http://people.apache.org/~ebourg/jexl/site/changes-report.html
>
> Tag:
>
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/
> (r1720787)
>
> Site:
> http://people.apache.org/~ebourg/jexl/site
>
> KEYS:
> https://www.apache.org/dist/commons/KEYS
>
>
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21
> Dec 2015 16:05:00 GMT
>
> [ ] +1 Release these artifacts
> [ ] +0 OK, but...
> [ ] -0 OK, but really should fix...
> [ ] -1 I oppose this release because...
>
> Thanks you,
>
> Emmanuel Bourg
>
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Emmanuel Bourg <eb...@apache.org>.
Le 20/12/2015 22:45, Gary Gregory a écrit :
> -0: Can't validate downloads from
> https://dist.apache.org/repos/dist/dev/commons/jexl/ as there is no SHA1 or
> MD5 files.

Thank you for catching this, I added them.

I copied the src/bin archives from the target directory and it didn't
contain the md5/sha1 files.

It would be nice to automate the part of the release process uploading
the archives to dist.apache.org/repos/dist/dev, with an Ant task bound
to the deploy phase for example.

Emmanuel Bourg


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


Re: [VOTE] Release JEXL 3.0 based on RC2

Posted by Gary Gregory <ga...@gmail.com>.
-0: Can't validate downloads from
https://dist.apache.org/repos/dist/dev/commons/jexl/ as there is no SHA1 or
MD5 files.

Gary

On Fri, Dec 18, 2015 at 8:05 AM, Emmanuel Bourg <eb...@apache.org> wrote:

> Hi all,
>
> JEXL 3.0 RC2 is now available for review.
>
> Distribution:
> https://dist.apache.org/repos/dist/dev/commons/jexl/ (r11664)
>
> Maven artifacts:
>
> https://repository.apache.org/content/repositories/orgapachecommons-1133/org/apache/commons/commons-jexl3/3.0/
>
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/jexl/RELEASE-NOTES.txt
> http://people.apache.org/~ebourg/jexl/site/changes-report.html
>
> Tag:
>
> http://svn.apache.org/repos/asf/commons/proper/jexl/tags/COMMONS_JEXL_3_0-RC2/
> (r1720787)
>
> Site:
> http://people.apache.org/~ebourg/jexl/site
>
> KEYS:
> https://www.apache.org/dist/commons/KEYS
>
>
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now, i.e. after Mon, 21
> Dec 2015 16:05:00 GMT
>
> [ ] +1 Release these artifacts
> [ ] +0 OK, but...
> [ ] -0 OK, but really should fix...
> [ ] -1 I oppose this release because...
>
> Thanks you,
>
> Emmanuel Bourg
>
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory