Re: svn commit: r1480300 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java

2013-05-08 Thread sebb
On 8 May 2013 17:45, Gary Gregory wrote: > On Wed, May 8, 2013 at 12:14 PM, sebb wrote: > >> On 8 May 2013 15:51, Gary Gregory wrote: >> > Sebb, >> > >> > Are you suggesting making copyInputStreamToFile(InputStream, File, >> boolean) >> > private and adding a public method like copyToFile(InputS

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Luc Maisonobe
Gary Gregory a écrit : >On Wed, May 8, 2013 at 4:32 PM, Gary Gregory >wrote: > >> In the Maven output I see: >> >> [INFO] Skipping JaCoCo execution due to missing execution data file >> > >Which I from running "mvn clean site". Perhaps should you try "mvn clean test site"? I did not run cle

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Gary Gregory
On Wed, May 8, 2013 at 4:32 PM, Gary Gregory wrote: > In the Maven output I see: > > [INFO] Skipping JaCoCo execution due to missing execution data file > Which I from running "mvn clean site". Gary > > Gary > > > On Wed, May 8, 2013 at 4:18 PM, Luc Maisonobe wrote: > >> >> >> >> Gary Gregor

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Gary Gregory
In the Maven output I see: [INFO] Skipping JaCoCo execution due to missing execution data file Gary On Wed, May 8, 2013 at 4:18 PM, Luc Maisonobe wrote: > > > > Gary Gregory a écrit : > > >On Wed, May 8, 2013 at 3:58 PM, Luc Maisonobe > >wrote: > > > >> Le 08/05/2013 21:11, Gary Gregory a é

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Luc Maisonobe
Gary Gregory a écrit : >On Wed, May 8, 2013 at 3:58 PM, Luc Maisonobe >wrote: > >> Le 08/05/2013 21:11, Gary Gregory a écrit : >> > I updated my local commons-io to 29-SNAPSHOT and I do not see a >coverage >> > report. >> > >> > How is that fixed? >> >> Well, I did not have to do anything for

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Gary Gregory
On Wed, May 8, 2013 at 3:58 PM, Luc Maisonobe wrote: > Le 08/05/2013 21:11, Gary Gregory a écrit : > > I updated my local commons-io to 29-SNAPSHOT and I do not see a coverage > > report. > > > > How is that fixed? > > Well, I did not have to do anything for [math]. Did you install the > snapshot

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Luc Maisonobe
Le 08/05/2013 21:58, Luc Maisonobe a écrit : > Le 08/05/2013 21:11, Gary Gregory a écrit : >> I updated my local commons-io to 29-SNAPSHOT and I do not see a coverage >> report. >> >> How is that fixed? > > Well, I did not have to do anything for [math]. Did you install the > snapshot locally with

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Luc Maisonobe
Le 08/05/2013 21:11, Gary Gregory a écrit : > I updated my local commons-io to 29-SNAPSHOT and I do not see a coverage > report. > > How is that fixed? Well, I did not have to do anything for [math]. Did you install the snapshot locally with "mvn install"? Don't you have the skipReports property

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Gary Gregory
I updated my local commons-io to 29-SNAPSHOT and I do not see a coverage report. How is that fixed? Gary On Fri, Apr 5, 2013 at 7:44 AM, luc wrote: > Le 2013-04-05 00:10, Gary Gregory a écrit : > >> On Thu, Apr 4, 2013 at 4:03 PM, sebb wrote: >> >> CP 28 moved Cobertura to a profile called

Re: svn commit: r1480351 - in /commons/proper/commons-parent/trunk: RELEASE-NOTES.txt pom.xml

2013-05-08 Thread Gary Gregory
I just did ;) Gary On May 8, 2013, at 14:38, Luc Maisonobe wrote: > Le 08/05/2013 18:52, Gary Gregory a écrit : >> Why not use the current version of maven-surefire-plugin? > > I did not change that. We could update it. > > Luc > >> >> Gary >> >> >> On Wed, May 8, 2013 at 12:49 PM, wrote: >> >

Re: svn commit: r1480351 - in /commons/proper/commons-parent/trunk: RELEASE-NOTES.txt pom.xml

2013-05-08 Thread Luc Maisonobe
Le 08/05/2013 18:52, Gary Gregory a écrit : > Why not use the current version of maven-surefire-plugin? I did not change that. We could update it. Luc > > Gary > > > On Wed, May 8, 2013 at 12:49 PM, wrote: > >> Author: luc >> Date: Wed May 8 16:49:20 2013 >> New Revision: 1480351 >> >> URL

Re: [ALL] Cobertura and Parent POM

2013-05-08 Thread Luc Maisonobe
Hi all, Le 05/04/2013 22:49, Gary Gregory a écrit : > On Fri, Apr 5, 2013 at 4:30 PM, Phil Steitz wrote: > >> On 4/4/13 1:03 PM, sebb wrote: >>> CP 28 moved Cobertura to a profile called "reporting". >>> >>> The profile was activated by default, but could be disabled by using >>> >>> -DskipRepor

Re: svn commit: r1480351 - in /commons/proper/commons-parent/trunk: RELEASE-NOTES.txt pom.xml

2013-05-08 Thread Gary Gregory
Why not use the current version of maven-surefire-plugin? Gary On Wed, May 8, 2013 at 12:49 PM, wrote: > Author: luc > Date: Wed May 8 16:49:20 2013 > New Revision: 1480351 > > URL: http://svn.apache.org/r1480351 > Log: > replaced cobertura with jacoco for coverage reports > > Modified: >

Re: svn commit: r1480300 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java

2013-05-08 Thread Gary Gregory
On Wed, May 8, 2013 at 12:14 PM, sebb wrote: > On 8 May 2013 15:51, Gary Gregory wrote: > > Sebb, > > > > Are you suggesting making copyInputStreamToFile(InputStream, File, > boolean) > > private and adding a public method like copyToFile(InputStream, File) > that > > calls it with false? > > No

Re: svn commit: r1480300 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java

2013-05-08 Thread sebb
On 8 May 2013 15:51, Gary Gregory wrote: > Sebb, > > Are you suggesting making copyInputStreamToFile(InputStream, File, boolean) > private and adding a public method like copyToFile(InputStream, File) that > calls it with false? Not exactly. I'm saying that it does not make sense to me for that

Re: svn commit: r1480300 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java

2013-05-08 Thread Gary Gregory
Sebb, Are you suggesting making copyInputStreamToFile(InputStream, File, boolean) private and adding a public method like copyToFile(InputStream, File) that calls it with false? Gary On Wed, May 8, 2013 at 10:45 AM, sebb wrote: > On 8 May 2013 15:36, wrote: > > Author: ggregory > > Date: W

Re: svn commit: r1480300 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java

2013-05-08 Thread sebb
On 8 May 2013 15:36, wrote: > Author: ggregory > Date: Wed May 8 14:36:32 2013 > New Revision: 1480300 > > URL: http://svn.apache.org/r1480300 > Log: > > Add FileUtils.copyInputStreamToFile API with option to leave the > source open. > See copyInputStreamToFile(final InputStrea