Re: Release Commons Jelly 1.0.1 Based on RC2

2017-08-30 Thread Rob Tompkins
> On Aug 30, 2017, at 1:22 AM, Bruno P. Kinoshita > wrote: > > Hi Rob, > Thanks for RM'ing Commons Jelly 1.0.1. Here's my initial tests results: > > Downloaded JDK from Oracle archives, version 5.0u22. > > Then downloaded Apache Ant following what was in the Dockerfile. > > curl > http:

Re: [All][Math] New component: "Commons Geometry"?

2017-08-30 Thread Emmanuel Bourg
Le 21/08/2017 à 21:41, Gary Gregory a écrit : > What about this for a compromise: create Commons Math 5 as a multi-module > project and bring in as submodules only the newly minted components and > whatever gets spun out of Math 3/4. +1 for multiple modules instead of multiple components. Emmanue

Re: [BCEL] Breaking changes in 6.1 needed by Tomcat (?)

2017-08-30 Thread Gary Gregory
We recently dealt with this kind of compatibility issue in Log4j 2 and we decided to subclass the interface in question. I suggest we do the same here. Not pretty but it is type safe and bullet-proof. Gary On Tue, Aug 29, 2017 at 5:17 AM, sebb wrote: > On 28 August 2017 at 20:07, Gary Gregory

Re: [VOTE] Release Apache Commons CSV 1.5 based on RC1

2017-08-30 Thread Gary Gregory
Nit: the CONTRIBUTING.md is not in the src zip. We should also add a BUILDING.md. Gary On Tue, Aug 29, 2017 at 10:09 PM, Bruno P. Kinoshita < brunodepau...@yahoo.com.br.invalid> wrote: > Hi Oliver, > > Thank you for taking your time to review this RC! > > Interesting, never paid much attention t

Re: Release Commons Jelly 1.0.1 Based on RC2

2017-08-30 Thread Gary Gregory
The src zip contains a "_MAXOSX" folder. Surely we do not need that. Gary On Tue, Aug 29, 2017 at 8:18 AM, Rob Tompkins wrote: > Hello, > > Commons Jelly 1.0.1 R2 is available for review here: > https://dist.apache.org/repos/dist/dev/commons/jelly (svn revision > 21342) > > Note, if anyone ne

[CANCEL] Release Commons Jelly 1.0.1 Based on RC2

2017-08-30 Thread Rob Tompkins
> On Aug 30, 2017, at 10:51 AM, Gary Gregory wrote: > > The src zip contains a "_MAXOSX" folder. Surely we do not need that. Agreed. Cancelling. I will roll an RC3 in the coming days. My vote is a -1. > > Gary > > On Tue, Aug 29, 2017 at 8:18 AM, Rob Tompkins wrote: > >> Hello, >> >> Comm

Re: [CANCEL] Release Commons Jelly 1.0.1 Based on RC2

2017-08-30 Thread Gary Gregory
Hi Rob, Since we are re-rolling, may I ask that the release notes be updated since they currently state "This is the first stable release of Jelly", which it is now the second. Also, should we add a BUILDING.md and a CONTRIBUTING.md? Since there are some old tools at play, better building docs w

Re: [CANCEL] Release Commons Jelly 1.0.1 Based on RC2

2017-08-30 Thread Rob Tompkins
> On Aug 30, 2017, at 11:20 AM, Gary Gregory wrote: > > Hi Rob, > > Since we are re-rolling, may I ask that the release notes be updated since > they currently state "This is the first stable release of Jelly", which it > is now the second. > > Also, should we add a BUILDING.md and a CONTRIBU

Re: [BCEL] Breaking changes in 6.1 needed by Tomcat (?)

2017-08-30 Thread Benedikt Ritter
Hi, looking through the JavaDoc comments in the Visitor interface, it looks like we added additional methods in the 6.0 release. 6.0 was a major release but without package and maven coords change. Seems like we accepted this kind of changes for 6.0. I’m not sure what point I’m trying to make h

Re: [BCEL] Releasing 6.1

2017-08-30 Thread Benedikt Ritter
Hello, I’ve committed the patches. BCEL-286 [1] can not be applied to trunk because it updates a file that does not exist. Cheers, Benedikt [1] https://issues.apache.org/jira/browse/BCEL-286 > Am 29.08.2017 um 00:39 schrieb Mark Roberts : > >

[GitHub] commons-bcel pull request #12: BCEL-276 LocalVariableTypeTable is not update...

2017-08-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-bcel/pull/12 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] commons-dbutils pull request #2: This is not a real pull request, just give ...

2017-08-30 Thread drinkjava2
GitHub user drinkjava2 opened a pull request: https://github.com/apache/commons-dbutils/pull/2 This is not a real pull request, just give some suggestions Hi there, This is not a real pull request, because I can't find somewhere to open an issue for DbUtils, so just use a pull r

[GitHub] commons-dbutils issue #2: This is not a real pull request, just give some su...

2017-08-30 Thread britter
Github user britter commented on the issue: https://github.com/apache/commons-dbutils/pull/2 The bug tracker is at https://issues.apache.org/jira/browse/DBUTILS. Please see our README.md. If you want to propose changes, please make them minimal and focused (see our CONTRIBUTING.md). T

[GitHub] commons-dbutils issue #2: This is not a real pull request, just give some su...

2017-08-30 Thread bindul
Github user bindul commented on the issue: https://github.com/apache/commons-dbutils/pull/2 @drinkjava2 DBUtils uses Apache JIRA for issue tracking at https://issues.apache.org/jira/projects/DBUTILS --- If your project is set up for it, you can reply to this email and have your reply

Re: [All][Math] New component: "Commons Geometry"?

2017-08-30 Thread Gilles
On Wed, 30 Aug 2017 15:28:49 +0200, Emmanuel Bourg wrote: Le 21/08/2017 à 21:41, Gary Gregory a écrit : What about this for a compromise: create Commons Math 5 as a multi-module project and bring in as submodules only the newly minted components and whatever gets spun out of Math 3/4. +1 for

[GitHub] commons-dbutils issue #2: This is not a real pull request, just give some su...

2017-08-30 Thread drinkjava2
Github user drinkjava2 commented on the issue: https://github.com/apache/commons-dbutils/pull/2 I have no time to register on Apache only for post a suggestion. It's weird you disabled github issue but worked everyday on here. DbUtils is perfect, but the biggest problem is, may man

Re: [BCEL] Breaking changes in 6.1 needed by Tomcat (?)

2017-08-30 Thread sebb
On 30 August 2017 at 20:11, Benedikt Ritter wrote: > Hi, > > looking through the JavaDoc comments in the Visitor interface, it looks like > we added additional methods in the 6.0 release. 6.0 was a major release but > without package and maven coords change. Seems like we accepted this kind of