Re: commons-io git commit: Remove redundant type arguments.

2016-09-22 Thread Kristian Rosenvold
Adding .gitattributes just ensures any future changes will be correct. If there are committed files with incorrect line endings in the repo they will not be automatically be fixed. I fixed the remaining incorrect files in 9e2b2c09732ca596331f7ca34ba4e0f03d70093d. Kristian 2016-09-22 8:30 GMT+02:

[io] commons-io git repository ready

2016-09-01 Thread Kristian Rosenvold
Infra finally got round to fixing https://issues.apache.org/jira/browse/INFRA-12077, and the repository can now be seen at git://git.apache.org/commons-io.git I have checked the repository and it looks ok. Anyone else want to take a look that would be fine :) I will proceed with the documented pr

Re: [CSV] Git Migration

2016-07-08 Thread Kristian Rosenvold
Maybe if you switch the url from commons-cvs to commons-csv :) https://git-wip-us.apache.org/repos/asf?p=commons-csv.git K 2016-07-08 17:40 GMT+02:00 Gary Gregory : > I notice that commons-compress is here: > > https://git-wip-us.apache.org/repos/asf/commons-compress.git > > But there is no > >

Re: [lang] LANG-1247: FastDatePrinter generates Date objects wastefully closes #168

2016-07-06 Thread Kristian Rosenvold
2016-07-06 10:12 GMT+02:00 sebb : > Have a look at the format method. > > It uses Calendar much as below. > >>> +final Calendar c = newCalendar(); >>> +c.setTimeInMillis(millis); >>> +return applyRules(c, buf); Yeah, I see now. Seems like I got lost in the overloads :) Th

Re: [VOTE] Release Apache Commons BCEL 6.0 based on RC7

2016-07-05 Thread Kristian Rosenvold
+1 binding. Kristian 2016-07-05 21:21 GMT+02:00 Oliver Heger : > Because of the failing test I also feel a bit uneasy, but nevertheless I > am +1: It is only a test class, the problem occurs only in a specific > setup, it is already fixed in trunk, and the release is really overdue. > > Thanks >

Re: [lang] LANG-1247: FastDatePrinter generates Date objects wastefully closes #168

2016-07-05 Thread Kristian Rosenvold
2016-07-05 22:44 GMT+02:00 : > LANG-1247: FastDatePrinter generates Date objects wastefully > closes #168 > public StringBuffer format(final long millis, final StringBuffer buf) { > -return format(new Date(millis), buf); > +final Calendar c = newCalendar(); > +c.setTim

Re: Reformatting commons-io to single code style ?

2016-06-27 Thread Kristian Rosenvold
2016-06-27 12:18 GMT+02:00 sebb : > This causes lots of grief when trying to track where a particular > section of code was introduced. Tooling has moved on, I'm really not sure this argument is particularly relevant for a git repository. If I want to track provenance I'd use git pickaxe or simila

Reformatting commons-io to single code style ?

2016-06-27 Thread Kristian Rosenvold
This has probably been discussed a million times before, so I'll keep it short. commons-io has wonderfully inconsistent code style even within individual code files. Once the move to git completes, I'd like to reformat the entire code base (including javadoc) to a single style. I don't really care

Re: [VOTE] Move Apache Commons Primitives to dormant

2016-06-13 Thread Kristian Rosenvold
+1 12. jun. 2016 5.18 p.m. skrev "Benedikt Ritter" : > Hi, > > there hasn't been any activity in the primitives component for a long > while: > > - last release dates back to 2003-11-05 > - using svn log -l 50 > http://svn.apache.org/repos/asf/commons/proper/primitives/trunk I did not > find a sin

Re: [VOTE][LAZY] Migrate Apache Commons IO to git

2016-06-10 Thread Kristian Rosenvold
arman : > +1 > > On Tue, Jun 7, 2016 at 2:37 AM Kristian Rosenvold > wrote: > >> Hello, >> >> I'd like to call a VOTE by LAZY >> consensus for migrating the Apache Commons IO component to git. Please >> object to this vote if you see a problem with th

[VOTE][LAZY] Migrate Apache Commons IO to git

2016-06-06 Thread Kristian Rosenvold
Hello, I'd like to call a VOTE by LAZY consensus for migrating the Apache Commons IO component to git. Please object to this vote if you see a problem with this. Otherwise this vote will be considered as passed after 72 hours from now (10th June 2016, 09:00 CET) Thank you, Kristian

Re: [io] Make requirement Java 7?

2016-04-28 Thread Kristian Rosenvold
2016-04-26 15:39 GMT+02:00 Emmanuel Bourg : > We are not alone :) RedHat is still maintaining OpenJDK 6 and providing > security fixes for things like critical production systems not meant to > be changed every two years. > > Yes, and they are getting paid for it. If they need extended maintenance

Re: [ANN] Apache Commons IO 2.5 Released

