sounds good.
Brett Porter wrote:
Sorry, I think I hadn't yet merged that from artifact trunk. Let me do
that for consistency as it was a little different and includes a test.
On 09/07/2008, at 2:54 AM, [EMAIL PROTECTED] wrote:
Author: jdcasey
Date: Tue Jul 8 09:54:18 2008
New Revision: 674876
URL: http://svn.apache.org/viewvc?rev=674876&view=rev
Log:
Taking another shot at cleaning up the NPE related to wagon beta-3
and the digest that goes null on a transfer error...this time, remove
the checksum observers before verifying the checksum(s).
Modified:
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
Modified:
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
URL:
http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java?rev=674876&r1=674875&r2=674876&view=diff
==============================================================================
---
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
(original)
+++
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
Tue Jul 8 09:54:18 2008
@@ -468,6 +468,16 @@
if ( downloaded )
{
+ if ( sha1ChecksumObserver != null )
+ {
+ wagon.removeTransferListener(
sha1ChecksumObserver );
+ }
+
+ if ( md5ChecksumObserver != null )
+ {
+ wagon.removeTransferListener(
md5ChecksumObserver );
+ }
+
// keep the checksum files from showing up on
the download monitor...
if ( downloadMonitor != null )
{
--
Brett Porter
[EMAIL PROTECTED]
http://blogs.exist.com/bporter/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
John Casey
Developer, PMC Member - Apache Maven (http://maven.apache.org)
Blog: http://www.ejlife.net/blogs/buildchimp/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]