Re: [VOTE] Release Compress 1.10 based on RC2

2015-01-30 Thread sebb
On 31 January 2015 at 00:10, sebb wrote: > On 30 January 2015 at 15:41, sebb wrote: >> I've just had a look at the new class ZCompressorInputStream. > > Sorry, I meant LZWInputStream Which I just realised is mainly a copy of the internal version. Pity the SVN history does not show this. Given t

Re: [VOTE] Release Compress 1.10 based on RC2

2015-01-30 Thread sebb
On 30 January 2015 at 15:41, sebb wrote: > I've just had a look at the new class ZCompressorInputStream. Sorry, I meant LZWInputStream > This has lots of mutable protected fields. > > For example, > > protected int clearCode = -1; > protected int codeSize = 9; > > clearCode has a protect

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread sebb
On 30 January 2015 at 17:34, Emmanuel Bourg wrote: > Le 30/01/2015 17:48, Gary Gregory a écrit : > >> Well, we cannot break BC in a minor release for a public class. Are we >> sure BC breaks? >> >> If we want to make this internal package public, it needs to be copied to >> the new location, not

Re: [VOTE] Release Compress 1.10 based on RC2

2015-01-30 Thread Bruno P. Kinoshita
+1 not binding. Build works fine with: Maven home: /opt/apache-maven-3.2.3 Java version: 1.7.0_75, vendor: Oracle Corporation Java home: /usr/lib/jvm/java-7-openjdk-amd64/jre Default locale: en_US, platform encoding: UTF-8 OS name: "linux", version: "3.13.0-44-generic", arch: "amd64", family: "u

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Emmanuel Bourg
Le 30/01/2015 17:48, Gary Gregory a écrit : > Well, we cannot break BC in a minor release for a public class. Are we > sure BC breaks? > > If we want to make this internal package public, it needs to be copied to > the new location, not moved. Then ZCompressorInputStream would still extend > the

Re: [VOTE] Release Compress 1.10 based on RC2

2015-01-30 Thread sebb
I've just had a look at the new class ZCompressorInputStream. This has lots of mutable protected fields. For example, protected int clearCode = -1; protected int codeSize = 9; clearCode has a protected setter, but no getter. What's the point of that? As far as I can tell, these are fie

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread sebb
On 30 January 2015 at 16:48, Gary Gregory wrote: > On Fri, Jan 30, 2015 at 6:06 AM, Stefan Bodewig wrote: > >> On 2015-01-30, sebb wrote: >> >> > On 30 January 2015 at 08:20, Stefan Bodewig wrote: >> >> On 2015-01-28, Benedikt Ritter wrote: >> >> >>> - PROPOSAL.txt is not in the src archives. Sh

Re: [ALL] [RDF] Commons RDF approach

2015-01-30 Thread Reto Gmür
On Wed, Jan 28, 2015 at 4:22 PM, Sergio Fernández wrote: > Hi everybody, > > after the intense discussions last weeks, this week we're coming down to > what could be a suitable approach for Commons RDF. > > Summarizing, we understand that the Apache Commons project wants to keep > the communicati

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Benedikt Ritter
Hello Stefan, 2015-01-30 20:41 GMT+01:00 Stefan Bodewig : > On 2015-01-30, Gary Gregory wrote: > > > On Fri, Jan 30, 2015 at 6:06 AM, Stefan Bodewig > wrote: > > >> The superclass of ZCompressorInputStream has changed from one we've > >> removed with the old __internal__ package to a new one in

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Stefan Bodewig
On 2015-01-30, Gary Gregory wrote: > On Fri, Jan 30, 2015 at 6:06 AM, Stefan Bodewig wrote: >> The superclass of ZCompressorInputStream has changed from one we've >> removed with the old __internal__ package to a new one in the now >> supported lzw package. > Well, we cannot break BC in a minor

Re: [ALL] [RDF] Commons RDF approach

2015-01-30 Thread Benedikt Ritter
2015-01-30 15:02 GMT+01:00 Sergio Fernández : > On 30/01/15 14:36, Stian Soiland-Reyes wrote: > >> I hope you would still feel welcome to become an unofficial mentor >> (teaching us the "Apache Commons way" perhaps) even if you haven't got the >> right badges (yet). :) >> > > Well, I'd like to hav

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Gary Gregory
On Fri, Jan 30, 2015 at 6:06 AM, Stefan Bodewig wrote: > On 2015-01-30, sebb wrote: > > > On 30 January 2015 at 08:20, Stefan Bodewig wrote: > >> On 2015-01-28, Benedikt Ritter wrote: > > >>> - PROPOSAL.txt is not in the src archives. Should it be? > > >> This is only there for historical reason

Re: [ALL] [RDF] Commons RDF approach

2015-01-30 Thread Sergio Fernández
On 30/01/15 14:36, Stian Soiland-Reyes wrote: I hope you would still feel welcome to become an unofficial mentor (teaching us the "Apache Commons way" perhaps) even if you haven't got the right badges (yet). :) Well, I'd like to have Benedikt as actual mentor. It happened many times that a pro

Re: [ALL] [RDF] Commons RDF approach

2015-01-30 Thread Stian Soiland-Reyes
I hope you would still feel welcome to become an unofficial mentor (teaching us the "Apache Commons way" perhaps) even if you haven't got the right badges (yet). :) Other incubator projects have also had unofficial mentors - it's a good way into Incubator PMC. On 29 Jan 2015 14:29, "Benedikt Ritt

[VOTE] Release Compress 1.10 based on RC2

2015-01-30 Thread Stefan Bodewig
OK, next attempt. Compress 1.10 RC2 is available for review here: https://dist.apache.org/repos/dist/dev/commons/compress/ (svn revision 7884) Maven artifacts are here: https://repository.apache.org/content/repositories/orgapachecommons-1081/org/apache/commons/commons-compress/1.10/

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Stefan Bodewig
On 2015-01-30, sebb wrote: > On 30 January 2015 at 08:20, Stefan Bodewig wrote: >> On 2015-01-28, Benedikt Ritter wrote: >>> - PROPOSAL.txt is not in the src archives. Should it be? >> This is only there for historical reasons, I don't even think the other >> components still have a file like t

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread sebb
On 30 January 2015 at 08:20, Stefan Bodewig wrote: > On 2015-01-28, Benedikt Ritter wrote: > >> - PROPOSAL.txt is not in the src archives. Should it be? > > This is only there for historical reasons, I don't even think the other > components still have a file like this. I've removed it completely

Re: [ALL] [RDF] Commons RDF approach

2015-01-30 Thread Andy Seaborne
On 30/01/15 07:45, Sergio Fernández wrote: On 29/01/15 15:28, Benedikt Ritter wrote: So maybe it's better if someone else steps up (we have people here who meet the necessary requirements), and I will just join the incubator commons rdf community and help you that way. Yes, having you in the c

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-30 Thread Stefan Bodewig
On 2015-01-28, Benedikt Ritter wrote: > - PROPOSAL.txt is not in the src archives. Should it be? This is only there for historical reasons, I don't even think the other components still have a file like this. I've removed it completely. > - there is a typo in the release notes. The first line s