2016-04-24 Thread Kristian Rosenvold
22. apr. 2016 15.38 skrev "sebb" : > On 22 April 2016 at 12:29, Benson Margulies wrote: > > I thought I saw it. I guess I was wrong. I wonder what's taking so long? > > Even when Central is working well, it can take several hours (up to a > day) to synch. > > That is partly why the instructions s

Re: [ANN] Apache Commons IO 2.5 Released

2016-04-22 Thread Kristian Rosenvold
Looks like something is amiss with the central sync, the artifacts are at https://repository.apache.org/content/groups/public/commons-io/commons-io/ You probably need to file an issue with mvncentral. While you're at that, you can request in the same issue that all of the "commons" artifacts be s

Re: [io] 2.5 vote...

2016-01-14 Thread Kristian Rosenvold
RC2 was cancelled. I was planning to pick up RC3 RSN, maybe this weekend. Kristian 2016-01-14 23:21 GMT+01:00 Gary Gregory : > Kristian, others? What happened to releasing 2.5 or tallying the RC vote? > > Thank you,Gary > > > -- > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org > Java Pers

Re: [VOTE] Release commons-io RC2

2015-12-26 Thread Kristian Rosenvold
be > fixed or suppressed. I am not sure whether or not the Findbugs > errors indicate actual problems. If they do, these should be fixed; > otherwise suppression should be configured. > > Phil > > > > On 12/23/15 2:14 PM, Kristian Rosenvold wrote: > > We have fix

Re: [VOTE] Release commons-io RC2

2015-12-24 Thread Kristian Rosenvold
58ebaf9e2ab4612845d8 > > Gary > > On Wed, Dec 23, 2015 at 1:14 PM, Kristian Rosenvold > > wrote: > > > We have fixed quite a few bugs and added some significant > > enhancements since commons-io was released, > > so I would like to release commons-io 2.5 &g

[VOTE] Release commons-io RC2

2015-12-23 Thread Kristian Rosenvold
We have fixed quite a few bugs and added some significant enhancements since commons-io was released, so I would like to release commons-io 2.5 commons-io 2.5 RC2 is available for review here: https://dist.apache.org/repos/dist/dev/commons/io/ (svn revision 11732) Maven artifacts are here:

Re: [io] Java 7

2015-12-16 Thread Kristian Rosenvold
I'd still like to make 2.5 a 1.6 release, it shouldnt be that far off. Can you put it on a branch for now ? 16. des. 2015 18.46 skrev "Gary Gregory" : > Hi All: > > I was about to add some APIs to FileUtils that work with Java 7 Paths but > [io] is still on Java 6. > > Are we OK to move to Java 7?

[io] IBM JDK and broken UTF-16 (Related to release 2.5 RC 1)

2015-12-11 Thread Kristian Rosenvold
I've been digging deeply into the IBM JDK 6/7 related breakages on IO RC 2.5. A lot of them can be explained by different capabilities of XML parsers in the different JDKs, and I have come up with a decent heuristic for detecting this and ignoring the tests. There are also a couple of legacy oddb

Re: [VOTE] Release commons-io 2.5 based on RC1

2015-12-09 Thread Kristian Rosenvold
ible". Let me know about the charset and any other problems you might encounter and I'll try to fix them before spinning RC2 (hopefully this weekend). Kristian 2015-11-26 14:20 GMT+01:00 Jörg Schaible : > Kristian Rosenvold wrote: > > > We have fixed quite a few bugs and ad

[VOTE] Release commons-io 2.5 based on RC1 - Cancelled

2015-11-24 Thread Kristian Rosenvold
ev\projects\OpenSource\io\commons-io-2.5-src\test\io\file1-test.txt >> as the parent directory does not exist >> at >> >> org.apache.commons.io.testtools.FileBasedTestCase.createFile(FileBasedTestCase.java:60) >> at >> org.apache.commons.io.FileU

[VOTE] Release commons-io 2.5 based on RC1

2015-11-23 Thread Kristian Rosenvold
We have fixed quite a few bugs and added some significant enhancements since commons-io was released, so I would like to release commons-io 2.5 Foo 2.5 RC1 is available for review here: https://dist.apache.org/repos/dist/dev/commons/io/ (svn revision 11266) Maven artifacts are here:

commons-io 2.5 RC1 tomorrow

2015-11-21 Thread Kristian Rosenvold
I've been through the latest changes on the ValidatingObjectInputStream and I think they look good. RC1 vote starts in approx 20 hrs, if anyone else wants to take a final look. Kristian - To unsubscribe, e-mail: dev-unsubscr...@

Re: SafeObjectInputStream in Commons?

2015-11-13 Thread Kristian Rosenvold
I'd think commons-io too. I have once again startes moves to release the next version so if you're quick I can review & incorporate it. Remember testcases :) Kristian 13. nov. 2015 18.00 skrev "Bertrand Delacretaz" : > Hi, > > I've just subscribed to this list after briefly discussing this with >

Re: [io] When will version 2.5 be released?

2015-10-22 Thread Kristian Rosenvold
It should happen about -90 days but go hung up in summer vacation, which is extensive over here. The release is basically ready (minus one small issue with an added method to an interface), but I'll see if I can bring this back on track "real soon". Watch this space :) Kristian 2015-10-22 20:54

