You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ja...@apache.org on 2008/08/13 08:26:51 UTC
svn commit: r685460 -
/maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java
Author: janb
Date: Tue Aug 12 23:26:51 2008
New Revision: 685460
URL: http://svn.apache.org/viewvc?rev=685460&view=rev
Log:
Convert printlns to log messages.
Modified:
maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java
Modified: maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java?rev=685460&r1=685459&r2=685460&view=diff
==============================================================================
--- maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java (original)
+++ maven/sandbox/trunk/mercury/mercury-transport/mercury-transport-http/src/main/java/org/apache/maven/mercury/spi/http/server/StagingBatchFilter.java Tue Aug 12 23:26:51 2008
@@ -152,13 +152,14 @@
if ( !files[i].renameTo( dest ) )
{
ok = false;
- System.err.println("Couldn't rename "+files[i].getCanonicalPath()+" to "+dest.getCanonicalPath());
+ _context.log("Unable to rename file "+files[i].getAbsolutePath()+" to "+dest.getAbsolutePath());
}
}
if ( ok )
{
ok = batchDir.delete();
- System.err.println("Deleting the batch dir = "+ok);
+ if (!ok)
+ _context.log("Unable to delete batch dir "+batchDir.getAbsolutePath());
}
return ok;
}
@@ -196,7 +197,6 @@
*/
private boolean deleteFile( File f )
{
-
if ( f == null )
{
return true;
@@ -204,8 +204,9 @@
if ( f.isFile() )
{
boolean ok = f.delete();
- System.err.println("Resulting of deleting file "+f.getAbsolutePath()+" : "+ok);
- return ok;
+ if (!ok)
+ _context.log("Unable to delete file "+f.getAbsolutePath());
+ return ok;
}
else if ( f.isDirectory() )
{
@@ -222,7 +223,7 @@
if ( !f.delete() )
{
ok = false;
- System.err.println("Result of deleting dir "+f.getAbsolutePath()+" : "+ok);
+ _context.log("Unable to delete dir "+f.getAbsolutePath());
}
return ok;