Re: [validator] Inconsistent behavior in UrlValidator

2015-09-27 Thread Kristian Rosenvold
> www.sandglass-software.com > > On 9/27/2015 8:07 AM, Kristian Rosenvold wrote: >> >> 127.0.0.1 is not always the address for localhost. This is a can of worms >> big enough to drive a medium-sized container ship into >> >> Kristian >> 27. sep. 2015 4.13 p.m. skrev

Re: [validator] Inconsistent behavior in UrlValidator

2015-09-27 Thread Kristian Rosenvold
127.0.0.1 is not always the address for localhost. This is a can of worms big enough to drive a medium-sized container ship into Kristian 27. sep. 2015 4.13 p.m. skrev "Benedikt Ritter" : > Hm... since localhost is usually only an alias for 127.0.0.1 it doesn't > really make sense to allow one bu

Re: Version number for next commons-io

2015-09-19 Thread Kristian Rosenvold
2015-09-19 13:58 GMT+02:00 Kristian Rosenvold : > Just to be clear on this, the breach is adding an interface to Oops. The breach is adding a /method/. > > org.apache.commons.io.input.TailerListener#endOfFileReached > and will probably only affect a few users. I'm documenting

Re: Version number for next commons-io

2015-09-19 Thread Kristian Rosenvold
on numbers are cheap :) Kristian 2015-09-19 12:32 GMT+02:00 Kristian Rosenvold : > The next release is binary compatible except for *1* method that has > been added to a (fairly infrequently used) interface. Does that still > mean I should burn 2.5 and go for 3.0. And would t

Version number for next commons-io

2015-09-19 Thread Kristian Rosenvold
The next release is binary compatible except for *1* method that has been added to a (fairly infrequently used) interface. Does that still mean I should burn 2.5 and go for 3.0. And would that be 3.0 or 3.0.0 ? Kristian - To unsu

Re: Missing nexus "stage" permissions....

2015-09-10 Thread Kristian Rosenvold
As you can see, that did the trick ! Thanks a lot. Kristian 2015-09-10 10:18 GMT+02:00 sebb : > On 10 September 2015 at 08:42, Kristian Rosenvold > wrote: >> Nexus seems to think I am missing permissions to stage; >> >> >> 2015-09-10 0

Missing nexus "stage" permissions....

2015-09-10 Thread Kristian Rosenvold
Nexus seems to think I am missing permissions to stage; 2015-09-10 07:39:36 ERROR [9864571-1641733] - com.sonatype.nexus.staging.rest.deploy.DeployResource - Got exception during processing request "PUT http://repository.apache.org/service/local/staging/deploy/maven2/commons-io/commons-io/2.5/com

Re: [VOTE] Move COMPRESS to git

2015-08-23 Thread Kristian Rosenvold
+1 22. aug. 2015 21.29 skrev "Stefan Bodewig" : > Hi all > > more thann half a year ago I promised to call for a vote for migrating > to git as soon as 1.10 has been released. Well that took longer than > expected :-) > > Anyway, here is the vote: > > +1 Move to git > -1 Stick with svn > > vote w

Re: [VOTE] Release Commons Compress Based on RC3

2015-08-18 Thread Kristian Rosenvold
I found the source of the regression, outside c-c. So here's my +1 for the release. Kristian 2015-08-18 12:12 GMT+02:00 Kristian Rosenvold : > I'm investigating a regression in the maven test suites with this > version of c-c (related to zip file attributes). At the moment I do

Re: [VOTE] Release Commons Compress Based on RC3

2015-08-18 Thread Kristian Rosenvold
I'm investigating a regression in the maven test suites with this version of c-c (related to zip file attributes). At the moment I do not know if c-c is the culprit. Please give me an extra day before closing this vote. Kristian 2015-08-16 20:02 GMT+02:00 Oliver Heger : > Building with Java 1.5

Re: commit problem

2015-07-06 Thread Kristian Rosenvold
There may have been some infra related issues on this, you should try again Kristian 2015-07-07 6:11 GMT+02:00 Kristian Rosenvold : > Maybe the committer email address is not your apache.org address ? > > Kristian > 6. jul. 2015 11.52 p.m. skrev "Luc Maisonobe" : >

Re: commit problem

2015-07-06 Thread Kristian Rosenvold
Maybe the committer email address is not your apache.org address ? Kristian 6. jul. 2015 11.52 p.m. skrev "Luc Maisonobe" : > Hi, > > I just get failures when pushing commits to [math] with an error message: > > > pre-receive hook declined > You are not authorized to edit this repository. > > Is

Re: Wlcome to the Apache Commons PMC

2015-07-04 Thread Kristian Rosenvold
Thanks ! Now I just need to get down from the Norwegian mountains. I prefer them i wintertime with lots of snow Kristian 4. jul. 2015 9.31 a.m. skrev "Gary Gregory" : > Hi Kristian, > > Welcome to the Apache Commons PMC! > > Please read the following if you have not done already done so: >

Re: [io] xml parser blues :)

2015-06-24 Thread Kristian Rosenvold
4 June 2015 at 11:57, sebb wrote: > > On 24 June 2015 at 11:09, Kristian Rosenvold > wrote: > >> (sorry, wrong keyboard shortcut) > >> > >> Invalid encoding name "UTF_32BE". > >> org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 42;

Re: [io] xml parser blues :)

2015-06-24 Thread Kristian Rosenvold
(Method.java:483) at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140) Does anyone have any clue as to how I can fix this ? Kristian 2015-06-24 12:08 GMT+02:00 Kristian Rosenvold : > BOMInputStreamTest#testReadXmlWithoutBOMUtf32Be and 3 others fail in > "site" generation but not regular unit tests. &g

[io] xml parser blues :)

2015-06-24 Thread Kristian Rosenvold
BOMInputStreamTest#testReadXmlWithoutBOMUtf32Be and 3 others fail in "site" generation but not regular unit tests. I have identified that the problem is happening because xercesImpl/2.4.0/xercesImpl-2.4.0.jar is creeping in during site build With regular unit tests it picks up rt.jar!/com/sun/org

[io] Preparing for 2.5 release

2015-06-23 Thread Kristian Rosenvold
My personal itch list is getting very short, and includes the following items: @since tag checkup https://issues.apache.org/jira/browse/IO-469 https://issues.apache.org/jira/browse/IO-481 Fix 4 testcases that fail (only) in site generation due to xml parser issues. My ambition is to fix these thi

Re: [io] How picky about backward compatibility ?

2015-06-23 Thread Kristian Rosenvold
2015-06-23 18:08 GMT+02:00 sebb : > On 23 June 2015 at 08:54, Kristian Rosenvold > wrote: > > I think I'll go for the release-notes approach; given that we can find an > > appropriate technical solution. > > > > Right now I'm leaning towards simply changi

Re: svn commit: r1686747 [1/3] - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/ main/java/org/apache/commons/io/filefilter/ main/java/org/apache/commons/io/input/ main/java/org/apac

2015-06-23 Thread Kristian Rosenvold
2015-06-23 18:11 GMT+02:00 sebb : > > > org.apache.commons.io.output.DeferredFileOutputStream#thresholdReached > > possible file handle leak upon exception > > > > Can you update the SVN log message accordingly? > Done > > > > > > >> > >> Also looks like many of the checkstyle fixes are just line

Re: [io] How picky about backward compatibility ?

2015-06-23 Thread Kristian Rosenvold
fine grained control. If one of the existing constructors is used, simply use a new IOException(). Kristian 2015-06-22 20:06 GMT+02:00 Benedikt Ritter : > 2015-06-21 14:34 GMT+02:00 Kristian Rosenvold < > kristian.rosenv...@gmail.com> > : > > > 2015-06-21 14:16 GMT+02:00

Re: Jira permissions gone .. ?

2015-06-22 Thread Kristian Rosenvold
> Benedikt > > 2015-06-22 19:53 GMT+02:00 Benedikt Ritter : > > > Hello Kristian, > > > > sorry for the late reply. I'll have a look. > > > > Benedikt > > > > 2015-06-19 19:06 GMT+02:00 Kristian Rosenvold : > > > >> I seem to h

Re: svn commit: r1686747 [1/3] - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/ main/java/org/apache/commons/io/filefilter/ main/java/org/apache/commons/io/input/ main/java/org/apac

2015-06-21 Thread Kristian Rosenvold
2015-06-22 2:53 GMT+02:00 sebb : > >What was the Findbugs error? > > Best not to mix these in a single commit. > Sorry about that. org.apache.commons.io.output.DeferredFileOutputStream#thresholdReached possible file handle leak upon exception > > Also looks like many of the checkstyle fixes ar

Need permissions to make release....

2015-06-21 Thread Kristian Rosenvold
As already mentioned, I'm missing jira permissions. I'm also missing permissions to add my key to the KEYS file (my key is in https://dist.apache.org/repos/dist/release/maven/KEYS is someone wants to do this) I also suspect that I'll fully be lacking any permissions to publish to repository.apache

Re: svn commit: r1686472 - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/FileUtils.java main/java/org/apache/commons/io/Java7Support.java test/java/org/apache/commons/io/FileUtilsCl

2015-06-21 Thread Kristian Rosenvold
2015-06-21 16:14 GMT+02:00 sebb : > > > But there is no unnecessary unboxing here, hence no bug. > > >Exactly, so the IntelliJ warning is wrong. > > No. I added the redundant call back since that's the way you seem to prefer round here. Kristian

Re: svn commit: r1686472 - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/FileUtils.java main/java/org/apache/commons/io/Java7Support.java test/java/org/apache/commons/io/FileUtilsCl

2015-06-21 Thread Kristian Rosenvold
2015-06-21 14:53 GMT+02:00 sebb : > On 21 June 2015 at 13:24, Kristian Rosenvold > wrote: > > 2015-06-21 13:43 GMT+02:00 sebb : > > > >> > >> > I'm not sure I understand; the equivalent of > >> > > >> > Boolean

Re: [io] How picky about backward compatibility ?

2015-06-21 Thread Kristian Rosenvold
2015-06-21 14:16 GMT+02:00 sebb : > > > Is there a genuine use-case which requires that the same IOE instance > be used? (Other than the unit tests!) > If not, then we could look into relaxing the behaviour (and fixing the > Javadoc) for a future release. > If there is a use-case for always using t

Re: svn commit: r1686472 - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/FileUtils.java main/java/org/apache/commons/io/Java7Support.java test/java/org/apache/commons/io/FileUtilsCl

2015-06-21 Thread Kristian Rosenvold
2015-06-21 13:43 GMT+02:00 sebb : > > > I'm not sure I understand; the equivalent of > > > > Boolean result = (Boolean) isSymbolicLink.invoke(null, path); > > return result.booleanValue(); > > > > Gives a style warning (uneccssary unboxing) in IntelliJ. > > That warning is wrong. > > Unboxing is c

Re: svn commit: r1686472 - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/FileUtils.java main/java/org/apache/commons/io/Java7Support.java test/java/org/apache/commons/io/FileUtilsCl

2015-06-21 Thread Kristian Rosenvold
2015-06-21 13:43 GMT+02:00 sebb : > > Final variables are guaranteed to be safely published across threads. > Is that true for non-final variables if they are established in a static > block? > Yes. All static blocks are guranteed to be visible (all class initialization is basically a synchronized

[io] svn commit r1686512

2015-06-20 Thread Kristian Rosenvold
I reverted some of the package protected -> private changes, since the tests did not even compile with this change. Kristian

[io] How picky about backward compatibility ?

2015-06-19 Thread Kristian Rosenvold
I reverted the fix in http://svn.apache.org/viewvc?view=revision&revision=r1686456 simply because it broke backward compatibility in a subtle way. The BrokenNNStream classes supply the same instance of the IOException on all methods (and there are tests that assert it) Given some of the discussion

Re: svn commit: r1686472 - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/FileUtils.java main/java/org/apache/commons/io/Java7Support.java test/java/org/apache/commons/io/FileUtilsCl

2015-06-19 Thread Kristian Rosenvold
2015-06-19 21:01 GMT+02:00 sebb : > On 19 June 2015 at 19:00, wrote: > > +class Java7Support > > +{ > > + > > +private static final boolean IS_JAVA7; > > + > > +private static Method isSymbolicLink; > > + > > +private static Method delete; > > + > > +private static Method toPath;

Re: [jira] [Created] (IO-481) org.apache.commons.io.FileUtils#waitFor waits too long

2015-06-19 Thread Kristian Rosenvold
Forwarded message -- > From: Kristian Rosenvold (JIRA) > Date: Fri, Jun 19, 2015 at 11:20 AM > Subject: [jira] [Created] (IO-481) org.apache.commons.io.FileUtils#waitFor > waits too long > To: iss...@commons.apache.or

Jira permissions gone .. ?

2015-06-19 Thread Kristian Rosenvold
I seem to have lost my jira permissions to assign issues to myself and close them. I believe I had them not so long ago... ? Kristian (krosenvold)

Re: svn commit: r1686260 [1/5] - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/ main/java/org/apache/commons/io/input/ main/java/org/apache/commons/io/monitor/ test/java/org/apache/

2015-06-18 Thread Kristian Rosenvold
; > Gary > > On Thu, Jun 18, 2015 at 10:04 AM, Kristian Rosenvold < > krosenv...@apache.org> > wrote: > > > I can do the release, but I'm not entirely sure I'd have the necessary > > priveiliges (since I'm not even a committer in commons). Then

Re: svn commit: r1686260 [1/5] - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/ main/java/org/apache/commons/io/input/ main/java/org/apache/commons/io/monitor/ test/java/org/apache/

2015-06-18 Thread Kristian Rosenvold
:00 Gary Gregory : > [io] needs a release bad, it's been way to long. I can live with 1.6 for a > release if is soon. Then switch to Java 7. > > Are you willing to RM? > > Gary > > On Thu, Jun 18, 2015 at 9:28 AM, Kristian Rosenvold > > wrote: > > > We (

Re: svn commit: r1686260 [1/5] - in /commons/proper/io/trunk/src: main/java/org/apache/commons/io/ main/java/org/apache/commons/io/input/ main/java/org/apache/commons/io/monitor/ test/java/org/apache/

2015-06-18 Thread Kristian Rosenvold
We (maven) /just/ managed to switch to 1.6 baseline. So I'm +1 for making at least one more release with 1.6 ;) Kristian 2015-06-18 18:16 GMT+02:00 Gary Gregory : > On Thu, Jun 18, 2015 at 8:57 AM, wrote: > > > Author: krosenvold > > Date: Thu Jun 18 15:57:56 2015 > > New Revision: 1686260 > >

Re: [io] Is commons-io dormant?

2015-05-12 Thread Kristian Rosenvold
2015-05-12 23:09 GMT+02:00 Kervin Pierre : > I'd like to help if possible. What would be involved? > There is an abundance of issues in Jira that should be triaged somewhat before pushing a release. You can fix issues without patches, review existing patches and/or comment on issues. Looking thr

Re: [io] Is commons-io dormant?

2015-05-12 Thread Kristian Rosenvold
I'll spend some time this weekend iterating over issues for io and apply patches that appear appropriate and uncontroversial. Kristian 2015-05-11 7:57 GMT+02:00 Benedikt Ritter : > I'll be on vacation the next week. I could do it in two weeks at the > earliest. > > Benedikt > > 2015-05-11 5:26

Re: [LANG] Add ThreadUtils

2015-04-12 Thread Kristian Rosenvold
If think later (in 2025 :-) you just make interface ThreadPredicate extend java.util.function.Predicate. Kristian 2015-04-12 13:14 GMT+02:00 Benedikt Ritter : > Hi, > > there is currently a discussion on github about the addition of a low level > utility class which helps to retrieve Threads [1

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-28 Thread Kristian Rosenvold
2015-01-28 19:30 GMT+01:00 Benedikt Ritter : > Hello Stefan, > > - I've build this RC with Maven 3.2.5 and Java 6, 7, 8 and 9 EA. Java 6, 7 > and 8 work fine (although the site can not be build with Java 8 due to > doclint). Java 9 fails, because it does not support target option 1.5. I just fixed

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-27 Thread Kristian Rosenvold
2015-01-27 15:24 GMT+01:00 Gary Gregory : > I'd prefer a clean RC to give a positive vote. So I'll abstain for now. That seems to be the norm :) K - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional comm

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-27 Thread Kristian Rosenvold
I've run through all of our tests and the release looks good. The testcase should fail 1 in 1000 times as is and has been fixed. I'm +1 on the release as such (but we'd normally recut or leave this decision to the RM). Krisian 2015-01-27 7:35 GMT+01:00 Kristian Rosenv

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-26 Thread Kristian Rosenvold
ion back to this problem. > > Gary > > On Mon, Jan 26, 2015 at 4:52 PM, Kristian Rosenvold > wrote: > >> Testcase fixed in r1654901 >> >> Kristian >> >> >> 2015-01-26 22:37 GMT+01:00 Kristian Rosenvold < >> kristian.rosenv

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-26 Thread Kristian Rosenvold
Testcase fixed in r1654901 Kristian 2015-01-26 22:37 GMT+01:00 Kristian Rosenvold : > 2015-01-26 22:27 GMT+01:00 Gary Gregory : >> Tests: >> >> Running: 'mvn clean site' gave me >> >> Failed tests: >> ZipTestCase.testCopyRawZip64EntryFromFil

Re: [VOTE] Release Compress 1.10 Based on RC1

2015-01-26 Thread Kristian Rosenvold
2015-01-26 22:27 GMT+01:00 Gary Gregory : > Tests: > > Running: 'mvn clean site' gave me > > Failed tests: > ZipTestCase.testCopyRawZip64EntryFromFile:361->assertSameFileContents:414 > arrays first differed at element [10]; expected:<-16> but was:<-15> The problem appears to be that time does no

Re: [compress] Preparations for 1.10

2015-01-24 Thread Kristian Rosenvold
After an intense few minutes discussing the color of the bike shed with myself (package name) I moved the zip-unspecific parallel stuff to org.apache.commons.compress.parallel in r1654572. This concludes the changes Emmanuel suggested. I have already responded to a few of the comments that did not

Re: [compress] Preparations for 1.10

2015-01-23 Thread Kristian Rosenvold
2015-01-23 11:52 GMT+01:00 Emmanuel Bourg : > - PasswordRequiredException: the exception is in the sevenz package, do > we want to move it elsewhere so it can be used later for other formats > like zip (if that makes sense). Makes sense, where should we put that ? > - there are some missing @sin

Re: [compress] Preparations for 1.10

2015-01-23 Thread Kristian Rosenvold
Thanks for lots of great comments ! Just a few comments about some of the items; Regarding the *Supplier interfaces, the basic idea is that once commons-compress reaches jdk 1.8 language level (somewhere in the late 2030's :-) some of such interfaces may be modified to extend java.util.function.Su

Re: [ALL] JIRA permissions for Kristian Rosenvold

2015-01-23 Thread Kristian Rosenvold
Sorry; all is ok. The asf jira has a slightly different config than the one I'm used to. Kristian 2015-01-23 10:46 GMT+01:00 Kristian Rosenvold : > Can you check that you did the right thing, Mark ? It seems like I > have even less permissions now; I can't even edit "o

Re: [ALL] JIRA permissions for Kristian Rosenvold

2015-01-23 Thread Kristian Rosenvold
Can you check that you did the right thing, Mark ? It seems like I have even less permissions now; I can't even edit "own issues" any more...? Kristian 2015-01-23 10:13 GMT+01:00 Mark Thomas : > On 23/01/2015 09:10, Stefan Bodewig wrote: >> On 2015-01-23, luc wrote: >> >>> Le 2015-01-23 04:00, s

Re: [compress] ConcurrentJarCreator

2015-01-20 Thread Kristian Rosenvold
Yup I'm done, added a small blurb in changes.xml in r1653282. Btw it still seems like my JIRA karma is a bit weak ? Kristian 2015-01-20 16:00 GMT+01:00 Stefan Bodewig : > On 2015-01-12, Kristian Rosenvold wrote: > >> We had somewhat of a discussion regarding this class on the

Re: [ALL] Too much traffic on the "dev" ML

2015-01-16 Thread Kristian Rosenvold
I'd say the problem is probably that you have too little mailing list traffic incoming. Subscribe to a few more and you /will/ have to start making inbox rules :) Kristian (Who had the dubious honor of receiving more email than the rest of my company altogether last year - 20 people)

Re: svn commit: r1650632 - in /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip: ZipArchiveEntry.java ZipArchiveOutputStream.java

2015-01-12 Thread Kristian Rosenvold
I see you fixed this. Thanks. Kristian 2015-01-11 19:58 GMT+01:00 Stefan Bodewig : > On 2015-01-09, wrote: > >> @@ -336,20 +338,48 @@ public class ZipArchiveEntry extends jav >> * @since 1.1 >> */ >> public ZipExtraField[] getExtraFields(boolean includeUnparseable) { >>+

[compress] ConcurrentJarCreator

2015-01-12 Thread Kristian Rosenvold
We had somewhat of a discussion regarding this class on the maven dev list over the weekend, some people wanted this code inside commons-compress: Code is here: https://github.com/krosenvold/plexus-archiver/blob/2.x/src/main/java/org/codehaus/plexus/archiver/zip/ConcurrentJarCreator.java I have

Re: [compress] Finished with my changes

2015-01-10 Thread Kristian Rosenvold
2015-01-10 15:29 GMT+01:00 Stefan Bodewig : > what about the (unrelated) COMPRESS-290? I'll see what I can do. Should be simple compared to the other stuff I've been dealing with :) >> and I'll just be running through all the maven test cases tonight. >> Unless bugs pop up I will not be making any

[compress] Preview release: Plexus Archiver (for maven) Parallel ZIP edition

2015-01-10 Thread Kristian Rosenvold
I just released plexus-archiver version 2.10-beta-1 to maven central. This is a "technology preview" of the multithreaded Zip feature I have been adding to commons-compress for the last few weeks, and will basically use all available CPU cores when compressing the archive. To test/use this featur

[compress] Finished with my changes

2015-01-09 Thread Kristian Rosenvold
I am finished with all my "itches" for the next version of c-compress and I'll just be running through all the maven test cases tonight. Unless bugs pop up I will not be making any further significant changes. I see Stefan has already been doing minor adjustments to some of my changes. If there's

Re: Wiki Accounts

2015-01-06 Thread Kristian Rosenvold
I just created "KristianRosenvold" account on the wiki. Kristian 2015-01-06 20:46 GMT+01:00 sebb : > On 6 January 2015 at 19:19, Stefan Bodewig wrote: >> Hi >> >> Luc and Kristian have asked to be added as contributors to the Wiki. I >> seem to be able to grant that, but need your wiki names in

Re: [VOTE][LAZY] Migrate Apache Commons Lang to git

2015-01-06 Thread Kristian Rosenvold
It would appear I do not have access to the commons wiki. If someone could grant me wiki privs (and Jira at the same time), I can do some basics for the git wiki page. Kristian 2015-01-06 18:01 GMT+01:00 sebb : > -1 > > When the first components moved to Git, the agreement was that these > wou

Re: [compress] zip64 writing seems to be broken

2015-01-05 Thread Kristian Rosenvold
2015-01-05 15:12 GMT+01:00 sebb : > On 5 January 2015 at 13:43, Stefan Bodewig wrote: >> On 2015-01-04, Kristian Rosenvold wrote: >> >>> Most surprising to me is that it seems like the overhead of lots of >>> small calls to RandomAccessFile.write seems to be a l

Re: [compress] zip64 writing seems to be broken

2015-01-04 Thread Kristian Rosenvold
Great stuff ! > getBytesWritten vs getTotalBytesWritten - svn revision 1649322 > > Maybe we should rename getBytesWritten to something like > getBytesWrittenForLastEntry to make the difference more obvious? I had hard time keeping those "written" counters correct - which you found out :) I renam

Re: [compress] zip64 writing seems to be broken

2015-01-04 Thread Kristian Rosenvold
Not entirely unsurprisingly this broken in r1648585. I'll try to understand it tonight Kristia 2015-01-04 11:37 GMT+01:00 Stefan Bodewig : > On 2015-01-04, Stefan Bodewig wrote: > >> I'll try to reproduce this as a unit test for compress later today, > > svn revision 1649312 - run via > > $ mvn

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2015-01-02 Thread Kristian Rosenvold
gt; On 2014-12-31, Kristian Rosenvold wrote: > >> On a related note, I just added the *last* substantial change I intend >> to do. I will do a tweak or two, and I'm sure that last class will >> trigger a truckload of comments, even just the name of the class:) > > I'm

Re: svn commit: r1648704 - in /commons/proper/compress/trunk/src: main/java/org/apache/commons/compress/archivers/zip/ test/java/org/apache/commons/compress/archivers/zip/

2015-01-02 Thread Kristian Rosenvold
2015-01-02 16:22 GMT+01:00 Stefan Bodewig : > On 2014-12-31, wrote: > ... all else fixed in r1649128.. >>static ScatterZipOutputStream >> createDeferred(ScatterGatherBackingStoreSupplier >> scatterGatherBackingStoreSupplier) > > would then be an instance method. Do you need it to be publi

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2014-12-31 Thread Kristian Rosenvold
2014-12-31 11:50 GMT+01:00 Stefan Bodewig : > On 2014-12-30, Kristian Rosenvold wrote: >> The whole ZipArchiveOutputStream class reminds me of a few of the >> 3000+ LOC java classes I refactored in maven core; sometimes the only >> acceptable solution is to extract *all*

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2014-12-30 Thread Kristian Rosenvold
2014-12-29 15:05 GMT+01:00 Stefan Bodewig : > On 2014-12-29, Kristian Rosenvold wrote: > >> The refactoring os ZipArchiveOutputStream to use StreamCompressor is now >> done in >> the branch https://github.com/krosenvold/commons-compress > > Some c

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2014-12-28 Thread Kristian Rosenvold
The refactoring os ZipArchiveOutputStream to use StreamCompressor is now done in the branch https://github.com/krosenvold/commons-compress As refactorings come it doesn't "feel" too good (extracting all the methods to create zip headers to a different class would probably be a refactoring with a l

[compress] Refactoring to interfaces in Scatter* logic

2014-12-28 Thread Kristian Rosenvold
Stefan, I looked at your changes in the github repo https://github.com/bodewig/commons-compress/commits/scatter-backing-store I think they look great. Please commit them :) Kristian - To unsubscribe, e-mail: dev-unsubscr...@comm

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2014-12-28 Thread Kristian Rosenvold
2014-12-28 11:35 GMT+01:00 Stefan Bodewig : > On 2014-12-26, Kristian Rosenvold wrote: >> A) Think about manifest handling in genreal > > This applies to plexus since CC isn't doing any manifest handling at all > right now. Yes, I'm thinking along the lines of just mak

Re: [compress] Importance of retaining exact compatibility for ZipArchiveOutputStream ?

2014-12-26 Thread Kristian Rosenvold
Yup; I'm taking care of the duplication in trunk on my github fork. The other interesting branch to look at is the somewhat stale "concurrentSupport" branch and in particular the class ConcurrentZipCreator. This is my primary goal, I just went off on a round of yak-shaving first. I seem to have fin

Re: svn commit: r1646531 - in /commons/proper/compress/trunk: ./ src/main/java/org/apache/commons/compress/archivers/zip/ src/test/java/org/apache/commons/compress/ src/test/java/org/apache/commons/co

2014-12-23 Thread Kristian Rosenvold
I'll put taking a look at addRawArchiveEntry with ZipEntry on my todo list. I have quite a few threads converging into the last class already; but everything should be done in a few days now. copyRawEntries could definitely be somewhere else. Originally I implemented it on ZipArchiveOutputStream,

Re: svn commit: r1647329 - in /commons/proper/compress/trunk/src: main/java/org/apache/commons/compress/archivers/zip/ test/java/org/apache/commons/compress/archivers/ test/java/org/apache/commons/com

2014-12-23 Thread Kristian Rosenvold
Thanks for the comments, fixed in r1647582. Is there such a thing as a "commons" IntelliJ code style file ? Kristian 2014-12-23 11:46 GMT+01:00 sebb : > On 22 December 2014 at 15:24, wrote: >> Author: krosenvold >> Date: Mon Dec 22 15:24:02 2014 >> New Revision: 1647329 >> >> URL: http://svn.

Re: [io] Make org.apache.commons.io.output.ByteArrayOutputStream resversible ?

2014-12-23 Thread Kristian Rosenvold
doh. That's what I get for looking at released versions instead of reading code at trunk like real grownups do. Thanks, Kristian 2014-12-23 11:08 GMT+01:00 sebb : > On 23 December 2014 at 07:55, Kristian Rosenvold > wrote: >> I'd like to make a public (synchr

Re: svn commit: r1647329 - in /commons/proper/compress/trunk/src: main/java/org/apache/commons/compress/archivers/zip/ test/java/org/apache/commons/compress/archivers/ test/java/org/apache/commons/com

2014-12-23 Thread Kristian Rosenvold
It depends :) All the commits I did so far represent distinct pieces of functionality that someone could choose to use by itself, so if c-compress was released tomorrow they'd be usable. That being said, I'm still polishing the last high-level class that provides the consumer-friendlier stuff to c

  1   2   >